8d/e50b7dde6b7e074bb0a461b377ca16fa1e57ca

   1 Return-Path: <david@tethera.net>\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 olra.theworths.org (Postfix) with ESMTP id F2A87431FBC\r
   6         for <notmuch@notmuchmail.org>; Fri, 19 Sep 2014 12:57:56 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: 0\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
  12         autolearn=disabled\r
  13 Received: from olra.theworths.org ([127.0.0.1])\r
  14         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  15         with ESMTP id kAReeLGY+PCs for <notmuch@notmuchmail.org>;\r
  16         Fri, 19 Sep 2014 12:57:51 -0700 (PDT)\r
  17 Received: from yantan.tethera.net (yantan.tethera.net [199.188.72.155])\r
  18         (using TLSv1 with cipher DHE-RSA-AES128-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 1042F431FAE\r
  21         for <notmuch@notmuchmail.org>; Fri, 19 Sep 2014 12:57:51 -0700 (PDT)\r
  22 Received: from remotemail by yantan.tethera.net with local (Exim 4.80)\r
  23         (envelope-from <david@tethera.net>)\r
  24         id 1XV4JN-0007A3-Ki; Fri, 19 Sep 2014 16:57:45 -0300\r
  25 Received: (nullmailer pid 8606 invoked by uid 1000); Fri, 19 Sep 2014\r
  26         19:57:40 -0000\r
  27 From: David Bremner <david@tethera.net>\r
  28 To: Jani Nikula <jani@nikula.org>, notmuch@notmuchmail.org\r
  29 Subject: Re: [PATCH] cli: add --output=address-{from,to,all} to notmuch search\r
  30 In-Reply-To: <1410021689-15901-1-git-send-email-jani@nikula.org>\r
  31 References: <871troub1v.fsf@qmul.ac.uk>\r
  32         <1410021689-15901-1-git-send-email-jani@nikula.org>\r
  33 User-Agent: Notmuch/0.18.1+97~g56c48dc (http://notmuchmail.org) Emacs/24.3.1\r
  34         (x86_64-pc-linux-gnu)\r
  35 Date: Fri, 19 Sep 2014 21:57:40 +0200\r
  36 Message-ID: <87k34zbcy3.fsf@maritornes.cs.unb.ca>\r
  37 MIME-Version: 1.0\r
  38 Content-Type: text/plain\r
  39 X-BeenThere: notmuch@notmuchmail.org\r
  40 X-Mailman-Version: 2.1.13\r
  41 Precedence: list\r
  42 List-Id: "Use and development of the notmuch mail system."\r
  43         <notmuch.notmuchmail.org>\r
  44 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  45         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  46 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  47 List-Post: <mailto:notmuch@notmuchmail.org>\r
  48 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  49 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  51 X-List-Received-Date: Fri, 19 Sep 2014 19:57:57 -0000\r
  52 \r
  53 Jani Nikula <jani@nikula.org> writes:\r
  54 \r
  55 > +         if (name && *name)\r
  56 > +             full_address = talloc_asprintf (NULL, "%s <%s>", name, addr);\r
  57 > +         else\r
  58 > +             full_address = talloc_asprintf (NULL, "<%s>", addr\r
  59 \r
  60 Is there some reason not to use sprinter as a talloc context here?\r
  61 \r
  62 > +\r
  63 > +         if (!full_address)\r
  64 > +             break;\r
  65 \r
  66 Is the error here out of memory? Maybe an error message would be a good\r
  67 idea.\r
  68 \r
  69 \r
  70 Obviously the docs need to be updated as well, and ideally the tests.  I\r
  71 think Jani was hinting that he didn't want to be the person to do\r
  72 that. Any volunteers?\r
  73 \r
  74 d\r