comparison mercurial/revlog.py @ 38149:bf59f95583c1

revlog: make variable name 'd' more explicit in _isgooddeltainfo d -> deltainfo
author Paul Morelle <paul.morelle@octobus.net>
date Wed, 07 Mar 2018 12:00:07 +0100
parents f79ba1d1b4b1
children 69ec6f98cfa6
comparison
equal deleted inserted replaced
38148:b17fa9041695 38149:bf59f95583c1
2091 except KeyError: 2091 except KeyError:
2092 raise RevlogError(_('unknown compression type %r') % t) 2092 raise RevlogError(_('unknown compression type %r') % t)
2093 2093
2094 return compressor.decompress(data) 2094 return compressor.decompress(data)
2095 2095
2096 def _isgooddeltainfo(self, d, textlen): 2096 def _isgooddeltainfo(self, deltainfo, textlen):
2097 """Returns True if the given delta is good. Good means that it is within 2097 """Returns True if the given delta is good. Good means that it is within
2098 the disk span, disk size, and chain length bounds that we know to be 2098 the disk span, disk size, and chain length bounds that we know to be
2099 performant.""" 2099 performant."""
2100 if d is None: 2100 if deltainfo is None:
2101 return False 2101 return False
2102 2102
2103 # - 'd.distance' is the distance from the base revision -- bounding it 2103 # - 'deltainfo.distance' is the distance from the base revision --
2104 # limits the amount of I/O we need to do. 2104 # bounding it limits the amount of I/O we need to do.
2105 # - 'd.compresseddeltalen' is the sum of the total size of deltas we 2105 # - 'deltainfo.compresseddeltalen' is the sum of the total size of
2106 # need to apply -- bounding it limits the amount of CPU we consume. 2106 # deltas we need to apply -- bounding it limits the amount of CPU
2107 # we consume.
2107 2108
2108 defaultmax = textlen * 4 2109 defaultmax = textlen * 4
2109 maxdist = self._maxdeltachainspan 2110 maxdist = self._maxdeltachainspan
2110 if not maxdist: 2111 if not maxdist:
2111 maxdist = d.distance # ensure the conditional pass 2112 maxdist = deltainfo.distance # ensure the conditional pass
2112 maxdist = max(maxdist, defaultmax) 2113 maxdist = max(maxdist, defaultmax)
2113 if (d.distance > maxdist or d.deltalen > textlen or 2114 if (deltainfo.distance > maxdist or deltainfo.deltalen > textlen or
2114 d.compresseddeltalen > textlen * 2 or 2115 deltainfo.compresseddeltalen > textlen * 2 or
2115 (self._maxchainlen and d.chainlen > self._maxchainlen)): 2116 (self._maxchainlen and deltainfo.chainlen > self._maxchainlen)):
2116 return False 2117 return False
2117 2118
2118 return True 2119 return True
2119 2120
2120 def _addrevision(self, node, rawtext, transaction, link, p1, p2, flags, 2121 def _addrevision(self, node, rawtext, transaction, link, p1, p2, flags,