bookmarks: jettison bmstore's write() method per deprecation policy
authorAugie Fackler <augie@google.com>
Wed, 04 May 2016 21:01:49 -0400
changeset 29084 052c9318e464
parent 29083 af6d4a49e361
child 29085 df838803c1d4
bookmarks: jettison bmstore's write() method per deprecation policy
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