Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Tudor Girba
Ok, I created a bug entry: http://code.google.com/p/pharo/issues/detail?id=7384 Cheers, Doru On Jan 31, 2013, at 10:18 PM, Stéphane Ducasse wrote: > I do not know > but indeed it would be good to have one good one instead of three half > > Stef > > On Jan 31, 2013, at 9:40 AM, Tudor Girba

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Stéphane Ducasse
I do not know but indeed it would be good to have one good one instead of three half Stef On Jan 31, 2013, at 9:40 AM, Tudor Girba wrote: > Does anyone know the difference between: > - OrderedIdentityDictionary from Spec and > - OrderPreservingDictionary from > http://www.squeaksource.com/Orde

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Stéphane Ducasse
It will come :) Stef > ... and we need Spec as a project under team Pharo, in sthub :) > > >> Stef wrote: >>> good idea. >>> Do you know the test coverage of the class? >>> Because it would be nice to run hapao on it and write some tests if there >>> are none. >> >> Actually there are no test

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Tudor Girba
Does anyone know the difference between: - OrderedIdentityDictionary from Spec and - OrderPreservingDictionary from http://www.squeaksource.com/OrderPreservingDict ? we use the latter in XMLSupport and I think we could probably consolidate. Doru On Thu, Jan 31, 2013 at 9:38 AM, Esteban Lorenza

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Stéphane Ducasse
Ok so it would be good to address the issue. We should get better and larger collection choice. Since you already add a look could you check it again. Stef On Jan 31, 2013, at 5:49 AM, Clément Bera wrote: > Actually this was in the bug tracker > > http://code.google.com/p/pharo/issues/detail?id

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Clément Bera
Actually this was in the bug tracker http://code.google.com/p/pharo/issues/detail?id=2759&can=1&q=orderedDictionary&colspec=ID%20Type%20Status%20Summary%20Milestone%20Difficulty error 2759 This OrderedIdentityDictionary is only used for user interface that is why it is on Spec-Core for now. See

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-31 Thread Esteban Lorenzano
... and we need Spec as a project under team Pharo, in sthub :) On Jan 31, 2013, at 8:28 AM, "Torsten Bergmann" wrote: > Stef wrote: >> good idea. >> Do you know the test coverage of the class? >> Because it would be nice to run hapao on it and write some tests if there >> are none. > > Actual

[Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-30 Thread Torsten Bergmann
Stef wrote: >good idea. >Do you know the test coverage of the class? >Because it would be nice to run hapao on it and write some tests if there are >none. Actually there are no tests - at least not in the Pharo image. Maybe Benjamin can comment if there are some tests in the Spec universe for it.

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-30 Thread Stéphane Ducasse
On Jan 30, 2013, at 4:44 PM, Torsten Bergmann wrote: > The class "OrderedIdentityDictionary" is currently defined in package > "Spec-Core". > > Since it is useful also in other scenarios and I do not want a dependency > on spec: wouldnt it make sense to move it to a more common "Collections-...

Re: [Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-30 Thread Frank Shearar
On 30 January 2013 19:44, Torsten Bergmann wrote: > The class "OrderedIdentityDictionary" is currently defined in package > "Spec-Core". > > Since it is useful also in other scenarios and I do not want a dependency > on spec: wouldnt it make sense to move it to a more common "Collections-..." > p

[Pharo-project] OrderedIdentityDictionary in a more general package

2013-01-30 Thread Torsten Bergmann
The class "OrderedIdentityDictionary" is currently defined in package "Spec-Core". Since it is useful also in other scenarios and I do not want a dependency on spec: wouldnt it make sense to move it to a more common "Collections-..." package. Since there is already "Collections-Unordered" I woul