f1/4f131bca2a004fb27340183f96eea41e5241b3

   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 1D98A431FBC\r
   6         for <notmuch@notmuchmail.org>; Sun,  1 Sep 2013 09:18:55 -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: -2.3\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-2.3 tagged_above=-999 required=5\r
  12         tests=[RCVD_IN_DNSWL_MED=-2.3] 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 Cu-pKHWVXGL7 for <notmuch@notmuchmail.org>;\r
  16         Sun,  1 Sep 2013 09:18:47 -0700 (PDT)\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 AAF4C431FC0\r
  20         for <notmuch@notmuchmail.org>; Sun,  1 Sep 2013 09:18:47 -0700 (PDT)\r
  21 Received: from earth-doxen.imss.caltech.edu (localhost [127.0.0.1])\r
  22         by earth-doxen-postvirus (Postfix) with ESMTP id 3019866E0195;\r
  23         Sun,  1 Sep 2013 09:18:45 -0700 (PDT)\r
  24 X-Spam-Scanned: at Caltech-IMSS on earth-doxen by amavisd-new\r
  25 Received: from finestructure.net (cpe-76-173-75-27.socal.res.rr.com\r
  26         [76.173.75.27]) (Authenticated sender: jrollins)\r
  27         by earth-doxen-submit (Postfix) with ESMTP id 27BE666E01A6;\r
  28         Sun,  1 Sep 2013 09:18:40 -0700 (PDT)\r
  29 Received: by finestructure.net (Postfix, from userid 1000)\r
  30         id 93EDF60088; Sun,  1 Sep 2013 09:18:37 -0700 (PDT)\r
  31 From: Jameson Graef Rollins <jrollins@finestructure.net>\r
  32 To: Mark Walters <markwalters1009@gmail.com>, notmuch@notmuchmail.org,\r
  33         amdragon@MIT.EDU\r
  34 Subject: Re: [PATCH] emacs: show: lazy part bugfix\r
  35 In-Reply-To: <1377246875-7784-1-git-send-email-markwalters1009@gmail.com>\r
  36 References: <1377246875-7784-1-git-send-email-markwalters1009@gmail.com>\r
  37 User-Agent: Notmuch/0.16+1~g9f861e2 (http://notmuchmail.org) Emacs/24.3.1\r
  38         (x86_64-pc-linux-gnu)\r
  39 Date: Sun, 01 Sep 2013 09:18:35 -0700\r
  40 Message-ID: <871u58a52c.fsf@servo.finestructure.net>\r
  41 MIME-Version: 1.0\r
  42 Content-Type: multipart/signed; boundary="=-=-=";\r
  43         micalg=pgp-sha256; protocol="application/pgp-signature"\r
  44 X-BeenThere: notmuch@notmuchmail.org\r
  45 X-Mailman-Version: 2.1.13\r
  46 Precedence: list\r
  47 List-Id: "Use and development of the notmuch mail system."\r
  48         <notmuch.notmuchmail.org>\r
  49 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  50         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  51 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  52 List-Post: <mailto:notmuch@notmuchmail.org>\r
  53 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  54 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  55         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  56 X-List-Received-Date: Sun, 01 Sep 2013 16:18:55 -0000\r
  57 \r
  58 --=-=-=\r
  59 Content-Type: text/plain\r
  60 Content-Transfer-Encoding: quoted-printable\r
  61 \r
  62 On Fri, Aug 23 2013, Mark Walters <markwalters1009@gmail.com> wrote:\r
  63 > The lazy part handling had a subtle bug. Notmuch stores the part\r
  64 > information as a text property with the displayed part so attachment\r
  65 > handling (saving viewing etc work).\r
  66 >\r
  67 > Now, some mime parts have subparts and to avoid overwriting the\r
  68 > sub-part data notmuch checks and if part data is already recorded it\r
  69 > does not overwrite it.\r
  70 >\r
  71 > Now with lazy part handling this could fail: there is already part\r
  72 > data stored. In the common case it works as the part type information\r
  73 > was stored when the lazy-part button was inserted. However, this fails\r
  74 > if the lazy part has sub-parts: notmuch had no idea these existed\r
  75 > until the lazy part insertion.\r
  76 >\r
  77 > We fix this by removing any existing part-information from the\r
  78 > relevant region before doing the lazy insertion.\r
  79 > ---\r
  80 > This was shown up by Istvan's patch in id:m3r4dtgz9k.fsf@zsu.kismala.com =\r
  81 and=20\r
  82 > the bug found by Jamie in id:87fvu4fl25.fsf@servo.finestructure.net\r
  83 >\r
  84 > I think this is essentially the right patch: I am not certain about\r
  85 > the +1 in the removing the property. It seems to be needed but maybe\r
  86 > something with front/back sticky would be better.\r
  87 >\r
  88 > Also this definitely needs more testing before going into master: this\r
  89 > code is definitely fragile.\r
  90 \r
  91 Hey, Mark.  I have just tested this patch and it seems to be fixing the\r
  92 issue I was seeing.  With Istvan's patch the once hidden parts are now\r
  93 exposed with buttons, and with this patch the buttons now work as\r
  94 expected.\r
  95 \r
  96 I can't really speak to the correctness of this patch, but it does seem\r
  97 to be working, and doesn't seem to have any adverse side effects that I\r
  98 can see.\r
  99 \r
 100 As a reminder, this patch and Istvan's do fix an actual bug in notmuch,\r
 101 so both should be pushed unless anyone see's anything wrong with either\r
 102 of them.\r
 103 \r
 104 Thanks so much for the fixes, guys.\r
 105 \r
 106 jamie.\r
 107 \r
 108 --=-=-=\r
 109 Content-Type: application/pgp-signature\r
 110 \r
 111 -----BEGIN PGP SIGNATURE-----\r
 112 Version: GnuPG v1.4.14 (GNU/Linux)\r
 113 \r
 114 iQIcBAEBCAAGBQJSI2jbAAoJEO00zqvie6q8NKEQALkAsW0q5KT9Ata/LJVLaUaO\r
 115 NHre3YBLR08QbiBAOr2Ey5hQBx06dswsxXUMD+WesgADSwN0+cS52MTnMtVW5evY\r
 116 62FYCWiu4pRsyCnBk0PgbmaYbrnu2UPaz7HmJiT2pFCa2S6TVrsoEilnJOUjCTaz\r
 117 eULUbfxIM3SesbJsZmIoU1pj9r7CwtYRuMm1gFes8TK0d+39f8jmQSKAMviF4iN7\r
 118 1E2DaN/t6IQH2OYWRIGmuXT0hJ9/nw4CRIEsYT54QEvyIf06OKIW3wHZY7Stu42W\r
 119 VoIqi2/IjCjI/BM0Ai2JRANXYKOUoObU2CUBpgf83f/x6Kudp1H714H5jL87eOB4\r
 120 3wMrVR+qDGT09L/ckQmPpDZ0a6iyCkINxKKR7NNJbe2dfGOvfek+du3VGbax0HSG\r
 121 NOv5CSIaZbV/Ed9vfPWgFwVmnZSZmutN4AvRjMyNFcIJzZOE3QhRFPJYIOKldxK4\r
 122 4vNb+pqja7XlC7rcqJ46uZEAT84kb6EifvB7VM30u2kqkNndGcx+IIF0MctBXn2P\r
 123 QbXXEWdQfG6JARcpuBTmaoDbrLtyhh8ZGXxXMxY9wDdNeEW8fzR7GcPW91U/y0bN\r
 124 7tj0g856qpHmU+2+hF/v271tVtipAF67i7ktfiXPp+w9ZRcz/ThAGG/Z9dNQvIT3\r
 125 qzCKGKCE92F87BQ5GSeg\r
 126 =EMOu\r
 127 -----END PGP SIGNATURE-----\r
 128 --=-=-=--\r