Rev 4625: (Matthew Fuller) Tweak some code in RevisionSpec in file:///home/pqm/archives/thelove/bzr/%2Btrunk/

[Canonical.com Patch Queue Manager]

At file:///home/pqm/archives/thelove/bzr/%2Btrunk/

------------------------------------------------------------
revno: 4625 [merge]
revision-id: pqm at pqm.ubuntu.com-20090818151725-1frh9n7l1p8c75pb
parent: pqm at pqm.ubuntu.com-20090818082317-8g5738hvi41aqhsh
parent: fullermd at over-yonder.net-20090725082642-gd7rhcn3kgjvv9dx
committer: Canonical.com Patch Queue Manager <pqm at pqm.ubuntu.com>
branch nick: +trunk
timestamp: Tue 2009-08-18 16:17:25 +0100
message:
  (Matthew Fuller) Tweak some code in RevisionSpec
modified:
  bzrlib/revisionspec.py         revisionspec.py-20050907152633-17567659fd5c0ddb
=== modified file 'bzrlib/revisionspec.py'
--- a/bzrlib/revisionspec.py	2009-03-23 14:59:43 +0000
+++ b/bzrlib/revisionspec.py	2009-07-25 08:26:42 +0000
@@ -161,9 +161,9 @@
             return spectype(spec, _internal=True)
         else:
             for spectype in SPEC_TYPES:
-                trace.mutter('Returning RevisionSpec %s for %s',
-                             spectype.__name__, spec)
                 if spec.startswith(spectype.prefix):
+                    trace.mutter('Returning RevisionSpec %s for %s',
+                                 spectype.__name__, spec)
                     return spectype(spec, _internal=True)
             # RevisionSpec_revno is special cased, because it is the only
             # one that directly handles plain integers
@@ -185,7 +185,6 @@
             called directly. Only from RevisionSpec.from_string()
         """
         if not _internal:
-            # XXX: Update this after 0.10 is released
             symbol_versioning.warn('Creating a RevisionSpec directly has'
                                    ' been deprecated in version 0.11. Use'
                                    ' RevisionSpec.from_string()'