Hi Eks, Nice catch. I committed the fix.
On Sun, Aug 24, 2008 at 1:44 AM, eks dev <[EMAIL PROTECTED]> wrote: > Hi Solr, > > I have been reading the code of this class, and stumbled upon it... > > public Object clone() { > CharArrayMap<V> map = null; > try { > map = (CharArrayMap<V>)super.clone(); > map.keys = keys.clone(); > map.values = keys.clone(); //You wanted to say values.clone() > > > and by the way, would it not be better to declare values as V[] and reduce > the number of casts? > > also equals() method performance, I read somewhere loops backwards in while > are slightly faster for this? > > > Send instant messages to your online friends http://uk.messenger.yahoo.com > -- Regards, Shalin Shekhar Mangar.