<div dir="ltr"><br><br><div class="gmail_quote">On Tue, May 5, 2015 at 10:40 AM 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 1428991393 25200<br>
#      Mon Apr 13 23:03:13 2015 -0700<br>
# Node ID 3fae1a985ae9ef452aedb0e6c93ad02297248b4c<br>
# Parent  736ffdcc73cb164b24ac97e58209f6f9995fd676<br>
bookmarks: rename readcurrent to readactive<br>
<br>
Today, the terms 'active' and 'current' are interchangeably used throughout the<br>
codebase in reference to the active bookmark (the bookmark that will be updated<br>
with the next commit). This leads to confusion among developers and users.<br>
This patch is part of a series to standardize the usage to 'active' throughout<br>
the mercurial codebase and user interface.<br>
<br>
diff --git a/hgext/rebase.py b/hgext/rebase.py<br>
--- a/hgext/rebase.py<br>
+++ b/hgext/rebase.py<br>
@@ -888,7 +888,7 @@ def abort(repo, originalwd, target, stat<br>
             repair.strip(repo.ui, repo, strippoints)<br>
<br>
     if activebookmark:<br>
-        bookmarks.setcurrent(repo, activebookmark)<br>
+        bookmarks.activate(repo, activebookmark)<br>
<br>
     clearstatus(repo)<br>
     repo.ui.warn(_('rebase aborted\n'))<br>
diff --git a/mercurial/bookmarks.py b/mercurial/bookmarks.py<br>
--- a/mercurial/bookmarks.py<br>
+++ b/mercurial/bookmarks.py<br>
@@ -107,12 +107,17 @@ class bmstore(dict):<br>
             fp.write("%s %s\n" % (hex(node), encoding.fromlocal(name)))<br>
<br>
 def readcurrent(repo):<br>
-    '''Get the current bookmark<br>
+    warnings.warn('deprecated function bookmarks.readcurrent() called. ' +<br>
+                  'update extension to call bookmarks.readactive() instead.',<br>
+                  category=DeprecationWarning, stacklevel=2)<br>
+    return readactive(repo)<br></blockquote><div><br></div><div>Wasn't this supposed to just be deleted or did I misunderstand Augie's comment?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
-    If we use gittish branches we have a current bookmark that<br>
-    we are on. This function returns the name of the bookmark. It<br>
-    is stored in .hg/bookmarks.current<br>
-    '''<br>
+def readactive(repo):<br>
+    """<br>
+    Get the active bookmark. We can have an active bookmark that updates<br>
+    itself as we commit. This function returns the name of that bookmark.<br>
+    It is stored in .hg/bookmarks.current<br>
+    """<br>
     mark = None<br>
     try:<br>
         file = repo.vfs('bookmarks.current')<br>
diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py<br>
--- a/mercurial/localrepo.py<br>
+++ b/mercurial/localrepo.py<br>
@@ -419,7 +419,7 @@ class localrepository(object):<br>
<br>
     @repofilecache('bookmarks.current')<br>
     def _bookmarkcurrent(self):<br>
-        return bookmarks.readcurrent(self)<br>
+        return bookmarks.readactive(self)<br>
<br>
     def bookmarkheads(self, bookmark):<br>
         name = bookmark.split('@', 1)[0]<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>