# HG changeset patch # User Augie Fackler # Date 1462410109 14400 # Node ID 052c9318e46415c6e479f10fd689dd11ea8605a2 # Parent af6d4a49e3615ac6cfa63b9c10811c28520434ea bookmarks: jettison bmstore's write() method per deprecation policy diff -r af6d4a49e361 -r 052c9318e464 mercurial/bookmarks.py --- a/mercurial/bookmarks.py Thu May 05 15:12:43 2016 -0500 +++ b/mercurial/bookmarks.py Wed May 04 21:01:49 2016 -0400 @@ -109,39 +109,6 @@ location='plain') tr.hookargs['bookmark_moved'] = '1' - def write(self): - '''Write bookmarks - - Write the given bookmark => hash dictionary to the .hg/bookmarks file - in a format equal to those of localtags. - - We also store a backup of the previous state in undo.bookmarks that - can be copied back on rollback. - ''' - msg = 'bm.write() is deprecated, use bm.recordchange(transaction)' - self._repo.ui.deprecwarn(msg, '3.7') - # TODO: writing the active bookmark should probably also use a - # transaction. - self._writeactive() - if self._clean: - return - repo = self._repo - if (repo.ui.configbool('devel', 'all-warnings') - or repo.ui.configbool('devel', 'check-locks')): - l = repo._wlockref and repo._wlockref() - if l is None or not l.held: - repo.ui.develwarn('bookmarks write with no wlock') - - tr = repo.currenttransaction() - if tr: - self.recordchange(tr) - # invalidatevolatilesets() is omitted because this doesn't - # write changes out actually - return - - self._writerepo(repo) - repo.invalidatevolatilesets() - def _writerepo(self, repo): """Factored out for extensibility""" rbm = repo._bookmarks