f6/c7e3cb96689c695623322e8f9b53d1f904e636

   1 Return-Path: <bremner@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 olra.theworths.org (Postfix) with ESMTP id 30947431FAF\r
   6         for <notmuch@notmuchmail.org>; Sat, 21 Jan 2012 11:00:28 -0800 (PST)\r
   7 X-Virus-Scanned: Debian amavisd-new at olra.theworths.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 6HOdnzvXR6LG for <notmuch@notmuchmail.org>;\r
  16         Sat, 21 Jan 2012 11:00:26 -0800 (PST)\r
  17 Received: from tempo.its.unb.ca (tempo.its.unb.ca [131.202.1.21])\r
  18         (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 1F0D6431FAE\r
  21         for <notmuch@notmuchmail.org>; Sat, 21 Jan 2012 11:00:26 -0800 (PST)\r
  22 Received: from zancas.localnet\r
  23         (fctnnbsc36w-156034071197.pppoe-dynamic.High-Speed.nb.bellaliant.net\r
  24         [156.34.71.197]) (authenticated bits=0)\r
  25         by tempo.its.unb.ca (8.13.8/8.13.8) with ESMTP id q0LJ0NQm004319\r
  26         (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=NO);\r
  27         Sat, 21 Jan 2012 15:00:24 -0400\r
  28 Received: from bremner by zancas.localnet with local (Exim 4.77)\r
  29         (envelope-from <bremner@unb.ca>)\r
  30         id 1RogAp-0002Ps-3E; Sat, 21 Jan 2012 15:00:23 -0400\r
  31 From: David Bremner <david@tethera.net>\r
  32 To: Austin Clements <amdragon@MIT.EDU>\r
  33 Subject: Re: [PATCH 2/2] notmuch-reply.c: uncrustify\r
  34 In-Reply-To: <20120112152652.GA18625@mit.edu>\r
  35 References: <1324135695-15487-1-git-send-email-david@tethera.net>\r
  36         <1326197228-10356-1-git-send-email-david@tethera.net>\r
  37         <1326197228-10356-3-git-send-email-david@tethera.net>\r
  38         <20120112035739.GW20796@mit.edu> <87r4z56pbx.fsf@zancas.localnet>\r
  39         <20120112152652.GA18625@mit.edu>\r
  40 User-Agent: Notmuch/0.11+89~g85665a2 (http://notmuchmail.org) Emacs/23.3.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 Date: Sat, 21 Jan 2012 15:00:22 -0400\r
  43 Message-ID: <877h0kdgop.fsf@zancas.localnet>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: text/plain; charset=us-ascii\r
  46 Cc: notmuch@notmuchmail.org\r
  47 X-BeenThere: notmuch@notmuchmail.org\r
  48 X-Mailman-Version: 2.1.13\r
  49 Precedence: list\r
  50 List-Id: "Use and development of the notmuch mail system."\r
  51         <notmuch.notmuchmail.org>\r
  52 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  53         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  54 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  55 List-Post: <mailto:notmuch@notmuchmail.org>\r
  56 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  57 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  59 X-List-Received-Date: Sat, 21 Jan 2012 19:00:28 -0000\r
  60 \r
  61 On Thu, 12 Jan 2012 10:26:52 -0500, Austin Clements <amdragon@MIT.EDU> wrote:\r
  62 > Quoth David Bremner on Jan 12 at  9:08 am:\r
  63 > > @@ -425,7 +420,7 @@ notmuch_search_command (void *ctx, int argc, char *argv[])\r
  64 > >      int limit = -1; /* unlimited */\r
  65 > >  \r
  66 > >      enum { NOTMUCH_FORMAT_JSON, NOTMUCH_FORMAT_TEXT }\r
  67 > > -   format_sel = NOTMUCH_FORMAT_TEXT;\r
  68 > > +    format_sel = NOTMUCH_FORMAT_TEXT;\r
  69 > \r
  70 > This is unfortunate, but other than the format structure flattening,\r
  71 > the remaining changes look reasonable to me.\r
  72 \r
  73 I guess we have to accept that this is a big pile of heuristics, as long\r
  74 as there are not too many cases to be overriden, it should still be a\r
  75 net win.\r
  76 \r
  77 FWIW An alternative formatting that does not confuse the indenter is\r
  78 \r
  79 enum format_enum {\r
  80      NOTMUCH_FORMAT_JSON, NOTMUCH_FORMAT_TEXT\r
  81 } format_sel = NOTMUCH_FORMAT_TEXT;\r
  82 \r