This is an archive of the discontinued Mercurial Phabricator instance.

Differential D4806

context: stop catching and re-raising FilteredRepoLookupError
ClosedPublic

Authored by martinvonz on Sep 29 2018, 2:14 AM.

Details

Reviewers
None
Group Reviewers
hg-reviewers
Commits
rHG2327abace563: context: stop catching and re-raising FilteredRepoLookupError
rHG8daa7d3fd38d: context: stop catching and re-raising FilteredRepoLookupError
Summary

FilteredRepoLookupError is only raised by changectx's constructor and
the higher-level scmutil.revsymbol(), so there's no need to catch it
in changectx's constructor.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Revision Contents
Changeset List

PathPackages
Mmercurial/context.py (2 lines)

Diff 11489

mercurial/context.py

else: else:
raise error.ProgrammingError( raise error.ProgrammingError(
"unsupported changeid '%s' of type %s" % "unsupported changeid '%s' of type %s" %
(changeid, type(changeid))) (changeid, type(changeid)))
except (error.FilteredIndexError, error.FilteredLookupError): except (error.FilteredIndexError, error.FilteredLookupError):
raise error.FilteredRepoLookupError(_("filtered revision '%s'") raise error.FilteredRepoLookupError(_("filtered revision '%s'")
% pycompat.bytestr(changeid)) % pycompat.bytestr(changeid))
except error.FilteredRepoLookupError:
raise
except IndexError: except IndexError:
pass pass
raise error.RepoLookupError( raise error.RepoLookupError(
_("unknown revision '%s'") % changeid) _("unknown revision '%s'") % changeid)
def __hash__(self): def __hash__(self):
try: try:
return hash(self._rev) return hash(self._rev)