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

2013-06-02 Thread David Bremner
Mark Walters 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

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

2013-06-02 Thread Mark Walters
On Sun, 02 Jun 2013, david at tethera.net wrote: > From: David Bremner > > I found several places where a setq is immediately followed by a let > or a let*. This seems to be the pessimal combination, with the > implicit scope of the setq combined with the extra indentation of the let. > I

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

2013-06-02 Thread da...@tethera.net
From: David Bremner I found several places where a setq is immediately followed by a let or a let*. This seems to be the pessimal combination, with the implicit scope of the setq combined with the extra indentation of the let. I combined these cases into a single let* which I

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

2013-06-02 Thread david
From: David Bremner brem...@debian.org I found several places where a setq is immediately followed by a let or a let*. This seems to be the pessimal combination, with the implicit scope of the setq combined with the extra indentation of the let. I combined these cases into a single let* which I

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

2013-06-02 Thread Mark Walters
On Sun, 02 Jun 2013, da...@tethera.net wrote: From: David Bremner brem...@debian.org I found several places where a setq is immediately followed by a let or a let*. This seems to be the pessimal combination, with the implicit scope of the setq combined with the extra indentation of the let.

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

2013-06-02 Thread David Bremner
Mark Walters markwalters1...@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)))