39/91efe49a5f97b5a133a91b3064e55faf0f1975

   1 Return-Path: <cworth@cworth.org>\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 2F00B4196F0\r
   6         for <notmuch@notmuchmail.org>; Mon, 26 Apr 2010 10:51:24 -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: -2.89\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.89 tagged_above=-999 required=5\r
  12         tests=[ALL_TRUSTED=-1, BAYES_00=-1.9, T_MIME_NO_TEXT=0.01]\r
  13         autolearn=ham\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 8ABRWiR2-K6T; Mon, 26 Apr 2010 10:51:23 -0700 (PDT)\r
  17 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 62D44431FC1;\r
  19         Mon, 26 Apr 2010 10:51:23 -0700 (PDT)\r
  20 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  21         id B3167568DEC; Mon, 26 Apr 2010 10:51:22 -0700 (PDT)\r
  22 From: Carl Worth <cworth@cworth.org>\r
  23 To: dme@dme.org, notmuch@notmuchmail.org\r
  24 Subject: Re: [PATCH] emacs: Remove duplicate declaration of `notmuch-folders'\r
  25 In-Reply-To: <1272294583-18209-1-git-send-email-dme@dme.org>\r
  26 References: <1272294583-18209-1-git-send-email-dme@dme.org>\r
  27 User-Agent: Notmuch/0.2-202-g01ec4d3 (http://notmuchmail.org) Emacs/23.1.1\r
  28         (i486-pc-linux-gnu)\r
  29 Date: Mon, 26 Apr 2010 10:51:22 -0700\r
  30 Message-ID: <874oiydrvp.fsf@yoom.home.cworth.org>\r
  31 MIME-Version: 1.0\r
  32 Content-Type: multipart/signed; boundary="=-=-=";\r
  33         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  34 X-BeenThere: notmuch@notmuchmail.org\r
  35 X-Mailman-Version: 2.1.13\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: Mon, 26 Apr 2010 17:51:24 -0000\r
  47 \r
  48 --=-=-=\r
  49 Content-Transfer-Encoding: quoted-printable\r
  50 \r
  51 On Mon, 26 Apr 2010 16:09:43 +0100, dme@dme.org wrote:\r
  52 > From: David Edmondson <dme@dme.org>\r
  53 [no commit message]\r
  54 \r
  55 Thanks, David.\r
  56 \r
  57 Allow me the chance to try to convince you that *every* commit requires\r
  58 a sentence or two of justification, (beyond the one-line summary just\r
  59 stating "what" the patch does). There's *always* some utility to saying\r
  60 "why".\r
  61 \r
  62 Take this current patch which seems so simple. There were two\r
  63 declarations, so one had to go. What more could there be to say?\r
  64 \r
  65 For me, when reviewing this, I wondered "How did there come to be two\r
  66 different declarations?", and "Is this patch removing the right one?".\r
  67 So I had to go grep through the source to figure out what had happened.\r
  68 \r
  69 I could have avoided that if the commit had said something like:\r
  70 \r
  71         This variable was moved from notmuch.el to notmuch-lib.el some time\r
  72         ago, but the declaration in notmuch.el was left around. Clean\r
  73         that up.\r
  74 \r
  75 [And I just pushed the patch with this added.]\r
  76 \r
  77 Whether or not that convinces you, perhaps you can humor me and always\r
  78 try to come up with something to say, even when it seems there's nothing\r
  79 possible to add. If nothing else, it can be a fun challenge. If you need\r
  80 more beyond that, you can try writing your commit message as a haiku.\r
  81 \r
  82 =2DCarl\r
  83 \r
  84 =2D-=20\r
  85 carl.d.worth@intel.com\r
  86 \r
  87 --=-=-=\r
  88 Content-Type: application/pgp-signature\r
  89 \r
  90 -----BEGIN PGP SIGNATURE-----\r
  91 Version: GnuPG v1.4.10 (GNU/Linux)\r
  92 \r
  93 iD8DBQFL1dKa6JDdNq8qSWgRAlEZAJwIZkplMmDXqySGOM3ymp2F8ErgdQCfbNEj\r
  94 sthNcCwEq2KdRF7/J8eGHHc=\r
  95 =BF6j\r
  96 -----END PGP SIGNATURE-----\r
  97 --=-=-=--\r