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: java-dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: java-dev-h...@lucene.apache.org

Reply via email to