[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

Reply via email to