Re: notmuch-search-toggle-order and notmuch-tree
[notmuch-archives.git] / af / 1761a058fd923d9d89d297dafbb298bda2871e
1 Return-Path: <amdragon@mit.edu>\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 D171D429E40\r
6         for <notmuch@notmuchmail.org>; Wed,  4 Sep 2013 09:16:41 -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.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 YGzxzqT-17Aw for <notmuch@notmuchmail.org>;\r
16         Wed,  4 Sep 2013 09:16:35 -0700 (PDT)\r
17 Received: from dmz-mailsec-scanner-2.mit.edu (dmz-mailsec-scanner-2.mit.edu\r
18         [18.9.25.13])\r
19         by olra.theworths.org (Postfix) with ESMTP id 86EFC429E34\r
20         for <notmuch@notmuchmail.org>; Wed,  4 Sep 2013 09:16:35 -0700 (PDT)\r
21 X-AuditID: 1209190d-b7f078e000000937-4b-52275ce2d49d\r
22 Received: from mailhub-auth-3.mit.edu ( [18.9.21.43])\r
23         by dmz-mailsec-scanner-2.mit.edu (Symantec Messaging Gateway) with SMTP\r
24         id 2B.3E.02359.2EC57225; Wed,  4 Sep 2013 12:16:34 -0400 (EDT)\r
25 Received: from outgoing.mit.edu (outgoing-auth-1.mit.edu [18.9.28.11])\r
26         by mailhub-auth-3.mit.edu (8.13.8/8.9.2) with ESMTP id r84GGVlN030500; \r
27         Wed, 4 Sep 2013 12:16:34 -0400\r
28 Received: from awakening.csail.mit.edu (awakening.csail.mit.edu [18.26.4.91])\r
29         (authenticated bits=0)\r
30         (User authenticated as amdragon@ATHENA.MIT.EDU)\r
31         by outgoing.mit.edu (8.13.8/8.12.4) with ESMTP id r84GGT31002437\r
32         (version=TLSv1/SSLv3 cipher=DHE-RSA-AES128-SHA bits=128 verify=NOT);\r
33         Wed, 4 Sep 2013 12:16:30 -0400\r
34 Received: from amthrax by awakening.csail.mit.edu with local (Exim 4.80)\r
35         (envelope-from <amdragon@mit.edu>)\r
36         id 1VHFkq-0005ac-Tn; Wed, 04 Sep 2013 12:16:28 -0400\r
37 Date: Wed, 4 Sep 2013 12:16:28 -0400\r
38 From: Austin Clements <amdragon@MIT.EDU>\r
39 To: Jameson Graef Rollins <jrollins@finestructure.net>\r
40 Subject: Re: [PATCH v2] emacs: show: lazy part bugfix\r
41 Message-ID: <20130904161628.GC1426@mit.edu>\r
42 References: <1377246875-7784-1-git-send-email-markwalters1009@gmail.com>\r
43         <1378279835-28288-1-git-send-email-markwalters1009@gmail.com>\r
44         <20130904145639.GB1426@mit.edu>\r
45         <87hae07fhs.fsf@servo.finestructure.net>\r
46 MIME-Version: 1.0\r
47 Content-Type: text/plain; charset=utf-8\r
48 Content-Disposition: inline\r
49 Content-Transfer-Encoding: 8bit\r
50 In-Reply-To: <87hae07fhs.fsf@servo.finestructure.net>\r
51 User-Agent: Mutt/1.5.21 (2010-09-15)\r
52 X-Brightmail-Tracker:\r
53  H4sIAAAAAAAAA+NgFlrIKsWRmVeSWpSXmKPExsUixCmqrfsoRj3I4NVUE4s9+7wsVs/lsbh+\r
54         cyazA7PH3dNcHjtn3WX3eLbqFnMAcxSXTUpqTmZZapG+XQJXxsFOkYJvAhU9uy+wNTDu4+1i\r
55         5OSQEDCRePH8NRuELSZx4d56IJuLQ0hgH6PExc4mRpCEkMAGRolbbUoQiVNMEms7u9ggEksY\r
56         JS5/SgGxWQRUJJZMvMoKYrMJaEhs278crFlEwEyi58sfMJtZwFVixoVdTCC2sICpxOqJZ9hB\r
57         bF4BbYnnE9cyQSy4yihx5fIzqISgxMmZT1ggmtUl/sy7xNzFyAFkS0ss/8cBEZaXaN46mxnE\r
58         5gSaOa9/Nli5KNA9U05uY5vAKDwLyaRZSCbNQpg0C8mkBYwsqxhlU3KrdHMTM3OKU5N1i5MT\r
59         8/JSi3SN9HIzS/RSU0o3MYJjQpJ3B+O7g0qHGAU4GJV4eBuM1YOEWBPLiitzDzFKcjApifIa\r
60         hwOF+JLyUyozEosz4otKc1KLDzFKcDArifDKeQPleFMSK6tSi/JhUtIcLErivE+fng0UEkhP\r
61         LEnNTk0tSC2CycpwcChJ8P6OBmoULEpNT61Iy8wpQUgzcXCCDOcBGs4eAzK8uCAxtzgzHSJ/\r
62         ilFRSpyXDyQhAJLIKM2D64WlrFeM4kCvCPPygFTxANMdXPcroMFMQIPTPquCDC5JREhJNTCW\r
63         Ol+o0V9b0v+2xs3fe7b78lNqZ6QXrGCUzg4+9+5Aifuv7P+SNVPOZC9aqHDRrWVPnI208b64\r
64         Kg0Nb0GT6TLnvRQOKt92mjXNRMhOzb6Y6co55dasCbqWYsfWbjsRFBTswMa/jEmoYMHEG0qf\r
65         985R6a+6uOvmtPSJH6+aqH48mWnFUxvgdlWJpTgj0VCLuag4EQA7K6l7NAMAAA==\r
66 Cc: notmuch@notmuchmail.org\r
67 X-BeenThere: notmuch@notmuchmail.org\r
68 X-Mailman-Version: 2.1.13\r
69 Precedence: list\r
70 List-Id: "Use and development of the notmuch mail system."\r
71         <notmuch.notmuchmail.org>\r
72 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
73         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
74 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
75 List-Post: <mailto:notmuch@notmuchmail.org>\r
76 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
77 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
78         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
79 X-List-Received-Date: Wed, 04 Sep 2013 16:16:42 -0000\r
80 \r
81 Quoth Jameson Graef Rollins on Sep 04 at  8:50 am:\r
82 > On Wed, Sep 04 2013, Austin Clements <amdragon@MIT.EDU> wrote:\r
83 > >> Now, some mime parts have subparts and to avoid overwriting the\r
84 > >> sub-part data notmuch checks and if part data is already recorded it\r
85 > >> does not overwrite it.\r
86 > >> \r
87 > >> Now with lazy part handling this could fail: there is already part\r
88 > >> data stored. In the common case it works as the part type information\r
89 > >> was stored when the lazy-part button was inserted. However, this fails\r
90 > >> if the lazy part has sub-parts: notmuch had no idea these existed\r
91 > >> until the lazy part insertion.\r
92 > >\r
93 > > This says that things fail when a lazy part has sub-parts, but not\r
94 > > what the failure is.  What is the failure?  Can you give a specific\r
95 > > sequence of events and conditions that leads to and demonstrates the\r
96 > > failure?\r
97 > >\r
98 > > (I ask not just for commit posterity, but because I actually don't\r
99 > > know, though I may have figured it out after writing the comment\r
100 > > below.)\r
101\r
102 > Hey, Austin.  Here's an example of a mail that is effected the issue:\r
103\r
104 > └┬╴multipart/alternative 896783 bytes\r
105 >  ├─╴text/plain 379 bytes\r
106 >  └┬╴multipart/related 892556 bytes\r
107 >   ├─╴text/html 1236 bytes\r
108 >   └─╴image/jpeg inline [photo.JPG] 890841 bytes\r
109\r
110 > The multipart/related part is initially hidden.  Without Istvan's patch,\r
111 > there would be no button at all for the image/jpeg part, even when the\r
112 > multipart/related is exposed.  With Istvan's patch the image/jpeg button\r
113 > is there, but without Mark's patch the button would actually reference\r
114 > the entire multipart/alternative part, instead of just the image/jpeg.\r
115 > If I tried to save the image/jpeg I would get the entire\r
116 > multipart/alternative mime structure in plain text.\r
117 \r
118 Ah.  Thanks for the example!  After staring at the patch for long\r
119 enough, I figured it had to be something like this, but still didn't\r
120 have anything concrete.  It would be great to include this in the\r
121 commit message to show exactly what was broken and what's being fixed.\r
122 \r
123 Now that I'm sure what the problem was, the code itself LGTM.\r