3d/af444e711cafeb428bb02ba29a01fe36d35fa0

   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 arlo.cworth.org (Postfix) with ESMTP id B46B56DE1817\r
   6  for <notmuch@notmuchmail.org>; Mon, 26 Oct 2015 11:56:59 -0700 (PDT)\r
   7 X-Virus-Scanned: Debian amavisd-new at cworth.org\r
   8 X-Spam-Flag: NO\r
   9 X-Spam-Score: -8.929\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-8.929 tagged_above=-999 required=5 tests=[AM.WBL=-8,\r
  12   ALL_TRUSTED=-1, AWL=0.071] 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 f8Kl_-rWHseZ; Mon, 26 Oct 2015 11:56:58 -0700 (PDT)\r
  16 Received: from wondoo.home.cworth.org (unknown [10.0.0.1])\r
  17  (Authenticated sender: cworth)\r
  18  by arlo.cworth.org (Postfix) with ESMTPSA id 1B6456DE17E6;\r
  19  Mon, 26 Oct 2015 11:56:58 -0700 (PDT)\r
  20 Received: from wondoo.home.cworth.org (localhost [IPv6:::1])\r
  21  by wondoo.home.cworth.org (Postfix) with ESMTPS id 0AC5B14C492B;\r
  22  Mon, 26 Oct 2015 11:56:58 -0700 (PDT)\r
  23 To: Tomi Ollila <tomi.ollila@iki.fi>, David Bremner <david@tethera.net>,\r
  24  notmuch@notmuchmail.org\r
  25 Subject: Re: elisp + company completion patches, v7\r
  26 In-Reply-To: <m2lhaqu23j.fsf@guru.guru-group.fi>\r
  27 References: <1445786638-13763-1-git-send-email-david@tethera.net>\r
  28  <m2lhaqu23j.fsf@guru.guru-group.fi>\r
  29 User-Agent: Notmuch/0.20.2 (http://notmuchmail.org) Emacs/24.5.1\r
  30  (x86_64-pc-linux-gnu)\r
  31 Sender: cworth@cworth.org\r
  32 From: Carl Worth <cworth@cworth.org>\r
  33 Date: Mon, 26 Oct 2015 11:56:56 -0700\r
  34 Message-ID: <871tche9vr.fsf@wondoo.home.cworth.org>\r
  35 MIME-Version: 1.0\r
  36 Content-Type: multipart/signed; boundary="=-=-=";\r
  37  micalg=pgp-sha512; protocol="application/pgp-signature"\r
  38 X-BeenThere: notmuch@notmuchmail.org\r
  39 X-Mailman-Version: 2.1.20\r
  40 Precedence: list\r
  41 List-Id: "Use and development of the notmuch mail system."\r
  42  <notmuch.notmuchmail.org>\r
  43 List-Unsubscribe: <https://notmuchmail.org/mailman/options/notmuch>,\r
  44  <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  45 List-Archive: <http://notmuchmail.org/pipermail/notmuch/>\r
  46 List-Post: <mailto:notmuch@notmuchmail.org>\r
  47 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  48 List-Subscribe: <https://notmuchmail.org/mailman/listinfo/notmuch>,\r
  49  <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  50 X-List-Received-Date: Mon, 26 Oct 2015 18:56:59 -0000\r
  51 \r
  52 --=-=-=\r
  53 Content-Type: text/plain\r
  54 \r
  55 On Sun, Oct 25 2015, Tomi Ollila wrote:\r
  56 > I, if this does not sound nitpicking, would amend those 'This patch' and\r
  57 > 'With this patch' texts in commit messages away by some rewording -- those\r
  58 > pose as a bad example for someone reading the code -- and diminish the\r
  59 > effect when I complain that again in future patched ;)\r
  60 \r
  61 What's the motivation here?\r
  62 \r
  63 It's often the case that a good commit message needs to describe the way\r
  64 things were working prior to a commit, (describing the bug or missing\r
  65 feature or what have you), and also needs to describe the change that is\r
  66 affected by the commit itself.\r
  67 \r
  68 So I actually like seeing commit messages that say things like:\r
  69 \r
  70     Prior to this commit... (some bug existed...)\r
  71 \r
  72     In this commit... (some code is reworked to fix the bug...)\r
  73 \r
  74 But maybe I've missed the point of your message.\r
  75 \r
  76 -Carl\r
  77 \r
  78 --=-=-=\r
  79 Content-Type: application/pgp-signature; name="signature.asc"\r
  80 \r
  81 -----BEGIN PGP SIGNATURE-----\r
  82 Version: GnuPG v1\r
  83 \r
  84 iQIcBAEBCgAGBQJWLnd5AAoJEGACM7qeVNxh9pwQAKVp3EqS3EgE+3PvWWRSzh1W\r
  85 4MNyIZoinupCnZ1eulrvUYi9O+a1w0F/O7vYcfcw68K7nFCYAInhHJpX6YUwHqW0\r
  86 SXO16TOnpPCqjdvPxgKDSAoyviWuNc0CwW0bqVdA31N3Y+0eRP4KYJgYPyxU9XhK\r
  87 48uGJlHlRHwiOSHRvQzOAB+iOEOyy88Q6VJlaZLRwAQB//En/TQnLHBWxxkM1aO6\r
  88 +XqHqlrM02KhPMQXP8BduY+/7sZQYUZv1QHgV+RjbD8JVAuKW7ir4SGc71XtnbY+\r
  89 icB9/NCC3cs2+wtSC40K13TE7pymC6OSfK40By52fjrpTsd8IFe0rt6uqYWfprdx\r
  90 N4jzcxBpvvzSJrAQlK80Ko94uTjDVmSO/JlQ289wyBvkxHBZSKr9xCgEHHFXbRnh\r
  91 sctVIABkNfAP1e38Zsz7bnlPPhPRABUXjurKK/IEblt/KFEVBF2d/JmlCj8UKb0n\r
  92 Y+Emgbh4UXBQigpDKYGSfxoOnflh+0dZL4NdNVy4b43gg2SznmxApMEnyHO4j76z\r
  93 ZYNcSxQxaWjzjmP45qr3L0wE/rsoHcEXL3i3WP9DnwziKRi3fC0g/rKM3+4mYsUw\r
  94 UflzJYDeSb609EEX8M/cpBqWiDGhssWHcHSIX6ex/P5Iqj687sOAGt9kXG2uo1fa\r
  95 3rv6vItEo7+lFDcS/FVr\r
  96 =D39h\r
  97 -----END PGP SIGNATURE-----\r
  98 --=-=-=--\r