[PATCH 3 of 4 V2] bookmarks: rename readcurrent to readactive
Martin von Zweigbergk
martinvonz at google.com
Tue May 5 23:30:43 CDT 2015
On Tue, May 5, 2015 at 10:40 AM Ryan McElroy <rmcelroy at fb.com> wrote:
> # HG changeset patch
> # User Ryan McElroy <rmcelroy at fb.com>
> # Date 1428991393 25200
> # Mon Apr 13 23:03:13 2015 -0700
> # Node ID 3fae1a985ae9ef452aedb0e6c93ad02297248b4c
> # Parent 736ffdcc73cb164b24ac97e58209f6f9995fd676
> bookmarks: rename readcurrent to readactive
>
> Today, the terms 'active' and 'current' are interchangeably used
> throughout the
> codebase in reference to the active bookmark (the bookmark that will be
> updated
> with the next commit). This leads to confusion among developers and users.
> This patch is part of a series to standardize the usage to 'active'
> throughout
> the mercurial codebase and user interface.
>
> diff --git a/hgext/rebase.py b/hgext/rebase.py
> --- a/hgext/rebase.py
> +++ b/hgext/rebase.py
> @@ -888,7 +888,7 @@ def abort(repo, originalwd, target, stat
> repair.strip(repo.ui, repo, strippoints)
>
> if activebookmark:
> - bookmarks.setcurrent(repo, activebookmark)
> + bookmarks.activate(repo, activebookmark)
>
> clearstatus(repo)
> repo.ui.warn(_('rebase aborted\n'))
> diff --git a/mercurial/bookmarks.py b/mercurial/bookmarks.py
> --- a/mercurial/bookmarks.py
> +++ b/mercurial/bookmarks.py
> @@ -107,12 +107,17 @@ class bmstore(dict):
> fp.write("%s %s\n" % (hex(node), encoding.fromlocal(name)))
>
> def readcurrent(repo):
> - '''Get the current bookmark
> + warnings.warn('deprecated function bookmarks.readcurrent() called. ' +
> + 'update extension to call bookmarks.readactive()
> instead.',
> + category=DeprecationWarning, stacklevel=2)
> + return readactive(repo)
>
Wasn't this supposed to just be deleted or did I misunderstand Augie's
comment?
>
> - If we use gittish branches we have a current bookmark that
> - we are on. This function returns the name of the bookmark. It
> - is stored in .hg/bookmarks.current
> - '''
> +def readactive(repo):
> + """
> + Get the active bookmark. We can have an active bookmark that updates
> + itself as we commit. This function returns the name of that bookmark.
> + It is stored in .hg/bookmarks.current
> + """
> mark = None
> try:
> file = repo.vfs('bookmarks.current')
> diff --git a/mercurial/localrepo.py b/mercurial/localrepo.py
> --- a/mercurial/localrepo.py
> +++ b/mercurial/localrepo.py
> @@ -419,7 +419,7 @@ class localrepository(object):
>
> @repofilecache('bookmarks.current')
> def _bookmarkcurrent(self):
> - return bookmarks.readcurrent(self)
> + return bookmarks.readactive(self)
>
> def bookmarkheads(self, bookmark):
> name = bookmark.split('@', 1)[0]
> _______________________________________________
> Mercurial-devel mailing list
> Mercurial-devel at selenic.com
> http://selenic.com/mailman/listinfo/mercurial-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://selenic.com/pipermail/mercurial-devel/attachments/20150506/e8ed11fc/attachment.html>
More information about the Mercurial-devel
mailing list