[PATCH v4 0/6] Config-related patches

Jameson Graef Rollins jrollins at finestructure.net
Sat Apr 14 12:33:03 PDT 2012


On Sat, Apr 14 2012, Mark Walters <markwalters1009 at gmail.com> wrote:
> Two minor comments which you might like to consider (but definitely are
> not required):
>     1) You could check that there are no further arguments when the user
>     calls `notmuch config list'

This doesn't seem so necessary to me.  I'm fine if it just silently
ignores extra arguments.

>     2) In the man page you could explicitly say what the output is for a
>     configuration item which has not been set.

I'm also fine with it as it is.  I think it's implied that if the value
is null then the field is left blank.  That's ok.

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/20120414/05e9d046/attachment.pgp>


More information about the notmuch mailing list