RE: [JBoss-dev] CVS update: jboss/src/main/org/jboss/ejb CacheKey.java

[marc fleury]

|  +        // Make a copy just in case somebody re-uses the instance
|  +        this.id = mo.get();


Bill that comment you just commited is absolutely "obscure".

Reuses the instance... what instance.

Please say
// Make a copy of the id to enforce copy semantics and allow reuse of the
original primary key

or something like that

marc



_______________________________________________
Jboss-development mailing list
[EMAIL PROTECTED]
http://lists.sourceforge.net/lists/listinfo/jboss-development