Re: [PATCH] emacs: wash: make word-wrap bound message width
[notmuch-archives.git] / 5c / e3da925fb31dca0f96328a3806e72285ed0ec3
1 Return-Path: <bremner@unb.ca>\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 8C1EA431FAF\r
6         for <notmuch@notmuchmail.org>; Mon, 27 Feb 2012 17:52:49 -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: 0\r
10 X-Spam-Level: \r
11 X-Spam-Status: No, score=0 tagged_above=-999 required=5 tests=[none]\r
12         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 GPmYH9kJLO9b for <notmuch@notmuchmail.org>;\r
16         Mon, 27 Feb 2012 17:52:49 -0800 (PST)\r
17 Received: from tesseract.cs.unb.ca (tesseract.cs.unb.ca [131.202.240.238])\r
18         (using TLSv1 with cipher AES256-SHA (256/256 bits))\r
19         (No client certificate requested)\r
20         by olra.theworths.org (Postfix) with ESMTPS id 03C89431FAE\r
21         for <notmuch@notmuchmail.org>; Mon, 27 Feb 2012 17:52:48 -0800 (PST)\r
22 Received: from fctnnbsc30w-142166230117.dhcp-dynamic.fibreop.nb.bellaliant.net\r
23         ([142.166.230.117] helo=zancas.localnet)\r
24         by tesseract.cs.unb.ca with esmtpsa (TLS1.0:RSA_AES_256_CBC_SHA1:32)\r
25         (Exim 4.72) (envelope-from <bremner@unb.ca>)\r
26         id 1S2CF9-0004bY-Ks; Mon, 27 Feb 2012 21:52:45 -0400\r
27 Received: from bremner by zancas.localnet with local (Exim 4.77)\r
28         (envelope-from <bremner@unb.ca>)\r
29         id 1S2CF4-0008O2-6T; Mon, 27 Feb 2012 21:52:38 -0400\r
30 From: David Bremner <david@tethera.net>\r
31 To: David Edmondson <dme@dme.org>, notmuch@notmuchmail.org\r
32 Subject: Re: [PATCH 2/2 v2] emacs: Prefer '[No Subject]' to blank subjects.\r
33 In-Reply-To: <1327918561-16245-3-git-send-email-dme@dme.org>\r
34 References: <1327496913-4946-1-git-send-email-dme@dme.org>\r
35         <1327918561-16245-1-git-send-email-dme@dme.org>\r
36         <1327918561-16245-3-git-send-email-dme@dme.org>\r
37 User-Agent: Notmuch/0.11.1+240~g30b60bb (http://notmuchmail.org) Emacs/23.3.1\r
38         (x86_64-pc-linux-gnu)\r
39 Date: Mon, 27 Feb 2012 21:52:38 -0400\r
40 Message-ID: <8762eriv15.fsf@zancas.localnet>\r
41 MIME-Version: 1.0\r
42 Content-Type: text/plain; charset=us-ascii\r
43 X-Spam_bar: -\r
44 Cc: =?utf-8?Q?Antoin?= =?utf-8?Q?e_Beaupr=C3=A9?= <anarcat@anarcat.ath.cx>\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: Tue, 28 Feb 2012 01:52:49 -0000\r
58 \r
59 On Mon, 30 Jan 2012 10:16:01 +0000, David Edmondson <dme@dme.org> wrote:\r
60 > ---\r
61 >  emacs/notmuch-lib.el   |    9 +++++++++\r
62 >  emacs/notmuch-print.el |    8 ++++++--\r
63 >  emacs/notmuch-show.el  |    5 ++++-\r
64 >  emacs/notmuch.el       |   21 +++++++++------------\r
65 >  4 files changed, 28 insertions(+), 15 deletions(-)\r
66 \r
67 Jameson and Antoine have asked for this patch to be reverted. As far as\r
68 I understand it the printing code actually crashes on blank subjects if\r
69 this patch is removed, so it seems to me that that (slightly) outweighs\r
70 the annoyance caused by printing [No Subject].  I don't find the\r
71 argument that different people are annoyed\r
72 \r
73 I'd be willing to be convinced otherwise, but as it turns out the\r
74 question is somewhat academic; there have been enough changes that this\r
75 patch no longer reverts cleanly. So Someone (TM) will have to do some to\r
76 change the behaviour; it sounds like the sensible thing to do would be to move\r
77 notmuch-prettify-subject into notmuch-print.el and call it there.\r
78 \r
79 d\r