Issue 2711 Monticello wastes significant amounts of memory
The number of distinct authors of monticello packages is not so large. MCVersionInfo allInstances size 13984 Pharo 30716 30721 Moose 5.0 (on 30714) (MCVersionInfo allInstances collect: [ :each | each author ]) asSet size 129 Pharo 30716 269 Moose 5.0 Can we expect issues if we replace the author string by a symbol? WideString issues perhaps? initializeWithName: vName id: aUUID message: aString date: aDate time: aTime author: initials ancestors: aCollection stepChildren: stepCollection name := vName. id := aUUID. message := aString. date := aDate. time := aTime. author := initials asSymbol. ancestors := aCollection. stepChildren := stepCollection author ^ author asString and then replace all authors by their symbols in the image? We could do the same for date (1821 unique in 30716, 3027 in Moose 5) This is issue 2711 I donât think this should be a must-fix for 3.0 Stephan
Stephan Eggermont wrote
The number of distinct authors of monticello packages is not so large. ... Can we expect issues if we replace the author string by a symbol? ⦠I donât think this should be a must-fix for 3.0
If it's not for 4.0, how about a real object? Something like "Authors authorNamed: aString". ----- Cheers, Sean -- View this message in context: http://forum.world.st/Issue-2711-Monticello-wastes-significant-amounts-of-me... Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
Sean P. DeNigris wrote
If it's not for 4.0...
Should have been "not for 3.0"⦠duh... ----- Cheers, Sean -- View this message in context: http://forum.world.st/Issue-2711-Monticello-wastes-significant-amounts-of-me... Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.
participants (2)
-
Sean P. DeNigris -
Stephan Eggermont