a9/99ec3b7133b8cec20fbc1d593cdcf09456e523

   1 Return-Path: <dme@dme.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 CF3DE429E5B\r
   6         for <notmuch@notmuchmail.org>; Mon, 23 Jan 2012 00:16:12 -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.7\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.7 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_LOW=-0.7] 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 KIH2xp4l7BKO for <notmuch@notmuchmail.org>;\r
  16         Mon, 23 Jan 2012 00:16:12 -0800 (PST)\r
  17 Received: from mail-wi0-f181.google.com (mail-wi0-f181.google.com\r
  18         [209.85.212.181]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  19         (No client certificate requested)\r
  20         by olra.theworths.org (Postfix) with ESMTPS id 2213E429E54\r
  21         for <notmuch@notmuchmail.org>; Mon, 23 Jan 2012 00:16:12 -0800 (PST)\r
  22 Received: by wibhi8 with SMTP id hi8so399089wib.26\r
  23         for <notmuch@notmuchmail.org>; Mon, 23 Jan 2012 00:16:10 -0800 (PST)\r
  24 Received: by 10.180.14.72 with SMTP id n8mr11509298wic.22.1327306570800;\r
  25         Mon, 23 Jan 2012 00:16:10 -0800 (PST)\r
  26 Received: from hotblack-desiato.hh.sledj.net\r
  27         (host81-149-164-25.in-addr.btopenworld.com. [81.149.164.25])\r
  28         by mx.google.com with ESMTPS id l6sm157224wiv.11.2012.01.23.00.16.08\r
  29         (version=TLSv1/SSLv3 cipher=OTHER);\r
  30         Mon, 23 Jan 2012 00:16:09 -0800 (PST)\r
  31 Received: by hotblack-desiato.hh.sledj.net (Postfix, from userid 30000)\r
  32         id 682FBA0950; Mon, 23 Jan 2012 08:16:07 +0000 (GMT)\r
  33 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
  34 Subject: Re: [PATCH 2/3] emacs: Don't return the button from\r
  35         `notmuch-show-insert-part-header'.\r
  36 In-Reply-To: <877h0jmn9a.fsf@servo.finestructure.net>\r
  37 References: <1327052612-1040-1-git-send-email-dme@dme.org>\r
  38         <1327052612-1040-2-git-send-email-dme@dme.org>\r
  39         <877h0jmn9a.fsf@servo.finestructure.net>\r
  40 User-Agent: Notmuch/0.11+73~gd51b784 (http://notmuchmail.org) Emacs/24.0.92.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 From: David Edmondson <dme@dme.org>\r
  43 Date: Mon, 23 Jan 2012 08:16:03 +0000\r
  44 Message-ID: <cun8vky6dh8.fsf@hotblack-desiato.hh.sledj.net>\r
  45 MIME-Version: 1.0\r
  46 Content-Type: multipart/signed; boundary="=-=-=";\r
  47         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  48 X-Gm-Message-State:\r
  49  ALoCoQnrXz+hVSGJRwsilnrX1G544mYD3YLYHeOdhYXcfH9aQ7i9FhgYX4AwIlgucA3cA5yRM0o+\r
  50 Cc: notmuch <notmuch@notmuchmail.org>\r
  51 X-BeenThere: notmuch@notmuchmail.org\r
  52 X-Mailman-Version: 2.1.13\r
  53 Precedence: list\r
  54 List-Id: "Use and development of the notmuch mail system."\r
  55         <notmuch.notmuchmail.org>\r
  56 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  57         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  58 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  59 List-Post: <mailto:notmuch@notmuchmail.org>\r
  60 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  61 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  62         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  63 X-List-Received-Date: Mon, 23 Jan 2012 08:16:12 -0000\r
  64 \r
  65 --=-=-=\r
  66 Content-Type: text/plain\r
  67 \r
  68 On Sun, 22 Jan 2012 13:38:09 -0800, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
  69 > This patch seems to include multiple distinct changes.  There is a\r
  70 > change to notmuch-show-insert-part-header, but a seemingly unrelated\r
  71 > change to the insertion of signed/encrypted part buttons.  They should\r
  72 > be in separate patches.\r
  73 \r
  74 I can separate them.\r
  75 \r
  76 > I'm also not sure I understand why the proposed changes to the\r
  77 > signed/encrypted button insertion functions are necessary or desired.\r
  78 > Was there a problem with the logic as it was?  What is gained by\r
  79 > having one function filled with special casing to handle two things,\r
  80 > rather than having two distinct functions?\r
  81 \r
  82 There was no problem with the logic. The code in the two functions was\r
  83 almost identical, so I'd like to make any future changes in just one\r
  84 place.\r
  85 \r
  86 You didn't actually answer my question - is the logic in the new\r
  87 function correct?\r
  88 \r
  89 > Finally, this patch throws out all the changes from the previous patch,\r
  90 > making the previous patch superfluous.\r
  91 \r
  92 I'll merge the first patch into the later (and presumably get accused of\r
  93 submitting patches which include multiple distinct changes :-)).\r
  94 \r
  95 --=-=-=\r
  96 Content-Type: application/pgp-signature\r
  97 \r
  98 -----BEGIN PGP SIGNATURE-----\r
  99 Version: GnuPG v1.4.11 (GNU/Linux)\r
 100 \r
 101 iEYEARECAAYFAk8dF0MACgkQaezQq/BJZRYZ/wCeIGyyNpORbO7KX09PJ54uwB2y\r
 102 +IEAnitwy4y8BT0iMMvUhxlVEc/8q+kq\r
 103 =Diym\r
 104 -----END PGP SIGNATURE-----\r
 105 --=-=-=--\r