Cleaned up and docstringed libbe.cmdutil.unique_name().

Now the first bug will have a 3 char short name (used to be one char,
with the second bug having a 3 char name).

diff --git a/libbe/cmdutil.py b/libbe/cmdutil.py
index 079601e04ebaa42bbf09144dc86e9258fd6ce050..cc2b60f3fb4b727f4dc0b810e23d9b66433dec42 100644 (file)
--- a/libbe/cmdutil.py
+++ b/libbe/cmdutil.py
@@ -24,14 +24,17 @@ from StringIO import StringIO
 import utility
 
 def unique_name(bug, bugs):
-    chars = 1
+    """
+    Generate short names from uuids.  Picks the minimum number of
+    characters (>=3) from the beginning of the uuid such that the
+    short names are unique.
+    """
+    chars = 3
     for some_bug in bugs:
         if bug.uuid == some_bug.uuid:
             continue
         while (bug.uuid[:chars] == some_bug.uuid[:chars]):
             chars+=1
-        if chars < 3:
-            chars = 3
     return bug.uuid[:chars]
 
 class UserError(Exception):