I just took a look at the code.

There were a couple of details that are still not clear to me (as
around the "sentinel" use) but it looks much better than what we have
and it is e very complete implementation of the Map interface.

However (playing Jon) there is a code style issue there.

Michael, you sure save on lines of code but is is not so easy to read
that way.


Have fun,
Paulo Gaspar

> -----Original Message-----
> From: James Strachan [mailto:[EMAIL PROTECTED]]
> Sent: Friday, February 01, 2002 2:01 PM
> To: Jakarta Commons Developers List
> Subject: Re: [collections][PATCH] SequencedHashMap violates Map
> contract, has poor performance
>
>
> +1
>
> Sounds good to me.
>
> James
> ----- Original Message -----
> From: "Remy Maucherat" <[EMAIL PROTECTED]>
> > > I've submitted this patch twice now (once back in mid-November, once a
> > > week ago), but it still has yet to be committed.
> >
> > If nobody has time to apply or look at Michael's patches, how
> about giving
> > him karma on jakarta-commons ?
> >
> > Remy
> >
> >
> > --
> > To unsubscribe, e-mail:
> <mailto:[EMAIL PROTECTED]>
> > For additional commands, e-mail:
> <mailto:[EMAIL PROTECTED]>
>
>
> _________________________________________________________
> Do You Yahoo!?
> Get your free @yahoo.com address at http://mail.yahoo.com
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to