This is an archive of the discontinued Mercurial Phabricator instance.

Differential D3160

convert: remove unused/unnecessary variable "parentctx"
ClosedPublic

Authored by martinvonz on Apr 6 2018, 1:46 PM.

Details

Reviewers
indygreg
Group Reviewers
hg-reviewers
Commits
rHG65705fbc9c37: convert: remove unused/unnecessary variable "parentctx"

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Apr 6 2018, 1:46 PM
indygreg accepted this revision.Apr 6 2018, 2:48 PM
This revision is now accepted and ready to land.Apr 6 2018, 2:48 PM
Closed by commit rHG65705fbc9c37: convert: remove unused/unnecessary variable "parentctx" (authored by martinvonz). · Explain WhyApr 6 2018, 2:50 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mhgext/convert/hg.py (4 lines)

Diff 7787

hgext/convert/hg.py

if not closed and not man.cmp(m1node, man.revision(mnode)): if not closed and not man.cmp(m1node, man.revision(mnode)):
self.ui.status(_("filtering out empty revision\n")) self.ui.status(_("filtering out empty revision\n"))
self.repo.rollback(force=True) self.repo.rollback(force=True)
return parent return parent
return p2 return p2
def puttags(self, tags): def puttags(self, tags):
try: try:
parentctx = self.repo[self.tagsbranch] tagparent = self.repo[self.tagsbranch].node()
tagparent = parentctx.node()
except error.RepoError: except error.RepoError:
parentctx = None
tagparent = nodemod.nullid tagparent = nodemod.nullid
oldlines = set() oldlines = set()
for branch, heads in self.repo.branchmap().iteritems(): for branch, heads in self.repo.branchmap().iteritems():
for h in heads: for h in heads:
if '.hgtags' in self.repo[h]: if '.hgtags' in self.repo[h]:
oldlines.update( oldlines.update(
set(self.repo[h]['.hgtags'].data().splitlines(True))) set(self.repo[h]['.hgtags'].data().splitlines(True)))