Inspecting an OrderPreservingDictionary > entries show as nil and self prints entries
We need to add OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
Perfect! tThanks you Sven. Phil On Tue, Jun 24, 2014 at 5:54 PM, Sven Van Caekenberghe <sven@stfx.eu> wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
Where which package? Because we should track it and publish a new package. Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
Like all the #inspectorClass methods, *Spec-inspector, it was just an oversight. On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
I now see that OrderPreservingDictionary is in its own package, Collections-OrderPreservingDictionary The method should be added there in a normal protocol. Maybe in the form of ^ Smalltalk at: #EyeDictionaryInspector On 24 Jun 2014, at 22:22, Sven Van Caekenberghe <sven@stfx.eu> wrote:
Like all the #inspectorClass methods, *Spec-inspector, it was just an oversight.
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
On Wed, Jun 25, 2014 at 9:01 AM, Marcus Denker <marcus.denker@inria.fr> wrote:
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
It is very useful indeed. Question: what are the consequences of using OrderedDictionary vs Dictionary? I read the class comment but would welcome some additional info. Phil
Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
On 25 Jun 2014, at 9:26 , phil@highoctane.be wrote:
On Wed, Jun 25, 2014 at 9:01 AM, Marcus Denker <marcus.denker@inria.fr> wrote:
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
It is very useful indeed. Question: what are the consequences of using OrderedDictionary vs Dictionary? I read the class comment but would welcome some additional info.
Phil
Depends on the implementation, but in general the consequences are worse performance (and potentially memory use) for large (10+ entries? Havenât benched) Collections. If itâs done the naive way, as an OrderedCollection of Association with an expanded access protocol, O(n) rather than O(1) access is the big one. If done with pairing Dictionary/OrderedCollection internally to provide O(1) access, additional memory/insertion/removal overhead. Cheers, Henry
Am 25.06.2014 um 09:01 schrieb Marcus Denker <marcus.denker@inria.fr>:
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
SmallDictionary is an ordered dictionary. Norbert
Stef On 24/6/14 17:54, Sven Van Caekenberghe wrote:
We need to add
OrderPreservingDictionary class>>#inspectorClass ^ EyeDictionaryInspector
On 24 Jun 2014, at 17:49, phil@highoctane.be wrote:
Kind of weird.
Anyone knowing how to deal with that? Bug?
Phil
On 25 Jun 2014, at 09:49, Norbert Hartl <norbert@hartl.name> wrote:
Am 25.06.2014 um 09:01 schrieb Marcus Denker <marcus.denker@inria.fr>:
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
SmallDictionary is an ordered dictionary.
But it gets quite slow for large dictionaries⦠(it searches linearly always). Marcus
Am 25.06.2014 um 09:54 schrieb Marcus Denker <marcus.denker@inria.fr>:
On 25 Jun 2014, at 09:49, Norbert Hartl <norbert@hartl.name> wrote:
Am 25.06.2014 um 09:01 schrieb Marcus Denker <marcus.denker@inria.fr>:
On 24 Jun 2014, at 21:52, stepharo <stepharo@free.fr> wrote:
Where which package? Because we should track it and publish a new package.
We should think about have an OrderedDictionary in the base systemâ¦
SmallDictionary is an ordered dictionary.
But it gets quite slow for large dictionaries⦠(it searches linearly always).
Sure. I just wanted to say if someone has an urgent need like I had a few weeks ago when I needed it to fix the mongo driver. Norbert
participants (6)
-
Henrik Johansen -
Marcus Denker -
Norbert Hartl -
phil＠highoctane.be -
stepharo -
Sven Van Caekenberghe