Issue 7384 in pharo: Consolidate order preserving dictionaries

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Issue 7384 in pharo: Consolidate order preserving dictionaries

pharo
Status: New
Owner: ----
Labels: Type-Feature Target-Collection Milestone-3.0

New issue 7384 by [hidden email]: Consolidate order preserving  
dictionaries
http://code.google.com/p/pharo/issues/detail?id=7384

Right now, we have two Dictionary implementations that try to preserve the  
order of the items:

- OrderedIdentityDictionary from Spec and
- OrderPreservingDictionary from  
http://www.squeaksource.com/OrderPreservingDict

We need to consolidate these and build one solution.


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7384 in pharo: Consolidate order preserving dictionaries

pharo

Comment #1 on issue 7384 by [hidden email]: Consolidate order  
preserving dictionaries
http://code.google.com/p/pharo/issues/detail?id=7384

There is Dictionary and IdentityDictionary, too.
So maybe having both in the ordered variant makes sense?


_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker
Reply | Threaded
Open this post in threaded view
|

Re: Issue 7384 in pharo: Consolidate order preserving dictionaries

pharo

Comment #2 on issue 7384 by [hidden email]: Consolidate order  
preserving dictionaries
http://code.google.com/p/pharo/issues/detail?id=7384

Actually there are two more implementations

Uses composition: http://ss3.gemstone.com/ss/Dictionaries
Uses inheritance from Dictionary:  
http://www.squeaksource.com/TFOrderedDictionary.html

it would be nice to have some benchmarks





_______________________________________________
Pharo-bugtracker mailing list
[hidden email]
http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-bugtracker