Quoth Pieter Praet on Jan 19 at 8:19 pm: > All other config-related functions and args include the section > title in their name, so for the sake of consistency, mirror that. > > Also, the "auto"matic part is a given, so that was dropped.
LGTM other than one nit, below. > --- a/notmuch-config.c > +++ b/notmuch-config.c > @@ -89,9 +89,9 @@ static const char search_config_comment[] = > "\n" > " The following option is supported here:\n" > "\n" > - "\tauto_exclude_tags A ;-separated list of tags that will be\n" > - "\t excluded from search results by default. Using an excluded tag\n" > - "\t in a query will override that exclusion.\n"; > + "\texclude_tags A list (separated by ';') of the tags that will > be\n" > + "\t\texcluded from search results by default. Using an excluded tag in > a\n" > + "\t\tquery will override that exclusion.\n"; I'd propose that any tag whose name is more than seven characters long should have its description start under it, indented by a tab, like I did in id:"[email protected]". That looks nice and keeps the description indented consistently without ridiculous amounts of negative space and narrow wrapping. _______________________________________________ notmuch mailing list [email protected] http://notmuchmail.org/mailman/listinfo/notmuch
