d5/b7a8bc6b37df3af9dec676e72e5ae1ed9c97b0

   1 Return-Path: <bremner@tesseract.cs.unb.ca>\r
   2 X-Original-To: notmuch@notmuchmail.org\r
   3 Delivered-To: notmuch@notmuchmail.org\r
   4 Received: from localhost (localhost [127.0.0.1])\r
   5  by arlo.cworth.org (Postfix) with ESMTP id 3045A6DE1AEA\r
   6  for <notmuch@notmuchmail.org>; Mon,  6 Apr 2015 05:24:29 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0.467\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.467 tagged_above=-999 required=5 tests=[AWL=0.457, \r
  12  T_HEADER_FROM_DIFFERENT_DOMAINS=0.01] autolearn=disabled\r
  13 Received: from arlo.cworth.org ([127.0.0.1])\r
  14  by localhost (arlo.cworth.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15  with ESMTP id ZplKL_UUhihJ for <notmuch@notmuchmail.org>;\r
  16  Mon,  6 Apr 2015 05:24:27 -0700 (PDT)\r
  17 Received: from mx.xen14.node3324.gplhost.com (gitolite.debian.net\r
  18  [87.98.215.224])\r
  19  by arlo.cworth.org (Postfix) with ESMTPS id 3B7EE6DE1AC6\r
  20  for <notmuch@notmuchmail.org>; Mon,  6 Apr 2015 05:24:27 -0700 (PDT)\r
  21 Received: from remotemail by mx.xen14.node3324.gplhost.com with local (Exim\r
  22  4.80) (envelope-from <bremner@tesseract.cs.unb.ca>)\r
  23  id 1Yf649-000593-8c; Mon, 06 Apr 2015 12:23:45 +0000\r
  24 Received: (nullmailer pid 3280 invoked by uid 1000); Mon, 06 Apr 2015\r
  25  12:22:58 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: David Bremner <david@tethera.net>, notmuch@notmuchmail.org\r
  28 Subject: [Patch v2 1/4] cli: ignore config argument of notmuch_help_command\r
  29 Date: Mon,  6 Apr 2015 21:22:35 +0900\r
  30 Message-Id: <1428322958-2887-2-git-send-email-david@tethera.net>\r
  31 X-Mailer: git-send-email 2.1.4\r
  32 In-Reply-To: <1428322958-2887-1-git-send-email-david@tethera.net>\r
  33 References: <87d23ixnr7.fsf@maritornes.cs.unb.ca>\r
  34  <1428322958-2887-1-git-send-email-david@tethera.net>\r
  35 X-BeenThere: notmuch@notmuchmail.org\r
  36 X-Mailman-Version: 2.1.18\r
  37 Precedence: list\r
  38 List-Id: "Use and development of the notmuch mail system."\r
  39  <notmuch.notmuchmail.org>\r
  40 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  41  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  42 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  43 List-Post: <mailto:notmuch@notmuchmail.org>\r
  44 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  45 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  46  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  47 X-List-Received-Date: Mon, 06 Apr 2015 12:24:29 -0000\r
  48 \r
  49 We call it with NULL at one point anyway, so it needs to work with\r
  50 NULL. Since the only place we use talloc is right before exec, there\r
  51 is no harm in always using NULL.\r
  52 ---\r
  53  notmuch.c | 6 +++---\r
  54  1 file changed, 3 insertions(+), 3 deletions(-)\r
  55 \r
  56 diff --git a/notmuch.c b/notmuch.c\r
  57 index a5b2877..31672c3 100644\r
  58 --- a/notmuch.c\r
  59 +++ b/notmuch.c\r
  60 @@ -177,7 +177,7 @@ exec_man (const char *page)\r
  61  }\r
  62  \r
  63  static int\r
  64 -notmuch_help_command (notmuch_config_t *config, int argc, char *argv[])\r
  65 +notmuch_help_command (unused (notmuch_config_t * config), int argc, char *argv[])\r
  66  {\r
  67      command_t *command;\r
  68      help_topic_t *topic;\r
  69 @@ -202,14 +202,14 @@ notmuch_help_command (notmuch_config_t *config, int argc, char *argv[])\r
  70  \r
  71      command = find_command (argv[0]);\r
  72      if (command) {\r
  73 -       char *page = talloc_asprintf (config, "notmuch-%s", command->name);\r
  74 +       char *page = talloc_asprintf (NULL, "notmuch-%s", command->name);\r
  75         exec_man (page);\r
  76      }\r
  77  \r
  78      for (i = 0; i < ARRAY_SIZE (help_topics); i++) {\r
  79         topic = &help_topics[i];\r
  80         if (strcmp (argv[0], topic->name) == 0) {\r
  81 -           char *page = talloc_asprintf (config, "notmuch-%s", topic->name);\r
  82 +           char *page = talloc_asprintf (NULL, "notmuch-%s", topic->name);\r
  83             exec_man (page);\r
  84         }\r
  85      }\r
  86 -- \r
  87 2.1.4\r
  88 \r