On Thu, 26 May 2011 03:10:11 +0400, Dmitry Kurochkin <dmitry.kuroch...@gmail.com> wrote: > On Wed, 25 May 2011 15:46:40 -0700, Carl Worth <cwo...@cworth.org> wrote: > Well, emacs trunk is not broken :) The bug is in lisp code, so you can > fix it in .emacs by redefining `isearch-range-invisible' function. I do > that now.
Oh, in that case we can fix this is notmuch emacs lisp by just defining and using a fixed function. Is the broken function something we're calling directly? Or is it being called indirectly? (being called by other emacs lisp code that we are calling)? If we can incorporate the fix, that would be great. > Please consider pushing other patches from the series. They do not fix > any bug, but do simplify the code. The last patch uses list for > invisible overlay property as well. But it does not break isearch > because we do not search in hidden messages. Hmmm... we should probably do that. I'd like isearch in notmuch to search anything that is hidden. > BTW would be nice to have a set of known-to-fail tests, i.e. bugs that > are not fixed yet. If we had it, the above test could be implemented > and committed before we have the fix pushed. We do! Use test_expect_equal_failure (yes, the name is horrible!) instead of test_expect_equal and you should get what you want. -Carl -- carl.d.wo...@intel.com
pgp1vAI5Cj5J5.pgp
Description: PGP signature
_______________________________________________ notmuch mailing list notmuch@notmuchmail.org http://notmuchmail.org/mailman/listinfo/notmuch