bisect: extract the 'reset' logic into its own function
This is part of small clean up movement. The bisect module seem more appropriate
to host the bisect logic. The cleanup itself is motivated by some higher level
cleanup around vfs and locking.
--- a/mercurial/commands.py Wed Aug 24 04:31:49 2016 +0200
+++ b/mercurial/commands.py Wed Aug 24 04:13:53 2016 +0200
@@ -902,8 +902,7 @@
cmdutil.checkunfinished(repo)
if reset:
- if repo.vfs.exists("bisect.state"):
- repo.vfs.unlink("bisect.state")
+ hbisect.resetstate(repo)
return
state = hbisect.load_state(repo)
--- a/mercurial/hbisect.py Wed Aug 24 04:31:49 2016 +0200
+++ b/mercurial/hbisect.py Wed Aug 24 04:13:53 2016 +0200
@@ -159,6 +159,11 @@
f.write("%s %s\n" % (kind, hex(node)))
f.close()
+def resetstate(repo):
+ """remove any bisect state from the repository"""
+ if repo.vfs.exists("bisect.state"):
+ repo.vfs.unlink("bisect.state")
+
def get(repo, status):
"""
Return a list of revision(s) that match the given status: