Not really, there is no reason fort hat in my opinion. And Token.copyTo() does not do this, but should then!
I think, remove it. Strings can always by kept, as they are final and unmodifiable. ----- Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de eMail: [email protected] > -----Original Message----- > From: Mark Miller [mailto:[email protected]] > Sent: Friday, August 21, 2009 12:39 AM > To: [email protected] > Subject: TypeAttributeImpl > > Getting into the nitpick stuff ;) > > In TypeAttributeImpl: > > public void copyTo(AttributeImpl target) { > TypeAttribute t = (TypeAttribute) target; > t.setType(new String(type)); > } > > The class uses type.equals and type.hashCode - is there any reason to > create a new String object here? Just use type ... > > -- > - Mark > > http://www.lucidimagination.com > > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
