Re: Réf. : Re: [collection] MultiHashMap class improvements

[Stephen Colebourne <sc...@btopenworld.com>]

Patch applied, although I did a bit more work. Thanks.

Can you recheck that my additional javadoc/tidying didn't break anything you
were expecting to work.

Stephen


----- Original Message -----
From: <ju...@achatpro.com>
To: "Jakarta Commons Developers List" <co...@jakarta.apache.org>
Sent: Monday, October 14, 2002 8:12 AM
Subject: Réf. : Re: [collection] MultiHashMap class improvements




Hi.(See attached file: MultiHashMap.java)

I have removed the two methods of interface and updated the clone() method.

Julien




                    "Stephen
                    Colebourne"                Pour :  "Jakarta Commons
Developers List"
                    <scolebourne@btopen
<co...@jakarta.apache.org>
                    world.com>                 cc :
                                               Objet :      Re: [collection]
MultiHashMap class improvements
                    14/10/02 00:44
                    Veuillez répondre à
                    "Jakarta Commons
                    Developers List"






I have taken a look at the patch. Unfortunately I can't use it at present
because it would involve changing the MultiMap interface. Once released, we
generally do not change interfaces.

Stephen

----- Original Message -----
From: <ju...@achatpro.com>
To: <co...@jakarta.apache.org>
Sent: Wednesday, October 09, 2002 1:43 PM
Subject: [collection] MultiHashMap class improvements


> Hi.
>
> I use MultiHashMap class.  I added a few improvements :
>
> 1) fix bug  9573 : http://issues.apache.org/bugzilla/show_bug.cgi?id=9573
> MultiHashMap.values()  is backed by map now .
>  (update method values() + add interns class Values & ValueIterator)
>
> 2) fix another bug : ArrayList can be put in MultiHashMap as value now
>  (update method put(Object,Object) and add method
> readObject(ObjectInputStream))
>
> 3)update method clone() : each Collection container is cloned now.
>
> 4)add two useful methods in MultiMap & MultiHashMap class:
>
> public boolean contains( Object key, Object item )
> public void putAll( Object key, Collection collection )
>
> 5) add a method protected Collection getNewInstanceContainer( int size )
> So it will be possible later to create a AbstractMultiMap?
>
> JUnit tests are OK,
>
> Any comments are welcome
>
> JB
>
>
>
>
>
> --
> To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
> For additional commands, e-mail:
<ma...@jakarta.apache.org>
>



--
To unsubscribe, e-mail:   <
mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <
mailto:commons-dev-help@jakarta.apache.org>






----------------------------------------------------------------------------
----


> --
> To unsubscribe, e-mail:
<ma...@jakarta.apache.org>
> For additional commands, e-mail:
<ma...@jakarta.apache.org>


--
To unsubscribe, e-mail:   <ma...@jakarta.apache.org>
For additional commands, e-mail: <ma...@jakarta.apache.org>