##// END OF EJS Templates
shortest: rename "test" variable to "prefix"...
Martin von Zweigbergk -
r37880:6921d3ec default
parent child Browse files
Show More
@@ -1502,9 +1502,9 b' class revlog(object):'
1502 1502
1503 1503 def shortest(self, node, minlength=1):
1504 1504 """Find the shortest unambiguous prefix that matches node."""
1505 def isvalid(test):
1505 def isvalid(prefix):
1506 1506 try:
1507 if self._partialmatch(test) is None:
1507 if self._partialmatch(prefix) is None:
1508 1508 return False
1509 1509 except error.RevlogError:
1510 1510 return False
@@ -1512,11 +1512,11 b' class revlog(object):'
1512 1512 # single 'ff...' match
1513 1513 return True
1514 1514 try:
1515 i = int(test)
1515 i = int(prefix)
1516 1516 # if we are a pure int, then starting with zero will not be
1517 1517 # confused as a rev; or, obviously, if the int is larger
1518 1518 # than the value of the tip rev
1519 if test[0] == '0' or i > len(self):
1519 if prefix[0] == '0' or i > len(self):
1520 1520 return True
1521 1521 return False
1522 1522 except ValueError:
@@ -1527,9 +1527,9 b' class revlog(object):'
1527 1527 startlength = max(6, minlength)
1528 1528 length = startlength
1529 1529 while True:
1530 test = hexnode[:length]
1531 if isvalid(test):
1532 shortest = test
1530 prefix = hexnode[:length]
1531 if isvalid(prefix):
1532 shortest = prefix
1533 1533 if length == minlength or length > startlength:
1534 1534 return shortest
1535 1535 length -= 1
General Comments 0
You need to be logged in to leave comments. Login now