# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@logilab.fr>
# Date 1351092400 -7200
# Node ID ed225834b3720f249c74757963a96b1be98ccce0
# Parent  fb14a5dcdc62987512820531fe60719d650491b6
bookmark: simplify mutability check in `validdest`

There is no need to check phase by hand. We already have a ctx method for that.

diff -r fb14a5dcdc62 -r ed225834b372 mercurial/bookmarks.py
--- a/mercurial/bookmarks.py	Wed Nov 07 09:59:46 2012 -0800
+++ b/mercurial/bookmarks.py	Wed Oct 24 17:26:40 2012 +0200
@@ -7,7 +7,7 @@
 
 from mercurial.i18n import _
 from mercurial.node import hex
-from mercurial import encoding, error, util, obsolete, phases
+from mercurial import encoding, error, util, obsolete
 import errno, os
 
 def read(repo):
@@ -264,7 +264,7 @@
             plen = len(validdests)
             succs = set(c.node() for c in validdests)
             for c in validdests:
-                if c.phase() > phases.public:
+                if c.mutable():
                     # obsolescence marker does not apply to public changeset
                     succs.update(obsolete.allsuccessors(repo.obsstore,
                                                         [c.node()]))