[PATCH 2/2] emacs: Fix "no such file or directory" error

Austin Clements amdragon at MIT.EDU
Sun Jun 9 19:15:34 PDT 2013


Quoth Mark Walters on Jun 09 at 10:16 am:
> 
> Both of these patches look good to me +1. I was able to reproduce both
> bugs pretty reliably (the first one always unless masked by the second
> one which occurred about half the time). With these patches I cannot
> trigger either. Also all tests pass.
> 
> My only tiny concern is I couldn't find any documentation on whether the
> return value of the filter-function matters at all. Austin's original
> fix (via irc) returned t and this returns nil in the failing case (i.e.,
> when results-buf is dead).

Mm, interesting.  To be fair, my choice of "t" for the original fix
was completely arbitrary.  I think you're right that the Emacs
documentation doesn't have anything to say about the return values of
filter functions.  Furthermore, the example filter functions they give
don't have meaningful return values, so I'm pretty sure this is safe.
Also the code that calls the filter discards its result.


More information about the notmuch mailing list