[PATCH] emacs: replace setq + let with let*

David Bremner david at tethera.net
Sun Jun 2 17:01:23 PDT 2013


Mark Walters <markwalters1009 at gmail.com> writes:

>> -    (setq search (notmuch-hello-trim search))
>> -    (let ((history-delete-duplicates t))
>> +    (let* ((search (notmuch-hello-trim search))
>> +	   (history-delete-duplicates t))
>>        (add-to-history 'notmuch-search-history search)))
>>    (notmuch-search search notmuch-search-oldest-first nil nil
>
> These look good to me except I don't see why the above is a let* not a
> let?

I ended up dropping this hunk because I realized it introduced a bug;
the side-effect of the setq is needed outside the unless.

pushed the two patches (as amended) in this thread.

d





More information about the notmuch mailing list