[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