This is an archive of the discontinued Mercurial Phabricator instance.

Differential D12408

bookmarks: use new function for getting first line of string
ClosedPublic

Authored by martinvonz on Mar 25 2022, 1:38 AM.

Details

Reviewers
Alphare
Group Reviewers
hg-reviewers
Commits
rHGf808417db5cc: bookmarks: use new function for getting first line of string

Diff Detail

Repository
rHG Mercurial
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Revision Contents
Changeset List

PathPackages
Mmercurial/bookmarks.py (3 lines)

Diff 32808

mercurial/bookmarks.py

obsutil, obsutil,
pycompat, pycompat,
requirements, requirements,
scmutil, scmutil,
txnutil, txnutil,
util, util,
) )
from .utils import ( from .utils import (
stringutil,
urlutil, urlutil,
) )
# label constants # label constants
# until 3.5, bookmarks.current was the advertised name, not # until 3.5, bookmarks.current was the advertised name, not
# bookmarks.active, so we must use both to avoid breaking old # bookmarks.active, so we must use both to avoid breaking old
# custom styles # custom styles
activebookmarklabel = b'bookmarks.active bookmarks.current' activebookmarklabel = b'bookmarks.active bookmarks.current'
""" """
Get the active bookmark. We can have an active bookmark that updates Get the active bookmark. We can have an active bookmark that updates
itself as we commit. This function returns the name of that bookmark. itself as we commit. This function returns the name of that bookmark.
It is stored in .hg/bookmarks.current It is stored in .hg/bookmarks.current
""" """
# No readline() in osutil.posixfile, reading everything is # No readline() in osutil.posixfile, reading everything is
# cheap. # cheap.
content = repo.vfs.tryread(b'bookmarks.current') content = repo.vfs.tryread(b'bookmarks.current')
mark = encoding.tolocal((content.splitlines() or [b''])[0]) mark = encoding.tolocal(stringutil.firstline(content))
if mark == b'' or mark not in marks: if mark == b'' or mark not in marks:
mark = None mark = None
return mark return mark
def activate(repo, mark): def activate(repo, mark):
""" """
Set the given bookmark to be 'active', meaning that this bookmark will Set the given bookmark to be 'active', meaning that this bookmark will