ec/d39f5af1f60d0838ef9c88ee3148130edd46d5

   1 Return-Path: <dmitry.kurochkin@gmail.com>\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 9E612431FAF\r
   6         for <notmuch@notmuchmail.org>; Fri,  3 Feb 2012 22:12:14 -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.799\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=-0.799 tagged_above=-999 required=5\r
  12         tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1,\r
  13         FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_LOW=-0.7] autolearn=disabled\r
  14 Received: from olra.theworths.org ([127.0.0.1])\r
  15         by localhost (olra.theworths.org [127.0.0.1]) (amavisd-new, port 10024)\r
  16         with ESMTP id mQN9yDOGe07E for <notmuch@notmuchmail.org>;\r
  17         Fri,  3 Feb 2012 22:12:14 -0800 (PST)\r
  18 Received: from mail-bk0-f53.google.com (mail-bk0-f53.google.com\r
  19         [209.85.214.53]) (using TLSv1 with cipher RC4-SHA (128/128 bits))\r
  20         (No client certificate requested)\r
  21         by olra.theworths.org (Postfix) with ESMTPS id B0705431FAE\r
  22         for <notmuch@notmuchmail.org>; Fri,  3 Feb 2012 22:12:13 -0800 (PST)\r
  23 Received: by bke11 with SMTP id 11so4097351bke.26\r
  24         for <notmuch@notmuchmail.org>; Fri, 03 Feb 2012 22:12:12 -0800 (PST)\r
  25 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma;\r
  26         h=from:to:subject:in-reply-to:references:user-agent:date:message-id\r
  27         :mime-version:content-type:content-transfer-encoding;\r
  28         bh=pY1DfEV71xcyzBocBqO/60M7/nslxIo5zPkwH/0k5o8=;\r
  29         b=VpfJz4ijjVdHWlThMjDJ2hafhiCHA8wcbPuGrecp1V6C/0xXqfsFUo1Mwf5fK2C/H5\r
  30         heTGinbLwAW08k0cL28Zvl8I05CxNH7jP+WyrZUfbz5zLCjC9Nbx2GWhd/PTtE1zdaGf\r
  31         YoxLRjv9ZHvVW7jzLj/3jY6aiU5f6seKTCOm4=\r
  32 Received: by 10.204.157.17 with SMTP id z17mr4731923bkw.37.1328335931023;\r
  33         Fri, 03 Feb 2012 22:12:11 -0800 (PST)\r
  34 Received: from localhost ([91.144.186.21])\r
  35         by mx.google.com with ESMTPS id ek9sm23291155bkb.10.2012.02.03.22.12.10\r
  36         (version=TLSv1/SSLv3 cipher=OTHER);\r
  37         Fri, 03 Feb 2012 22:12:10 -0800 (PST)\r
  38 From: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>\r
  39 To: Adam Wolfe Gordon <awg+notmuch@xvx.ca>, notmuch@notmuchmail.org\r
  40 Subject: Re: [PATCH 0/2] Bug and potential fix: forwarded messages\r
  41 In-Reply-To:\r
  42  <CAMoJFUsfPwmLPxhK_mCbJ7N66u5+62QQne-oNJjVawSUzfFrjg@mail.gmail.com>\r
  43 References: <1328315554-16085-1-git-send-email-awg+notmuch@xvx.ca>\r
  44         <CAMoJFUsfPwmLPxhK_mCbJ7N66u5+62QQne-oNJjVawSUzfFrjg@mail.gmail.com>\r
  45 User-Agent: Notmuch/0.11+139~gd9b7cab (http://notmuchmail.org) Emacs/23.3.1\r
  46         (x86_64-pc-linux-gnu)\r
  47 Date: Sat, 04 Feb 2012 10:10:55 +0400\r
  48 Message-ID: <87obtfi0wg.fsf@gmail.com>\r
  49 MIME-Version: 1.0\r
  50 Content-Type: text/plain; charset=utf-8\r
  51 Content-Transfer-Encoding: quoted-printable\r
  52 X-BeenThere: notmuch@notmuchmail.org\r
  53 X-Mailman-Version: 2.1.13\r
  54 Precedence: list\r
  55 List-Id: "Use and development of the notmuch mail system."\r
  56         <notmuch.notmuchmail.org>\r
  57 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  58         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  59 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  60 List-Post: <mailto:notmuch@notmuchmail.org>\r
  61 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  62 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  63         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  64 X-List-Received-Date: Sat, 04 Feb 2012 06:12:14 -0000\r
  65 \r
  66 On Fri, 3 Feb 2012 23:01:23 -0700, Adam Wolfe Gordon <awg+notmuch@xvx.ca> w=\r
  67 rote:\r
  68 > Oh, and I just noticed that Dmitry has already fixed this, probably in\r
  69 > a better way [1] (though I maintain that there still may be a problem\r
  70 > with the approach in general).  I clearly haven't been following the\r
  71 > list closely enough this week.\r
  72 >=20\r
  73 \r
  74 Yep, creating buttons for regions at the beginning of message is fixed.\r
  75 But I like the idea of not hiding the message if there is no text.\r
  76 Though, I think we should create the button, just not hide the text\r
  77 initially.\r
  78 \r
  79 This change would conflict with my fix, so we should probably wait until\r
  80 it is applied before working on this.\r
  81 \r
  82 Regards,\r
  83   Dmitry\r
  84 \r
  85 > [1] id:"1327926286-16680-1-git-send-email-dmitry.kurochkin@gmail.com"\r
  86 >=20\r
  87 > On Fri, Feb 3, 2012 at 17:32, Adam Wolfe Gordon <awg+notmuch@xvx.ca> wrot=\r
  88 e:\r
  89 > > Hi everyone,\r
  90 > >\r
  91 > > I encountered today a bug that I remember being mentioned on IRC where\r
  92 > > some messages are shown in emacs as completely empty, even though their\r
  93 > > content is present and simple. For me, the bug shows up with messages t=\r
  94 hat\r
  95 > > have been forwarded by an Outlook user who didn't add any content to th=\r
  96 e top\r
  97 > > of the message. The first line of the message is "-----Original Message=\r
  98 -----",\r
  99 > > and the rest is the forwarded text. Notmuch tries to collapse the forwa=\r
 100 rded\r
 101 > > content into a button, but fails mysteriously.\r
 102 > >\r
 103 > > The first patch in this series adds a test (marked as broken) that demo=\r
 104 nstrates\r
 105 > > the bug. I think this should be pushed regardless of whether my solutio=\r
 106 n is\r
 107 > > the right one. My solution is to check whether we're collapsing the ent=\r
 108 ire\r
 109 > > message before doing so, and avoid turning it into a button if that's t=\r
 110 he case.\r
 111 > > I think this is a desirable behavior, since if someone has forwarded a =\r
 112 message\r
 113 > > without adding anything the user probably wants to read that message. B=\r
 114 ut,\r
 115 > > I'll admit that I didn't figure out the real cause of the problem, and I\r
 116 > > would be happy to hear other suggestions.\r
 117 > >\r
 118 > > It also occurs to me that this might indicate a bigger problem with how\r
 119 > > notmuch-wash.el handles messages starting with "-----Original Message--=\r
 120 ---".\r
 121 > > Notmuch seems to assume that this indicates the rest of the message is =\r
 122 quoted\r
 123 > > stuff that's been top-posted on. In my office this isn't necessarily th=\r
 124 e case,\r
 125 > > since Outlook produces that line at the top of every reply, and it's up=\r
 126  to\r
 127 > > the user whether to top-post or not (and not everyone does).\r
 128 > >\r
 129 > > I'll have to experiment a bit more to verify whether a problem (i.e. wh=\r
 130 ether\r
 131 > > Notmuch hides inappropriate things when someone replies inline with Out=\r
 132 look).\r
 133 > > Someone please correct me if I'm missing something in how the code oper=\r
 134 ates.\r
 135 > >\r
 136 > > Adam Wolfe Gordon (2):\r
 137 > > =C2=A0test: Add broken test for showing forwarded messages\r
 138 > > =C2=A0emacs: Fix broken showing of forwarded messages.\r
 139 > >\r
 140 > > =C2=A0emacs/notmuch-wash.el | =C2=A0 =C2=A05 +++--\r
 141 > > =C2=A0test/emacs =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| =C2=A0 32 +=\r
 142 +++++++++++++++++++++++++++++++\r
 143 > > =C2=A02 files changed, 35 insertions(+), 2 deletions(-)\r
 144 > >\r
 145 > > --\r
 146 > > 1.7.5.4\r
 147 > >\r
 148 > _______________________________________________\r
 149 > notmuch mailing list\r
 150 > notmuch@notmuchmail.org\r
 151 > http://notmuchmail.org/mailman/listinfo/notmuch\r