Re: [PATCH] -cM-N command line syntax

[Peter Samuelson]
> Is this, then, a worthwhile feature addition? I don't want to add
> syntax that nobody else wants. In particular, this patch highlights
> the existing inconsistency of ":" vs. "-" as range operators.

Hmmm, in tests/cmdline/log_tests.py XFail(log_chanage_range), I see
Lieven thought -c might someday take ranges with ":" instead of "-".
But since the output uses "-", I'd argue the cut-and-paste utility of
sticking with that outweighs the consistency of ":". Unless perhaps we
should support both, which would be trivial.

Peter
Received on 2010-10-03 03:35:24 CEST