Re: [PATCH] notmuch.c: add missing trailing newlines in two error messages

Subject: Re: [PATCH] notmuch.c: add missing trailing newlines in two error messages

Date: Sun, 31 Jul 2022 15:57:52 -0300

To: Tomi Ollila, notmuch@notmuchmail.org

Cc: tomi.ollila@iki.fi

From: David Bremner


Tomi Ollila <tomi.ollila@iki.fi> writes:

> Removed duplicate error check (and the message) in 3rd case where
> the same error message (w/o trailing newline) was present.
>
> In case of test/T040-setup.sh, command substitution deletes
> trailing newlines, so related test there cannot be changed
> (and therefore could not notice this user experience flaw).
> ---
>
> The current error message can be experienced with
>
> $ NOTMUCH_CONFIG=no-cnfig-thre notmuch new
>     and (the second change, removed in this commit)
> $ NOTMUCH_CONFIG=no-cnfig-thre notmuch config

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

Thread: