Branko Čibej wrote on Thu, 07 Nov 2019 20:01 +00:00:
> On 07.11.2019 20:25, Daniel Shahaf wrote:
> >> We might want to allow using multiple options, like "--show-item=revision
> >> --show-item=kind" in addition to comma-separated values.
> > I don't think this is high priority.
> 
> *And* it would require a major rewrite of our argument parser, whereas
> adding a comma-separated list format would not, and would also be
> compatible with -r and -c.

"Major"?  We could just change opt_state.show_item from «const char *»
to an array, and parse multiple --show-item arguments the same way we
parse «svn log -r 42 -r 43».  I think it'd be a trivial patch, as no other
subcommand uses --show-item, but its cost/benefit ratio would be
pretty high.

Cheers,

Daniel

Reply via email to