Author: bobtarling Date: 2008-04-29 16:56:03-0700 New Revision: 14524 Modified: trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java
Log: Highlight design flaw Modified: trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java Url: http://argouml.tigris.org/source/browse/argouml/trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java?view=diff&rev=14524&p1=trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java&p2=trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java&r1=14523&r2=14524 ============================================================================== --- trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java (original) +++ trunk/src/argouml-app/src/org/argouml/cognitive/ItemUID.java 2008-04-29 16:56:03-0700 @@ -68,6 +68,9 @@ * could be used to remove the dependancy to the model component here, which * I find a bit unaesthetic. So far, not enough to write it (though it is not * much work). + * + * TODO: This class seems to expect arguments to be guaranteed to implement + * some specific method. So why doesn't it expect an interface to ensure that? * * @author Michael Stockman */ @@ -121,6 +124,7 @@ * Generates a new unique ID and returns it as a String. The contents * of the String is supposed to be unique with respect to all Strings * generated by other instances of this class. + * TODO: We should consider using java.util.UUID for this instead. * * @return A String with unique content. */ --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
