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.