14/47c06b13026473dda2e9193e3564e8da0f3ff3

   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 16CCB6DE1B4C\r
   6  for <notmuch@notmuchmail.org>; Sun,  5 Apr 2015 14:33:41 -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.656\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.656 tagged_above=-999 required=5 tests=[AWL=0.646, \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 9M4LDGiL3dLz for <notmuch@notmuchmail.org>;\r
  16  Sun,  5 Apr 2015 14:33:39 -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 6AE506DE1B45\r
  20  for <notmuch@notmuchmail.org>; Sun,  5 Apr 2015 14:33:39 -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 1YerfR-0002Rm-G9; Sun, 05 Apr 2015 21:01:17 +0000\r
  24 Received: (nullmailer pid 29560 invoked by uid 1000); Sun, 05 Apr 2015\r
  25  21:00:40 -0000\r
  26 From: David Bremner <david@tethera.net>\r
  27 To: notmuch@notmuchmail.org\r
  28 Subject: [PATCH 1/4] cli: ignore config argument of notmuch_help_command\r
  29 Date: Mon,  6 Apr 2015 05:59:31 +0900\r
  30 Message-Id: <1428267574-28797-2-git-send-email-david@tethera.net>\r
  31 X-Mailer: git-send-email 2.1.4\r
  32 In-Reply-To: <1428267574-28797-1-git-send-email-david@tethera.net>\r
  33 References: <1428267574-28797-1-git-send-email-david@tethera.net>\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.18\r
  36 Precedence: list\r
  37 List-Id: "Use and development of the notmuch mail system."\r
  38  <notmuch.notmuchmail.org>\r
  39 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  40  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  41 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  42 List-Post: <mailto:notmuch@notmuchmail.org>\r
  43 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  44 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  45  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  46 X-List-Received-Date: Sun, 05 Apr 2015 21:33:41 -0000\r
  47 \r
  48 We call it with NULL at one point anyway, so it needs to work with\r
  49 NULL. Since the only place we use talloc is right before exec, there\r
  50 is no harm in always using NULL.\r
  51 ---\r
  52  notmuch.c | 6 +++---\r
  53  1 file changed, 3 insertions(+), 3 deletions(-)\r
  54 \r
  55 diff --git a/notmuch.c b/notmuch.c\r
  56 index a5b2877..31672c3 100644\r
  57 --- a/notmuch.c\r
  58 +++ b/notmuch.c\r
  59 @@ -177,7 +177,7 @@ exec_man (const char *page)\r
  60  }\r
  61  \r
  62  static int\r
  63 -notmuch_help_command (notmuch_config_t *config, int argc, char *argv[])\r
  64 +notmuch_help_command (unused (notmuch_config_t * config), int argc, char *argv[])\r
  65  {\r
  66      command_t *command;\r
  67      help_topic_t *topic;\r
  68 @@ -202,14 +202,14 @@ notmuch_help_command (notmuch_config_t *config, int argc, char *argv[])\r
  69  \r
  70      command = find_command (argv[0]);\r
  71      if (command) {\r
  72 -       char *page = talloc_asprintf (config, "notmuch-%s", command->name);\r
  73 +       char *page = talloc_asprintf (NULL, "notmuch-%s", command->name);\r
  74         exec_man (page);\r
  75      }\r
  76  \r
  77      for (i = 0; i < ARRAY_SIZE (help_topics); i++) {\r
  78         topic = &help_topics[i];\r
  79         if (strcmp (argv[0], topic->name) == 0) {\r
  80 -           char *page = talloc_asprintf (config, "notmuch-%s", topic->name);\r
  81 +           char *page = talloc_asprintf (NULL, "notmuch-%s", topic->name);\r
  82             exec_man (page);\r
  83         }\r
  84      }\r
  85 -- \r
  86 2.1.4\r
  87 \r