comparison mercurial/phases.py @ 25611:d89045a66e01

phase: rename getphaserevs to loadphaserevs This function is: - already loading the data in place, - used once in the code. So we drop the return value and change the name to make this obvious. We keep the function public because we'll have to use it in revset.
author Pierre-Yves David <pierre-yves.david@fb.com>
date Mon, 15 Jun 2015 16:04:14 -0700
parents 186321309d05
children a13c18c14ade
comparison
equal deleted inserted replaced
25610:37876ca00c0a 25611:d89045a66e01
192 for rev in roots: 192 for rev in roots:
193 revs[rev] = phase 193 revs[rev] = phase
194 for rev in repo.changelog.descendants(roots): 194 for rev in repo.changelog.descendants(roots):
195 revs[rev] = phase 195 revs[rev] = phase
196 196
197 def getphaserevs(self, repo): 197 def loadphaserevs(self, repo):
198 """ensure phase information is loaded in the object"""
198 if self._phaserevs is None: 199 if self._phaserevs is None:
199 try: 200 try:
200 if repo.ui.configbool('experimental', 201 if repo.ui.configbool('experimental',
201 'nativephaseskillswitch'): 202 'nativephaseskillswitch'):
202 self._computephaserevspure(repo) 203 self._computephaserevspure(repo)
203 else: 204 else:
204 res = self._getphaserevsnative(repo) 205 res = self._getphaserevsnative(repo)
205 self._phaserevs, self._phasesets = res 206 self._phaserevs, self._phasesets = res
206 except AttributeError: 207 except AttributeError:
207 self._computephaserevspure(repo) 208 self._computephaserevspure(repo)
208 return self._phaserevs
209 209
210 def invalidate(self): 210 def invalidate(self):
211 self._phaserevs = None 211 self._phaserevs = None
212 self._phasesets = None 212 self._phasesets = None
213 213
231 return public 231 return public
232 if rev < nullrev: 232 if rev < nullrev:
233 raise ValueError(_('cannot lookup negative revision')) 233 raise ValueError(_('cannot lookup negative revision'))
234 if self._phaserevs is None or rev >= len(self._phaserevs): 234 if self._phaserevs is None or rev >= len(self._phaserevs):
235 self.invalidate() 235 self.invalidate()
236 self._phaserevs = self.getphaserevs(repo) 236 self.loadphaserevs(repo)
237 return self._phaserevs[rev] 237 return self._phaserevs[rev]
238 238
239 def write(self): 239 def write(self):
240 if not self.dirty: 240 if not self.dirty:
241 return 241 return