4e/f1b571b6dd83e7f08794f5f829da3c019b6ced

   1 Return-Path: <ethan.glasser.camp@gmail.com>\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 BE603431FB6\r
   6         for <notmuch@notmuchmail.org>; Mon,  5 Nov 2012 20:12:46 -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: -0.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id Xdm7JVMHm1Oh for <notmuch@notmuchmail.org>;\r
  17         Mon,  5 Nov 2012 20:12:45 -0800 (PST)\r
  18 Received: from mail-vc0-f181.google.com (mail-vc0-f181.google.com\r
  19         [209.85.220.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id 3D6F7431FAE\r
  22         for <notmuch@notmuchmail.org>; Mon,  5 Nov 2012 20:12:45 -0800 (PST)\r
  23 Received: by mail-vc0-f181.google.com with SMTP id n11so14243vch.26\r
  24         for <notmuch@notmuchmail.org>; Mon, 05 Nov 2012 20:12:44 -0800 (PST)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=from:to:cc:subject:in-reply-to:references:user-agent:date\r
  27         :message-id:mime-version:content-type;\r
  28         bh=LiFV/9cDsFRudng+09arYVzDkRAz+0YNQtpYFBXy+7w=;\r
  29         b=ns1mapmLqXExWXAsFQvB7AU8L5MVYtFXg3yr4fFCc5wfE+gT68TJ5tBj3ycEmylWr0\r
  30         syIyuR034HuchIX2gu9OgqAm3Ci4Mn7tkTHDpb/dfBDmWK4cXkhM4UqnxKuwsvwIL6a8\r
  31         2qzhGKDoeFpfESeDSewppxz1OTUY3ts5vh1CV5WcMPWr7ke5egtfa+zlQ+x8YXSx1eyW\r
  32         U7ecRC7qYgXTulbr+RHqmO5f6WA9ovwtQh5YCxt6rZwvPxFHNuH3//U7Vw6U2UrJUs1s\r
  33         z6iK/NLwAwgGYn3t4zHgajl03pYgP6PZvCcEAKyedee4T7KlR9Y7uEFsFx9HYHCdl9X8\r
  34         paaA==\r
  35 Received: by 10.52.173.34 with SMTP id bh2mr10167796vdc.73.1352175164567;\r
  36         Mon, 05 Nov 2012 20:12:44 -0800 (PST)\r
  37 Received: from smtp.gmail.com (p70-80.acedsl.com. [66.114.70.80])\r
  38         by mx.google.com with ESMTPS id zx18sm9362259veb.3.2012.11.05.20.12.43\r
  39         (version=TLSv1/SSLv3 cipher=OTHER);\r
  40         Mon, 05 Nov 2012 20:12:44 -0800 (PST)\r
  41 From: Ethan Glasser-Camp <ethan.glasser.camp@gmail.com>\r
  42 To: Tomi Ollila <tomi.ollila@iki.fi>, notmuch@notmuchmail.org\r
  43 Subject: Re: [PATCH 1/1] uncrustify.cfg: added 3 new types for uncrustify to\r
  44         know\r
  45 In-Reply-To: <1351865226-916-1-git-send-email-tomi.ollila@iki.fi>\r
  46 References: <1351865226-916-1-git-send-email-tomi.ollila@iki.fi>\r
  47 User-Agent: Notmuch/0.14+45~g6ea9330 (http://notmuchmail.org) Emacs/23.4.1\r
  48         (x86_64-pc-linux-gnu)\r
  49 Date: Mon, 05 Nov 2012 23:12:39 -0500\r
  50 Message-ID: <87390nid3c.fsf@betacantrips.com>\r
  51 MIME-Version: 1.0\r
  52 Content-Type: text/plain; charset=us-ascii\r
  53 Cc: tomi.ollila@iki.fi\r
  54 X-BeenThere: notmuch@notmuchmail.org\r
  55 X-Mailman-Version: 2.1.13\r
  56 Precedence: list\r
  57 List-Id: "Use and development of the notmuch mail system."\r
  58         <notmuch.notmuchmail.org>\r
  59 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  60         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  61 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  62 List-Post: <mailto:notmuch@notmuchmail.org>\r
  63 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  64 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  65         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  66 X-List-Received-Date: Tue, 06 Nov 2012 04:12:46 -0000\r
  67 \r
  68 Tomi Ollila <tomi.ollila@iki.fi> writes:\r
  69 \r
  70 > Added FILE, notmuch_show_params_t and sprinter_t to be\r
  71 > types when uncrustifying sources. This affect spacing\r
  72 > when uncrustify is deciding for type declaration instead\r
  73 > of binary multiplication operation.\r
  74 \r
  75 This looks good to me. If you had plenty of time and no more patches to\r
  76 review, I'd prefer the slightly cleaner English:\r
  77 \r
  78 This affects how uncrustify puts spacing around pointers to these types,\r
  79 since it can parse them as type declarations instead of binary\r
  80 multiplication operations.\r
  81 \r
  82 \r
  83 ... but even suggesting this indicates I've moved past the bike shed and\r
  84 into somebody else's kitchen.\r
  85 \r
  86 Ethan\r