[RFC][PATCH v4] emacs: Re-implement advance/rewind functions of notmuch-show-mode.
Jameson Graef Rollins
jrollins at finestructure.net
Mon Dec 26 14:24:11 PST 2011
On Mon, 26 Dec 2011 22:00:21 +0000, David Edmondson <dme at dme.org> wrote:
> Understood. For me this fell inside the 'trivial other change' boundary.
Fwiw, I don't remember there ever being such a distinction. I think
we've always insisted that unrelated changes should be excluded. As a
general rule, I think all patches should be as atomic as they can
possibly be. I would much rather have more smaller, atomic patches than
fewer longer, composite ones.
> > And can you please explain why `when' is better than `if' here? Then I
> > will know which one to use the next time :)
>
> `if' allows only a single statement for `then', which results in code like:
>
> (if foo
> (progn
> (this)
> (that)
> (theother)))
>
> so if there is no `else' clause I've been preferring:
>
> (when foo
> (this)
> (that)
> (theother))
>
> but that's obviously personal and not important in this specific case.
That's good. I like it. The if construction always annoyed me a bit
for this reason. The when construction is definitely cleaner.
jamie.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 835 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20111226/374961f1/attachment-0001.pgp>
More information about the notmuch
mailing list