10/826587feb162667bf232e81cff6b511d281152

   1 Return-Path: <cworth@cworth.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 C03EC429E5A\r
   6         for <notmuch@notmuchmail.org>; Wed, 25 May 2011 18:03:03 -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.01\r
  10 X-Spam-Level: \r
  11 X-Spam-Status: No, score=0.01 tagged_above=-999 required=5\r
  12         tests=[T_MIME_NO_TEXT=0.01] 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 99pI2okTC6Nl for <notmuch@notmuchmail.org>;\r
  16         Wed, 25 May 2011 18:03:03 -0700 (PDT)\r
  17 Received: from arlo.cworth.org (arlo.cworth.org [50.43.72.2])\r
  18         by olra.theworths.org (Postfix) with ESMTP id 2AA06429E54\r
  19         for <notmuch@notmuchmail.org>; Wed, 25 May 2011 18:02:57 -0700 (PDT)\r
  20 Received: from yoom.home.cworth.org (localhost [127.0.0.1])\r
  21         by arlo.cworth.org (Postfix) with ESMTP id 2CDE229A4FA;\r
  22         Wed, 25 May 2011 18:02:56 -0700 (PDT)\r
  23 Received: by yoom.home.cworth.org (Postfix, from userid 1000)\r
  24         id 1EB7525417E; Wed, 25 May 2011 18:02:56 -0700 (PDT)\r
  25 From: Carl Worth <cworth@cworth.org>\r
  26 To: Dmitry Kurochkin <dmitry.kurochkin@gmail.com>, notmuch@notmuchmail.org\r
  27 Subject: Re: [PATCH 3/5] Fix hiding a message while some citations are shown\r
  28         in notmuch-show view.\r
  29 In-Reply-To: <87hb8ipeb0.fsf@gmail.com>\r
  30 References: <1306361416-5019-1-git-send-email-dmitry.kurochkin@gmail.com>\r
  31         <1306361416-5019-4-git-send-email-dmitry.kurochkin@gmail.com>\r
  32         <87oc2qa07g.fsf@yoom.home.cworth.org> <87k4depfyj.fsf@gmail.com>\r
  33         <87lixu9z5b.fsf@yoom.home.cworth.org> <87hb8ipeb0.fsf@gmail.com>\r
  34 User-Agent: Notmuch/0.5 (http://notmuchmail.org) Emacs/23.3.1\r
  35         (i486-pc-linux-gnu)\r
  36 Date: Wed, 25 May 2011 18:02:49 -0700\r
  37 Message-ID: <87ipsy9sue.fsf@yoom.home.cworth.org>\r
  38 MIME-Version: 1.0\r
  39 Content-Type: multipart/signed; boundary="=-=-=";\r
  40         micalg=pgp-sha1; protocol="application/pgp-signature"\r
  41 X-BeenThere: notmuch@notmuchmail.org\r
  42 X-Mailman-Version: 2.1.13\r
  43 Precedence: list\r
  44 List-Id: "Use and development of the notmuch mail system."\r
  45         <notmuch.notmuchmail.org>\r
  46 List-Unsubscribe: <http://notmuchmail.org/mailman/options/notmuch>,\r
  47         <mailto:notmuch-request@notmuchmail.org?subject=unsubscribe>\r
  48 List-Archive: <http://notmuchmail.org/pipermail/notmuch>\r
  49 List-Post: <mailto:notmuch@notmuchmail.org>\r
  50 List-Help: <mailto:notmuch-request@notmuchmail.org?subject=help>\r
  51 List-Subscribe: <http://notmuchmail.org/mailman/listinfo/notmuch>,\r
  52         <mailto:notmuch-request@notmuchmail.org?subject=subscribe>\r
  53 X-List-Received-Date: Thu, 26 May 2011 01:03:04 -0000\r
  54 \r
  55 --=-=-=\r
  56 Content-Transfer-Encoding: quoted-printable\r
  57 \r
  58 On Thu, 26 May 2011 03:10:11 +0400, Dmitry Kurochkin <dmitry.kurochkin@gmai=\r
  59 l.com> wrote:\r
  60 > On Wed, 25 May 2011 15:46:40 -0700, Carl Worth <cworth@cworth.org> wrote:\r
  61 > Well, emacs trunk is not broken :)  The bug is in lisp code, so you can\r
  62 > fix it in .emacs by redefining `isearch-range-invisible' function.  I do\r
  63 > that now.\r
  64 \r
  65 Oh, in that case we can fix this is notmuch emacs lisp by just defining\r
  66 and using a fixed function. Is the broken function something we're\r
  67 calling directly? Or is it being called indirectly? (being called by\r
  68 other emacs lisp code that we are calling)?\r
  69 \r
  70 If we can incorporate the fix, that would be great.\r
  71 \r
  72 > Please consider pushing other patches from the series.  They do not fix\r
  73 > any bug, but do simplify the code.  The last patch uses list for\r
  74 > invisible overlay property as well.  But it does not break isearch\r
  75 > because we do not search in hidden messages.\r
  76 \r
  77 Hmmm... we should probably do that. I'd like isearch in notmuch to\r
  78 search anything that is hidden.\r
  79 \r
  80 > BTW would be nice to have a set of known-to-fail tests, i.e. bugs that\r
  81 > are not fixed yet.  If we had it, the above test could be implemented\r
  82 > and committed before we have the fix pushed.\r
  83 \r
  84 We do! Use test_expect_equal_failure (yes, the name is horrible!)\r
  85 instead of test_expect_equal and you should get what you want.\r
  86 \r
  87 =2DCarl\r
  88 \r
  89 =2D-=20\r
  90 carl.d.worth@intel.com\r
  91 \r
  92 --=-=-=\r
  93 Content-Type: application/pgp-signature\r
  94 \r
  95 -----BEGIN PGP SIGNATURE-----\r
  96 Version: GnuPG v1.4.11 (GNU/Linux)\r
  97 \r
  98 iEYEARECAAYFAk3dprkACgkQ6JDdNq8qSWjOZACcChjCFqJrnrrBTSOMoaw4Cxy4\r
  99 lusAniCy0nJl47gVy+QUvbrMZm0Ijouy\r
 100 =R0BE\r
 101 -----END PGP SIGNATURE-----\r
 102 --=-=-=--\r