[PATCH 2/2] emacs: Fix "no such file or directory" error
Tomi Ollila
tomi.ollila at iki.fi
Sun Jun 9 23:21:29 PDT 2013
On Mon, Jun 10 2013, Austin Clements <amdragon at MIT.EDU> wrote:
> 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.
I also looked this a bit yesterday evening. For example this page
http://www.gnu.org/software/emacs/manual/html_node/elisp/Filter-Functions.html
discusses only about catching thrown errors -- i.e. no mention about
filter function return values. From that I'd draw a conclusion that
most probably the return value of filter function is not used for anything.
... and the patch looks good. +1 (removing needs-review)
Tomi
More information about the notmuch
mailing list