<div dir="ltr"><br><br><div class="gmail_quote">On Thu, May 7, 2015 at 2:00 PM Ryan McElroy <<a href="mailto:rmcelroy@fb.com">rmcelroy@fb.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Ryan McElroy <<a href="mailto:rmcelroy@fb.com" target="_blank">rmcelroy@fb.com</a>><br>
# Date 1429043510 25200<br>
#      Tue Apr 14 13:31:50 2015 -0700<br>
# Node ID 258d4654240ae6e6978acaeab753ed65c6843789<br>
# Parent  ccb7b2b9a2f0dc5ac57e2e13e48180a03ad6d175<br>
bookmarks: remove unused updatecurrentbookmark function (API)<br>
<br>
This function was not used anywhere in core and there is no indication that<br>
it is used elsewhere either.<br></blockquote><div><br></div><div>I've pushed this to the clowncopter, thanks.</div><div><br></div><div>The last caller seems to have been removed in f72b71ba756b (unbundle: don't advance bookmarks (issue4322) (BC), 2014-08-10).</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
diff --git a/mercurial/bookmarks.py b/mercurial/bookmarks.py<br>
--- a/mercurial/bookmarks.py<br>
+++ b/mercurial/bookmarks.py<br>
@@ -8,7 +8,7 @@<br>
 import os<br>
 from mercurial.i18n import _<br>
 from mercurial.node import hex, bin<br>
-from mercurial import encoding, error, util, obsolete, lock as lockmod<br>
+from mercurial import encoding, util, obsolete, lock as lockmod<br>
 import errno<br>
<br>
 class bmstore(dict):<br>
@@ -175,15 +175,6 @@ def isactivecurrent(repo):<br>
     parents = [p.node() for p in repo[None].parents()]<br>
     return (mark in marks and marks[mark] in parents)<br>
<br>
-def updatecurrentbookmark(repo, oldnode, curbranch):<br>
-    try:<br>
-        return update(repo, oldnode, repo.branchtip(curbranch))<br>
-    except error.RepoLookupError:<br>
-        if curbranch == "default": # no default branch!<br>
-            return update(repo, oldnode, repo.lookup("tip"))<br>
-        else:<br>
-            raise util.Abort(_("branch %s not found") % curbranch)<br>
-<br>
 def deletedivergent(repo, deletefrom, bm):<br>
     '''Delete divergent versions of bm on nodes in deletefrom.<br>
<br>
_______________________________________________<br>
Mercurial-devel mailing list<br>
<a href="mailto:Mercurial-devel@selenic.com" target="_blank">Mercurial-devel@selenic.com</a><br>
<a href="http://selenic.com/mailman/listinfo/mercurial-devel" target="_blank">http://selenic.com/mailman/listinfo/mercurial-devel</a><br>
</blockquote></div></div>