This is an archive of the discontinued Mercurial Phabricator instance.

errors: raise InputError on bad revset to revrange() iff provided by the user

Authored by martinvonz on Oct 1 2021, 3:51 PM.



Most callers of scmutil.revrange() pass in a revset provided by the
user. If there are problems resolving that, it should result in an
InputError and exit code 10 (when using detailed exit
codes). However, there are also some callers that pass in revsets not
provided by the user. InputError is not appropriate in those
cases. This patch therefore introduces a wrapper around
scmutil.revrange() that simply converts the exception type. I put it
in since that seems to be the lowest-level module in
the (poorly defined) UI layer.

Diff Detail

rHG Mercurial
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.