[PATCH 4/6] emacs: add option to notmuch-show-next-open-message to pop out to parent buffer if at end

Jameson Graef Rollins jrollins at finestructure.net
Mon Jan 23 10:51:06 PST 2012


On Mon, 23 Jan 2012 17:55:14 +0000, David Edmondson <dme at dme.org> wrote:
> On that basis, I hope no-one will complain if I fix them as a 'drive by'
> during another change...

Hrm.  I don't think that follows.  Unless the patch is already touching
that code, I would really prefer we continue to enforce that unrelated
changes go into separate patches.

There are plenty of other places in the code where we do use parens
around uninitialized variables.  If you really want to remove them I
would prefer to see a patch to fixes them all at once.  Doesn't seem
worth it to me, but everyone seems to be on an uncrustify kick recently,
so...

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/20120123/245d8309/attachment-0001.pgp>


More information about the notmuch mailing list