Re: [PATCH] new: Don't scan unchanged directories with no sub-directories
[notmuch-archives.git] / 28 / 3bc48aba4dd5debbf691b51d921da431911e92
1 Return-Path: <jani@nikula.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 123D1431FBC\r
6         for <notmuch@notmuchmail.org>; Mon, 18 Jun 2012 23:57:21 -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 rjTsEoPguXfA for <notmuch@notmuchmail.org>;\r
16         Mon, 18 Jun 2012 23:57:20 -0700 (PDT)\r
17 Received: from mail-gh0-f181.google.com (mail-gh0-f181.google.com\r
18         [209.85.160.181]) (using TLSv1 with cipher RC4-MD5 (128/128 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id 72A7F431FAF\r
21         for <notmuch@notmuchmail.org>; Mon, 18 Jun 2012 23:57:20 -0700 (PDT)\r
22 Received: by ghbz13 with SMTP id z13so5720770ghb.26\r
23         for <notmuch@notmuchmail.org>; Mon, 18 Jun 2012 23:57:20 -0700 (PDT)\r
24 X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;\r
25         d=google.com; s=20120113;\r
26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
27         :mime-version:content-type:x-gm-message-state;\r
28         bh=VOKSaaw517yLSz6MMLvf8IrWemPzMg4Xkfv92QCMvRQ=;\r
29         b=oIBGGtM+SPmLFnHE8YkIZndrZpQu8AQPuAsHNpVwFEy9a7RttdLZlfrEtiUXrAj/ut\r
30         kloX5JJRUNfZBQ3Zlv+vLe6u0WDkJmFv99iRM9Dk7M7MYoClZsxnqWkVMRebxyOz5cDd\r
31         4gpFmEuCUYPkkUuUWfjjdCG2Ejs9oDPosDUtxompwI1Otn/UevZ/4XRYw9tehuB3FcM2\r
32         flXHVGbC3dtW/2VTOV9aMAw+9YyIOVeb9Zj8OrK9HsupnSacCX+B3Ic3Aiyhq9RgkEzl\r
33         HgZlxDfQl4TJrbyoQPnXALax6qHQhi/xdJhYulfF0NT6pfQ2FF+oCHmpHI92Hp//O+tZ\r
34         4a8w==\r
35 Received: by 10.236.73.4 with SMTP id u4mr21277675yhd.78.1340089039973;\r
36         Mon, 18 Jun 2012 23:57:19 -0700 (PDT)\r
37 Received: from localhost ([2001:4b98:dc0:43:216:3eff:fe1b:25f3])\r
38         by mx.google.com with ESMTPS id j34sm33728870ani.14.2012.06.18.23.57.19\r
39         (version=SSLv3 cipher=OTHER); Mon, 18 Jun 2012 23:57:19 -0700 (PDT)\r
40 From: Jani Nikula <jani@nikula.org>\r
41 To: Jameson Graef Rollins <jrollins@finestructure.net>,\r
42  notmuch@notmuchmail.org\r
43 Subject: Re: [PATCH] emacs: add function to toggle display of all\r
44         multipart/alternative parts\r
45 In-Reply-To: <87zk80gvmb.fsf@servo.finestructure.net>\r
46 References: <1339775751-18002-1-git-send-email-jani@nikula.org>\r
47         <87zk80gvmb.fsf@servo.finestructure.net>\r
48 User-Agent: Notmuch/0.13.2+60~g0f46448 (http://notmuchmail.org) Emacs/23.1.1\r
49         (i686-pc-linux-gnu)\r
50 Date: Tue, 19 Jun 2012 06:57:17 +0000\r
51 Message-ID: <87ehpbyek2.fsf@nikula.org>\r
52 MIME-Version: 1.0\r
53 Content-Type: text/plain; charset=us-ascii\r
54 X-Gm-Message-State:\r
55  ALoCoQl+1NhA6AEEp+RbUIIj+DQfu7YdC5o7jbwp4Jsor+4KlWAnyYR6w/QnxJb3rz7a8BbY3LBj\r
56 X-BeenThere: notmuch@notmuchmail.org\r
57 X-Mailman-Version: 2.1.13\r
58 Precedence: list\r
59 List-Id: "Use and development of the notmuch mail system."\r
60         <notmuch.notmuchmail.org>\r
61 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
62         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
63 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
64 List-Post: <mailto:notmuch@notmuchmail.org>\r
65 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
66 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
67         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
68 X-List-Received-Date: Tue, 19 Jun 2012 06:57:21 -0000\r
69 \r
70 On Mon, 18 Jun 2012, Jameson Graef Rollins <jrollins@finestructure.net> wrote:\r
71 > On Fri, Jun 15 2012, Jani Nikula <jani@nikula.org> wrote:\r
72 >> Add function notmuch-show-toggle-multipart-alternative to make\r
73 >> notmuch-show-all-multipart/alternative-parts buffer local, toggle its\r
74 >> value in the buffer, and redisplay the thread with either all or\r
75 >> preferred alternative parts expanded.\r
76 >\r
77 > This is a very cool feature.  I haven't looked closely at this but can't\r
78 > this mechanism work the same as the other show-mode toggles, like crypto\r
79 > or indenting?  I wouldn't think it would require any more extra variable\r
80 > stuff than what is used for those other toggles.\r
81 \r
82 I don't know how I missed those - thanks for pointing them out to\r
83 me. They seem to do the extra variable stuff at the top level, alongside\r
84 the defvar, rather than in the toggle function like I do. I should\r
85 rearrange this toggle to be in line with the others.\r
86 \r
87 > Also, the *really* sweet thing would be if the toggle actually cycled\r
88 > through display of the alternatives, only displaying one at a time.\r
89 > That would be very swank.\r
90 \r
91 It is easy to agree with that. At a glance, it looks like it needs a\r
92 bunch more work than the simple toggle in this patch, though. One\r
93 difficulty is choosing the parts to cycle, as we may have multiple\r
94 messages with different sets of multipart/alternative in the same show\r
95 buffer. And the cycle would be buffer local, not "message local".\r
96 \r
97 I think it would be possible to have show/hide buttons for the parts,\r
98 and the buttons could add/remove the part type from a buffer local list\r
99 of alternative parts to show, and then refresh view. But that too would\r
100 affect all messages in the buffer, not just one part of one message.\r
101 \r
102 In short, with my limited elisp skills and even more limited time, I\r
103 don't see a nice solution this. That doesn't say anything about the\r
104 possible existence of a nice solution, though. ;)\r
105 \r
106 \r
107 BR,\r
108 Jani.\r