Re: [PATCH 1/1] emacs: Make notmuch-show-next-thread return nil on failure

Subject: Re: [PATCH 1/1] emacs: Make notmuch-show-next-thread return nil on failure

Date: Sat, 30 Apr 2022 13:18:07 -0300

To: Leo Okawa Ericson, Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: David Bremner


Leo Okawa Ericson <git@relevant-information.com> writes:

> Tomi Ollila <tomi.ollila@iki.fi> writes:
>
>>
>> I don't see why this indentation change is happening...
>>
>
> I thought the indentation was inconsistent with the rest of the file.  I
> tried running the auto-indenter on a other more complex functions which
> changed nothing, but it did change it on this function so I assumed it
> was good to include it.  If it's bad etiquette, wrong or unnecessary I'm happy
> to amend the patch.

In general indentation (and whitespace) cleanup is fine, but should be
seperate commits labelled as such.

d
_______________________________________________
notmuch mailing list -- notmuch@notmuchmail.org
To unsubscribe send an email to notmuch-leave@notmuchmail.org

Thread: