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.

Reply via email to