[PATCH 1/2] cli: convert "notmuch show" to use the new argument parser
Austin Clements
amdragon at MIT.EDU
Mon Feb 6 06:14:39 PST 2012
Quoth Jani Nikula on Feb 06 at 7:54 am:
> On Sun, 5 Feb 2012 23:12:05 -0500, Austin Clements <amdragon at MIT.EDU> wrote:
> > Yikes. I don't envy you this patch. Two minor nits, otherwise this
> > and the next patch LGTM.
>
> Thanks for the review. I'll roll another version, I think it will be
> better with your suggestions incorporated.
>
> > Quoth Jani Nikula on Feb 04 at 12:41 am:
> > > Use the new notmuch argument parser to handle arguments in "notmuch
> > > show". There are two corner case functional changes:
> > >
> > > 1) Also set params.raw = 1 when defaulting to raw format when part is
> > > requested but format is not specified.
> >
> > Huh. So "notmuch show --part=0 <query>" was broken before.
>
> Hmm, yes, it seems so. Do you think I should make this a separate fix?
I wouldn't bother. Since this usage would throw --format=raw into
"useless mode", clearly no one cared. No sense throwing good code
after bad.
More information about the notmuch
mailing list