d2/f541f03c2afbe45bdddff8dec7a968212543df

   1 Return-Path: <kfogel@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 AD0C3431FB6\r
   6         for <notmuch@notmuchmail.org>; Sun,  7 Apr 2013 14:50:12 -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.699\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.699 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, FREEMAIL_FROM=0.001,\r
  13         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 GMThlSfXQYaR for <notmuch@notmuchmail.org>;\r
  17         Sun,  7 Apr 2013 14:50:11 -0700 (PDT)\r
  18 Received: from mail-vb0-f54.google.com (mail-vb0-f54.google.com\r
  19         [209.85.212.54]) (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 2D25C431FAE\r
  22         for <notmuch@notmuchmail.org>; Sun,  7 Apr 2013 14:50:11 -0700 (PDT)\r
  23 Received: by mail-vb0-f54.google.com with SMTP id w16so3330523vbf.13\r
  24         for <notmuch@notmuchmail.org>; Sun, 07 Apr 2013 14:50:10 -0700 (PDT)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113;\r
  26         h=x-received:sender:from:to:cc:subject:references:reply-to:date\r
  27         :in-reply-to:message-id:user-agent:mime-version:content-type;\r
  28         bh=ad9uILkCvu9Beu+g8Yt3KRi0bsbZFfhGfEYHI7qgzEY=;\r
  29         b=DQJA2XBoX5iXJZQdQK/v/cdlWMP2twrT3VeqJ5WsOLuaCLNxRmTAC5DJSaFyuRmSWt\r
  30         duqxIO+I1cm/SeXeP+PsTeOPyrejBlz2fwwGsrSQQnBpuYs31v9x5Epshh8MOUksJctr\r
  31         rLYwwJDBM6HxQ3zppgeS4qOiUq9RiWU1pFMMdGqm9xWo6h0inztu2vNZHin2yP9f2xua\r
  32         HtAHlzLroltaM0GSm2exHOERNYvZ2IccQPjbGqMOq2KLRXl3Jh3EAhrVUDeP4CxmI2AY\r
  33         UFdlf626tb2M/syySJqO77U3So1wJU7XA6rpPVR33uE5Co4qRz5IV39l2FOUSgrT9hhk\r
  34         SV2g==\r
  35 X-Received: by 10.52.26.17 with SMTP id h17mr11777762vdg.101.1365371409360;\r
  36         Sun, 07 Apr 2013 14:50:09 -0700 (PDT)\r
  37 Received: from floss.red-bean.com ([75.93.240.169])\r
  38         by mx.google.com with ESMTPS id u20sm441546vdt.10.2013.04.07.14.50.08\r
  39         (version=TLSv1.2 cipher=RC4-SHA bits=128/128);\r
  40         Sun, 07 Apr 2013 14:50:08 -0700 (PDT)\r
  41 Sender: Karl Fogel <kfogel@gmail.com>\r
  42 From: Karl Fogel <kfogel@red-bean.com>\r
  43 To: David Bremner <david@tethera.net>\r
  44 Subject: Re: [PATCH] emacs: add missing paren to fix defun in\r
  45         notmuch-address.el.\r
  46 References: <87wqsfik5l.fsf@floss.red-bean.com>\r
  47         <87mwtah2hd.fsf@zancas.localnet>\r
  48 Date: Sun, 07 Apr 2013 17:50:05 -0400\r
  49 In-Reply-To: <87mwtah2hd.fsf@zancas.localnet> (David Bremner's message of\r
  50         "Sun, 07 Apr 2013 10:52:46 -0300")\r
  51 Message-ID: <877gkeau42.fsf@floss.red-bean.com>\r
  52 User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux)\r
  53 MIME-Version: 1.0\r
  54 Content-Type: text/plain\r
  55 X-Mailman-Approved-At: Sun, 07 Apr 2013 23:23:45 -0700\r
  56 Cc: notmuch@notmuchmail.org\r
  57 X-BeenThere: notmuch@notmuchmail.org\r
  58 X-Mailman-Version: 2.1.13\r
  59 Precedence: list\r
  60 Reply-To: Karl Fogel <kfogel@red-bean.com>\r
  61 List-Id: "Use and development of the notmuch mail system."\r
  62         <notmuch.notmuchmail.org>\r
  63 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  64         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  65 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  66 List-Post: <mailto:notmuch@notmuchmail.org>\r
  67 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  68 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  69         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  70 X-List-Received-Date: Sun, 07 Apr 2013 21:50:12 -0000\r
  71 \r
  72 David Bremner <david@tethera.net> writes:\r
  73 >Karl Fogel <kfogel@red-bean.com> writes:\r
  74 >> This patch fixes a trivial missing-paren problem in notmuch-address.el\r
  75 >> (and reindents the following defun accordingly).  I'm not subscribed\r
  76 >> to this list, so please keep me CC'd on any followups.\r
  77 >\r
  78 >Dear Karl;\r
  79 >\r
  80 >Thanks very much for the patch.\r
  81 >\r
  82 >Since the offending commit is now reverted, it would be great if\r
  83 >somebody (TM) would combine your patch with  238bf4cb09.\r
  84 \r
  85 Oh, it's trivial.  The problem with 238bf4cb09 was simply that the\r
  86 function (defun) `notmuch-bbdb/snarf-headers' was missing a closing\r
  87 paren.  A visible symptom of this was that the *next* defun after it,\r
  88 `notmuch-bbdb/snarf-from', was spuriously indented inward.  If anyone\r
  89 had tried reindenting further, all the code below it would also have\r
  90 indented inward, making the problem more obvious.\r
  91 \r
  92 So the solution is to:\r
  93 \r
  94   1) Re-apply the 238bf4cb09 patch\r
  95 \r
  96   2) Add a parenthesis to the end of `notmuch-bbdb/snarf-headers',\r
  97      such that the line "(bbdb-update-records addrs t t))" becomes\r
  98      "(bbdb-update-records addrs t t)))"\r
  99 \r
 100   3) Unindent the function `notmuch-bbdb/snarf-from' immediately below\r
 101      there, which just means pulling each line leftward two spaces\r
 102 \r
 103   4) Commit, push, profit :-).\r
 104 \r
 105 The above recipe is, of course, equivalent to re-applying the 238bf4cb09\r
 106 patch, then applying my patch (4c74ad313f608f0834961c63c70d1f811ef103b7)\r
 107 on top of it.  I'm not sure what the gitmost way to do that is, but if\r
 108 you want I can simply submit a combined change whose commit message\r
 109 makes clear what's going on.\r
 110 \r
 111 -Karl\r