obsolete: rename _addprecursors into _addpredecessors

Use util.nouideprecwarn because _addpredecessors doesn't have easy access to
an ui object.

The renaming is done according to
https://www.mercurial-scm.org/wiki/CEDVocabulary.

Differential Revision: https://phab.mercurial-scm.org/D244
This commit is contained in:
Boris Feld 2017-08-02 19:34:15 +02:00
parent 6a8cca1d6e
commit 284208b1ed

View File

@ -470,11 +470,18 @@ def _addsuccessors(successors, markers):
for mark in markers:
successors.setdefault(mark[0], set()).add(mark)
def _addprecursors(*args, **kwargs):
msg = ("'obsolete._addprecursors' is deprecated, "
"use 'obsolete._addpredecessors'")
util.nouideprecwarn(msg, '4.4')
return _addpredecessors(*args, **kwargs)
@util.nogc
def _addprecursors(precursors, markers):
def _addpredecessors(predecessors, markers):
for mark in markers:
for suc in mark[1]:
precursors.setdefault(suc, set()).add(mark)
predecessors.setdefault(suc, set()).add(mark)
@util.nogc
def _addchildren(children, markers):
@ -660,9 +667,9 @@ class obsstore(object):
@propertycache
def precursors(self):
precursors = {}
_addprecursors(precursors, self._all)
return precursors
predecessors = {}
_addpredecessors(predecessors, self._all)
return predecessors
@propertycache
def children(self):
@ -680,7 +687,7 @@ class obsstore(object):
if self._cached('successors'):
_addsuccessors(self.successors, markers)
if self._cached('precursors'):
_addprecursors(self.precursors, markers)
_addpredecessors(self.precursors, markers)
if self._cached('children'):
_addchildren(self.children, markers)
_checkinvalidmarkers(markers)