Raif S. Naffah wrote: > hello all, > > the attached patch --already committed-- adds Integer related utility methods > to: > > a. cache Integer.valueOf(String) results in a LRU of 100 places organised by > access order, > > b. simulate the version 5 Integer.valueOf(int) method, so client code can run > on a version 1.4 of the Java class library. > > a new Mauve test TestOfIntegerUtil (in gnu.testlet.gnu.java.security.util) > has > been added. > > 2006-07-22 Raif S. Naffah <[EMAIL PROTECTED]> > > * gnu/java/security/util/IntegerUtil.java: New file. > >
You didn't update all the classes that you changed in your patch that replaced 'new Integer' with 'Integer.valueOf'. In fact, I see some classes were changed (gnu.java.security.util.Sequence) but aren't mentioned in the changelog [1]. In fact, did you change any of the classes you originally modified to use this? 1. http://cvs.savannah.gnu.org/viewcvs/classpath/gnu/java/security/util/Sequence.java?rev=1.2&root=classpath&view=markup
