[Pharo-project] Dictionary use in SmartRefStream
While taking a look at Seaside issue 262 I noticed that SmartRefStream>>catalogValues: instVarList size: varsOnDisk creates a Dictionary with the default size, while having a pretty good idea of how large it should at least be. Is this called so little that that it doesn't matter?(it is only called in SmartRefStream>>readInsanceSize:clsname:refPosn:) Stephan Eggermont
I imagine that people did not pay attention. I think that this is important that we polish even the "details". Today I did a lecture and students got excited by Smalltalk the concept and I would love that they get excited by Smalltalk the implementation. This is why beauty in the details is important. So if you have a fix and some tests :) Stef On May 26, 2009, at 11:18 PM, Stephan Eggermont wrote:
While taking a look at Seaside issue 262 I noticed that
SmartRefStream>>catalogValues: instVarList size: varsOnDisk
creates a Dictionary with the default size, while having a pretty good idea of how large it should at least be. Is this called so little that that it doesn't matter?(it is only called in SmartRefStream>>readInsanceSize:clsname:refPosn:)
Stephan Eggermont
_______________________________________________ Pharo-project mailing list Pharo-project@lists.gforge.inria.fr http://lists.gforge.inria.fr/cgi-bin/mailman/listinfo/pharo-project
participants (2)
-
Stephan Eggermont -
Stéphane Ducasse