| indygreg |
| hg-reviewers |
The RevlogError and WdirUnsupported could be raised by
_partialmatch(), but not by the rest of isvalid(), so let's move the
rest out to make it clearer.
| Lint Skipped |
| Unit Tests Skipped |
| Path | Packages | |||
|---|---|---|---|---|
| M | mercurial/revlog.py (21 lines) |
| Commit | Parents | Author | Summary | Date |
|---|---|---|---|---|
| Martin von Zweigbergk | May 3 2018, 1:49 AM |
| Status | Author | Revision | |
|---|---|---|---|
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz | ||
| Closed | martinvonz |
| raise LookupError(id, self.indexfile, _('no match found')) | raise LookupError(id, self.indexfile, _('no match found')) | ||||
| def shortest(self, node, minlength=1): | def shortest(self, node, minlength=1): | ||||
| """Find the shortest unambiguous prefix that matches node.""" | """Find the shortest unambiguous prefix that matches node.""" | ||||
| def isvalid(test): | def isvalid(test): | ||||
| try: | try: | ||||
| if self._partialmatch(test) is None: | if self._partialmatch(test) is None: | ||||
| return False | return False | ||||
| except error.RevlogError: | |||||
| return False | |||||
| except error.WdirUnsupported: | |||||
| # single 'ff...' match | |||||
| return True | |||||
| try: | try: | ||||
| i = int(test) | i = int(test) | ||||
| # if we are a pure int, then starting with zero will not be | # if we are a pure int, then starting with zero will not be | ||||
| # confused as a rev; or, obviously, if the int is larger | # confused as a rev; or, obviously, if the int is larger | ||||
| # than the value of the tip rev | # than the value of the tip rev | ||||
| if test[0] == '0' or i > len(self): | if test[0] == '0' or i > len(self): | ||||
| return True | return True | ||||
| return False | return False | ||||
| except ValueError: | except ValueError: | ||||
| return True | return True | ||||
| except error.RevlogError: | |||||
| return False | |||||
| except error.WdirUnsupported: | |||||
| # single 'ff...' match | |||||
| return True | |||||
| hexnode = hex(node) | hexnode = hex(node) | ||||
| shortest = hexnode | shortest = hexnode | ||||
| startlength = max(6, minlength) | startlength = max(6, minlength) | ||||
| length = startlength | length = startlength | ||||
| while True: | while True: | ||||
| test = hexnode[:length] | test = hexnode[:length] | ||||
| if isvalid(test): | if isvalid(test): | ||||