From patchwork Wed Jun 28 21:21:57 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [6,of,8,V2] obsolete: delete three unused utility functions From: Pierre-Yves David X-Patchwork-Id: 21819 Message-Id: <3c886efd86fd9ddfba0e.1498684917@nodosa.octopoid.net> To: mercurial-devel@mercurial-scm.org Date: Wed, 28 Jun 2017 23:21:57 +0200 # HG changeset patch # User Pierre-Yves David # Date 1498520921 -7200 # Tue Jun 27 01:48:41 2017 +0200 # Node ID 3c886efd86fd9ddfba0e3cbc3cdff128c5712f8d # Parent 754eb75d32e7fa744d9319b8e0f01005fe9fb6f3 # EXP-Topic obsutil # Available At https://www.mercurial-scm.org/repo/users/marmoute/mercurial/ # hg pull https://www.mercurial-scm.org/repo/users/marmoute/mercurial/ -r 3c886efd86fd obsolete: delete three unused utility functions None of this function has been used in the past 5 years, so I think it is safe to just kill them. All code accessing rich markers is using 'getmarkers(...)' instead (or raw markers). diff --git a/mercurial/obsolete.py b/mercurial/obsolete.py --- a/mercurial/obsolete.py +++ b/mercurial/obsolete.py @@ -853,22 +853,6 @@ def getmarkers(repo, nodes=None, exclusi for markerdata in rawmarkers: yield marker(repo, markerdata) -def relevantmarkers(repo, node): - """all obsolete markers relevant to some revision""" - for markerdata in repo.obsstore.relevantmarkers(node): - yield marker(repo, markerdata) - - -def precursormarkers(ctx): - """obsolete marker marking this changeset as a successors""" - for data in ctx.repo().obsstore.precursors.get(ctx.node(), ()): - yield marker(ctx.repo(), data) - -def successormarkers(ctx): - """obsolete marker making this changeset obsolete""" - for data in ctx.repo().obsstore.successors.get(ctx.node(), ()): - yield marker(ctx.repo(), data) - # keep compatibility for the 4.3 cycle def allprecursors(obsstore, nodes, ignoreflags=0): movemsg = 'obsolete.allprecursors moved to obsutil.allprecursors'