Implement Comparable<Name>. The old compareTo() did funny things if a Name was

being compared with a non-Name but I'm almost certain that never happened and
was just Ray being kooky.


git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4821 542714f4-19e9-0310-aa3c-eee0fc999fb1

src/java/com/threerings/util/Name.java

@@ -32,7 +32,7 @@ import com.threerings.io.SimpleStreamableObject;
  * everywhere they are compared.
  */
 public class Name extends SimpleStreamableObject
-    implements Comparable
+    implements Comparable<Name>
 {
     /** A blank name for use in situations where it is needed.
      * <em>Note:</em> because names are used in distributed applications
@@ -40,6 +40,15 @@ public class Name extends SimpleStreamableObject
      * sufficient. You must use <code>BLANK.equals(targetName)</code>. */
     public static final Name BLANK = new Name("");
 
+    /**
+     * Returns true if this name is null or blank, false if it contains
+     * useful data. This works on derived classes as well.
+     */
+    public static boolean isBlank (Name name)
+    {
+        return (name == null || name.toString().equals(BLANK.toString()));
+    }
+
     /** Creates a blank instance for unserialization. */
     public Name ()
     {
@@ -95,13 +104,13 @@ public class Name extends SimpleStreamableObject
         return _name;
     }
 
-    // documentation inherited
+    @Override // from Object
     public int hashCode ()
     {
         return getNormal().hashCode();
     }
 
-    // documentation inherited
+    @Override // from Object
     public boolean equals (Object other)
     {
         if (other != null) {
@@ -116,23 +125,10 @@ public class Name extends SimpleStreamableObject
         return false;
     }
 
-    // documentation inherited
+    // from interface Comparable<Name>
-    public int compareTo (Object o)
+    public int compareTo (Name other)
     {
-        if (o instanceof Name) {
+        return getNormal().compareTo(other.getNormal());
-            return getNormal().compareTo(((Name)o).getNormal());
-        } else {
-            return getClass().getName().compareTo(o.getClass().getName());
-        }
-    }
-
-    /**
-     * Returns true if this name is null or blank, false if it contains
-     * useful data. This works on derived classes as well.
-     */
-    public static boolean isBlank (Name name)
-    {
-        return (name == null || name.toString().equals(BLANK.toString()));
     }
 
     /**