17/9dd3d7667b91041a4a9942078636d38c0dafd9

   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 98800429E5B\r
   6         for <notmuch@notmuchmail.org>; Mon, 23 Jan 2012 00:52:32 -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 XTXtmGWkE2bS for <notmuch@notmuchmail.org>;\r
  16         Mon, 23 Jan 2012 00:52:32 -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 120C6429E54\r
  20         for <notmuch@notmuchmail.org>; Mon, 23 Jan 2012 00:52:32 -0800 (PST)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id B3F8566E00AE;\r
  23         Mon, 23 Jan 2012 00:52:31 -0800 (PST)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-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 earth-doxen-submit (Postfix) with ESMTP id EEE0F66E00E3;\r
  28         Mon, 23 Jan 2012 00:52:28 -0800 (PST)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 28F91AF4; Mon, 23 Jan 2012 00:52:29 -0800 (PST)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: David Edmondson <dme@dme.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: <cun8vky6dh8.fsf@hotblack-desiato.hh.sledj.net>\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         <877h0jmn9a.fsf@servo.finestructure.net>\r
  39         <cun8vky6dh8.fsf@hotblack-desiato.hh.sledj.net>\r
  40 User-Agent: Notmuch/0.11+102~g958863a (http://notmuchmail.org) Emacs/23.3.1\r
  41         (x86_64-pc-linux-gnu)\r
  42 Date: Mon, 23 Jan 2012 00:52:26 -0800\r
  43 Message-ID: <87ipk22439.fsf@servo.finestructure.net>\r
  44 MIME-Version: 1.0\r
  45 Content-Type: multipart/signed; boundary="=-=-=";\r
  46         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  47 Cc: notmuch <notmuch@notmuchmail.org>\r
  48 X-BeenThere: notmuch@notmuchmail.org\r
  49 X-Mailman-Version: 2.1.13\r
  50 Precedence: list\r
  51 List-Id: "Use and development of the notmuch mail system."\r
  52         <notmuch.notmuchmail.org>\r
  53 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  54         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  55 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  56 List-Post: <mailto:notmuch@notmuchmail.org>\r
  57 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  58 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  59         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  60 X-List-Received-Date: Mon, 23 Jan 2012 08:52:32 -0000\r
  61 \r
  62 --=-=-=\r
  63 \r
  64 On Mon, 23 Jan 2012 08:16:03 +0000, David Edmondson <dme@dme.org> wrote:\r
  65 > There was no problem with the logic. The code in the two functions was\r
  66 > almost identical, so I'd like to make any future changes in just one\r
  67 > place.\r
  68 >\r
  69 > You didn't actually answer my question - is the logic in the new\r
  70 > function correct?\r
  71 \r
  72 Honestly I didn't look too closely yet since I'm not convinced we need\r
  73 the change at all.  I would prefer to keep the functions separate.  In\r
  74 my opinion, enough special casing would be required that it wouldn't be\r
  75 worth it, and it would make the code less clear.\r
  76 \r
  77 > I'll merge the first patch into the later (and presumably get accused of\r
  78 > submitting patches which include multiple distinct changes :-)).\r
  79 \r
  80 But if you're removing all the code anyway, it's not a distinct change.\r
  81 It's still just a replacement.\r
  82 \r
  83 jamie.\r
  84 \r
  85 --=-=-=\r
  86 Content-Type: application/pgp-signature\r
  87 \r
  88 -----BEGIN PGP SIGNATURE-----\r
  89 Version: GnuPG v1.4.11 (GNU/Linux)\r
  90 \r
  91 iQIcBAEBCAAGBQJPHR/KAAoJEO00zqvie6q8H7gP/0K1ekDHsNVmvvisLTDeR1/b\r
  92 jG/RX5pxtxg0r/OQBnGzLXnEYEI4bmthwbOHdOrlr8GSBQYYA9YA1RCNcKseIxfT\r
  93 DtLuvSpqNhii3U2W5Rhh0OGoZHLVNSgL7J6uh4CYIe/jnVZFg/9IHJVVxupmlH4N\r
  94 X1T1xBPOXFV9JhP/oUg46jRpCrApdU08slF3vT//nRuapnurSCdRGx7206g1+55h\r
  95 dnOwolFNlxGm6xodftt+VKD280XdBVTmVg/cKrvTrLYrnru6KRy6opsUz9/3RA+j\r
  96 HdH4zFhchP2Oj5lgnd5ZU/c+hxbRo4cQLnLsAOXVc/ua+bmcCHNX9QH1PnYteKjU\r
  97 5mOzCgmg523K8/rEO9XxDR2njBnSxL159ArBB/Am+Wism1VvOGSiT+3YxrRt+npD\r
  98 j9sfyyhfQXYthPRKecP091fTJX2tpO7KOP8d7eXyEJYZGImIogE18G4V/GXuWv6E\r
  99 0utIurYNehDS9Qn+mbP7JlOR0DR5714BUj0P+YuggdkXcEAeE4PH1enhsCIUiym6\r
 100 4qY3Sdoh7Ogi5IcKLzlJxAAowgLm2u6LyIJzKK+UJfXYCMgv+CUWO5dASQt26xuD\r
 101 6YoF30quhRhiwVjwEll0INONJdrjnNelfVQte2AltPaNvPTF8brniPveKE3bpSlC\r
 102 ywU881Ep4FjJO9ioS9Qe\r
 103 =Lbaw\r
 104 -----END PGP SIGNATURE-----\r
 105 --=-=-=--\r