From patchwork Tue Jun 27 08:56:49 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: [6,of,8] obsolete: delete three unused utility functions From: Pierre-Yves David X-Patchwork-Id: 21765 Message-Id: <2d143a083ded57fdf782.1498553809@nodosa.octopoid.net> To: mercurial-devel@mercurial-scm.org Date: Tue, 27 Jun 2017 10:56:49 +0200 # HG changeset patch # User Pierre-Yves David # Date 1498520921 -7200 # Tue Jun 27 01:48:41 2017 +0200 # Node ID 2d143a083ded57fdf7825bb8a263cf792e409e0c # Parent d04b168d55057262ffd07c71f9d822924d65ba4f # 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 2d143a083ded 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'