mq: don't warn about ambiguous patch name when using patch index (issue1439)

hgext/mq.py

@@ -838,14 +838,11 b' class queue:'
                 if s == 'qbase':
                     return self.series[0]
             return None
+
         if patch == None:
             return None
-
-        # we don't want to return a partial match until we make
-        # sure the file name passed in does not exist (checked below)
-        res = partial_name(patch)
-        if res and res == patch:
-            return res
+        if patch in self.series:
+            return patch
 
         if not os.path.isfile(self.join(patch)):
             try:
@@ -853,10 +850,11 b' class queue:'
             except(ValueError, OverflowError):
                 pass
             else:
-                if sno < len(self.series):
+                if -len(self.series) <= sno < len(self.series):
                     return self.series[sno]
+
             if not strict:
-                # return any partial match made above
+                res = partial_name(patch)
                 if res:
                     return res
                 minus = patch.rfind('-')

tests/test-mq-qgoto

@@ -25,3 +25,25 b' hg qrefresh'
 hg qgoto a.patch
 hg qgoto c.patch
 hg qgoto b.patch
+
+echo
+echo % Using index
+hg qgoto 0
+hg qgoto 2
+
+echo
+echo % No warnings when using index
+hg qnew bug314159
+echo d >> c
+hg qrefresh
+hg qnew bug141421
+echo e >> c
+hg qrefresh
+hg qgoto 1
+hg qgoto 3
+
+echo
+echo % Detect ambiguous non-index
+hg qgoto 14
+
+exit 0

tests/test-mq-qgoto.out

@@ -4,3 +4,21 b' applying b.patch'
 applying c.patch
 Now at: c.patch
 Now at: b.patch
+
+% Using index
+Now at: a.patch
+applying b.patch
+applying c.patch
+Now at: c.patch
+
+% No warnings when using index
+Now at: b.patch
+applying c.patch
+applying bug314159
+Now at: bug314159
+
+% Detect ambiguous non-index
+patch name "14" is ambiguous:
+  bug314159
+  bug141421
+abort: patch 14 not in series