98/3dbea7020fc9dfce6eb0f98c322ce565d81991

   1 Return-Path: <jrollins@finestructure.net>\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 6BEF0429E54\r
   6         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 13:38:20 -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: -2.29\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.29 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3, T_MIME_NO_TEXT=0.01] 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 vqYinBAlHyvk for <notmuch@notmuchmail.org>;\r
  16         Sun, 22 Jan 2012 13:38:20 -0800 (PST)\r
  17 Received: from outgoing-mail.its.caltech.edu (outgoing-mail.its.caltech.edu\r
  18         [131.215.239.19])\r
  19         by olra.theworths.org (Postfix) with ESMTP id D2A69429E40\r
  20         for <notmuch@notmuchmail.org>; Sun, 22 Jan 2012 13:38:19 -0800 (PST)\r
  21 Received: from fire-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by fire-doxen-postvirus (Postfix) with ESMTP id 416D42E50D89;\r
  23         Sun, 22 Jan 2012 13:38:15 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on fire-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-76-174-137-84.socal.res.rr.com\r
  26         [76.174.137.84]) (Authenticated sender: jrollins)\r
  27         by fire-doxen-submit (Postfix) with ESMTP id 825362E50DB8;\r
  28         Sun, 22 Jan 2012 13:38:12 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 77255AF4; Sun, 22 Jan 2012 13:38:12 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
  33 Subject: Re: [PATCH 2/3] emacs: Don't return the button from\r
  34         `notmuch-show-insert-part-header'.\r
  35 In-Reply-To: <1327052612-1040-2-git-send-email-dme@dme.org>\r
  36 References: <1327052612-1040-1-git-send-email-dme@dme.org>\r
  37         <1327052612-1040-2-git-send-email-dme@dme.org>\r
  38 User-Agent: Notmuch/0.11+93~gc3dc12a (http://notmuchmail.org) Emacs/23.3.1\r
  39         (x86_64-pc-linux-gnu)\r
  40 Date: Sun, 22 Jan 2012 13:38:09 -0800\r
  41 Message-ID: <877h0jmn9a.fsf@servo.finestructure.net>\r
  42 MIME-Version: 1.0\r
  43 Content-Type: multipart/signed; boundary="=-=-=";\r
  44         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  45 X-BeenThere: notmuch@notmuchmail.org\r
  46 X-Mailman-Version: 2.1.13\r
  47 Precedence: list\r
  48 List-Id: "Use and development of the notmuch mail system."\r
  49         <notmuch.notmuchmail.org>\r
  50 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  51         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  52 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  53 List-Post: <mailto:notmuch@notmuchmail.org>\r
  54 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  55 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  56         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  57 X-List-Received-Date: Sun, 22 Jan 2012 21:38:20 -0000\r
  58 \r
  59 --=-=-=\r
  60 \r
  61 On Fri, 20 Jan 2012 09:43:31 +0000, David Edmondson <dme@dme.org> wrote:\r
  62 > Instead, allow the caller to specify some parameters for the\r
  63 > button. Rework `notmuch-show-insert-part-multipart/signed' and\r
  64 > `notmuch-show-insert-part-multipart/encrypted' accordingly, moving\r
  65 > most of the code into a common\r
  66 > `notmuch-show-insert-part-multipart/signed-or-encrypted' to reduce\r
  67 > duplication.\r
  68 \r
  69 Hi, David.  A couple of issues with this patch:\r
  70 \r
  71 This patch seems to include multiple distinct changes.  There is a\r
  72 change to notmuch-show-insert-part-header, but a seemingly unrelated\r
  73 change to the insertion of signed/encrypted part buttons.  They should\r
  74 be in separate patches.\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 having\r
  79 one function filled with special casing to handle two things, rather\r
  80 than having two distinct functions?\r
  81 \r
  82 Finally, this patch throws out all the changes from the previous patch,\r
  83 making the previous patch superfluous.\r
  84 \r
  85 jamie.\r
  86 \r
  87 --=-=-=\r
  88 Content-Type: application/pgp-signature\r
  89 \r
  90 -----BEGIN PGP SIGNATURE-----\r
  91 Version: GnuPG v1.4.11 (GNU/Linux)\r
  92 \r
  93 iQIcBAEBCAAGBQJPHIHCAAoJEO00zqvie6q89gYQAKjyylLyOngLm8k43S6P6Wk/\r
  94 3q8P3H9bYVGQpTIRvn7I5H9YvqZrU98GlkFRcuI7CrwUuy8ojDMpWqC8QGf6hcWx\r
  95 uF9rdmaozaVxQMQPeWuEgaDlTBFag/brRs14tq/4osQijO4aLd7H/lZlV4KHxWR5\r
  96 qr7oJxVQ9uZmBFrgrJuFbD+gWA/DnfeEp79aM+lCeQm9KkpKn0AJwDFe/ayARGUv\r
  97 EL1uHJDenIK2A9MJuzqw/2Io6G2oMLMkRvQMdzuGWY5D9ZBHYZoBZ26AvzaHxHzt\r
  98 XRd6SdYcQroWCZo/M93t+s+sgEPmeaqbWoMohotlMJMALiitc+4fB1ldLfrKIIcj\r
  99 DanOQrXVJlTxedmPVymHhYy1Utc0XIblTOT8ax2QyOop2Mc/QbjPXBNfBXYCxx8f\r
 100 94vFAwW8OOv8qq7F1DmqKZMnKpa/gyc661P5WlkcT5IDP6PdT72JwQJ645QuKKHk\r
 101 6JJpM+WlEqIhOaIuLj41aPfie4XuMne/Mrd3UBn4hrB2ZOaOFnk8387KkRYfWVzU\r
 102 8b5bBtwLUZfZYnbPPHCxfxJddgwEFvyZ90nr3j0JeWr+EgKjYxZKMSTZyv7jzTbN\r
 103 zslA9mrxxF+5kSlH5KWWyTkIHrWVvpdpJ5+u123abI7CBJ/hY+enDD0xapkHH/lB\r
 104 cCxYdPWDSzX1134/6hAD\r
 105 =4NDl\r
 106 -----END PGP SIGNATURE-----\r
 107 --=-=-=--\r