February 2017 - Pharo-dev - lists.pharo.org

Re: [Pharo-dev] [QA] Scoped

by Hilaire

Thanks Ben. I think Marcus made a slide already. Hilaire Le 14/02/2017 Ã 11:45, Ben Coman a Ã©crit : > Maybe... > Browse selected package in a context that limits the scope of analysis tools > (e.g. 'Senders of...', 'Implementors of...', 'Class refs...') > > cheers -ben -- Dr. Geo http://drgeo.eu

Feb. 14, 2017

Re: [Pharo-dev] Points, Free type fonts cache and memory leaks

by Ben Coman

On Tue, Feb 14, 2017 at 9:54 PM, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > Hi, > > It is very common to see in actively used images that the number of > instances of Point is very high. Sometimes in order of hundreds of > thousands. There is several reasons for that: > > Pharo every few milliseconds checks for the new display size. That check > generates two instances of Point that are not garbage collected in > reasonably short time. So if you periodically check the result of "Point > allInstances size", you will see that it is quickly growing. > On the other hand. That Point instances are not referenced by anyone so as > soon as you do Smalltalk garbageCollect, the amount of Points in reduced. > And they are correctly garbage collected by the VM after some time which is > often longer than one minute. So no action needed. Even though these get cleared out, its still extra work for the GC. Presumably most of the time the display size is identical to the last one, so I wonder if the existing screen size might be passed to the primitive to test and avoid creating a new new point if its the same. Off the top of my head, something like this (untested)... MorphicUIManager >> checkForNewDisplaySize Display hasScreenSizeChanged ifFalse: [^ Display]. DisplayScreen startUp. World restoreMorphicDisplay. DisplayScreen >> hasScreenSizeChanged ^self class isActualScreenSizeX: width Y: height. DisplayScreen class >> isActualScreenSizeX: width Y: height <primitive: 106> InterpreterPrimitives >> primitiveScreenSize "primitive 106" "no arguments(original) ==> DisplayScreen_class>>actualScreenSize ; returning new Point from ioScreenSIze " "two arguments ==> DisplayScreen_class>>isActualyScreenSizeX;Y: ; returning Boolean of whether X,Y match ioScreenSize" | pointWord x y| pointWord := self ioScreenSize. x := pointWord >> 16 bitAnd: 65535. y := pointWord bitAnd: 65535. argumentCount = 2 ifTrue: [ ((x = (self stackValue: 1)) and: [y = (self stackTop)]) ifTrue: [self pop: 3 thenPush: objectMemory trueObject] ifFalse: [self pop: 3 thenPush: objectMemory trueObject]. ]. self pop: 1 thenPush: (self makePointwithxValue: x yValue: y) <snipped other sources of Points> cheers -ben

Feb. 14, 2017

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/60391 Home: https://github.com/pharo-project/pharo-core

Feb. 14, 2017

[pharo-project/pharo-core] 08e7f4: 60391

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: 08e7f4885ef882dcf23fd30632eb8c89ffa0c1ca https://github.com/pharo-project/pharo-core/commit/08e7f4885ef882dcf23fd306… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2017-02-14 (Tue, 14 Feb 2017) Changed paths: A Deprecated60.package/ShortRunArray.class/README.md A Deprecated60.package/ShortRunArray.class/definition.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60390.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60391.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60390.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60391.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 60391 19703 add empty ShortRunArray to Deprecated60 https://pharo.fogbugz.com/f/cases/19703 http://files.pharo.org/image/60/60391.zip

Feb. 14, 2017

Re: [Pharo-dev] Points, Free type fonts cache and memory leaks

by Pavel Krivanek

2017-02-14 15:38 GMT+01:00 Esteban Lorenzano <estebanlm(a)gmail.com>: > > > On 14 Feb 2017, at 14:54, Pavel Krivanek <pavel.krivanek(a)gmail.com> > wrote: > > > > Hi, > > > > It is very common to see in actively used images that the number of > instances of Point is very high. Sometimes in order of hundreds of > thousands. There is several reasons for that: > > > > Pharo every few milliseconds checks for the new display size. That check > generates two instances of Point that are not garbage collected in > reasonably short time. So if you periodically check the result of "Point > allInstances size", you will see that it is quickly growing. > > On the other hand. That Point instances are not referenced by anyone so > as soon as you do Smalltalk garbageCollect, the amount of Points in > reduced. And they are correctly garbage collected by the VM after some time > which is often longer than one minute. So no action needed. > > > > Then a lot of Points is stored in FreeTypeCache. That cache is a huge > beast and it tries to store a small form for every new rendered character > glyph. That means for every font, every pixel size etc. It sounds > reasonable but the main reason why the cache can be very huge is in the > fact that it stores a new glyph for every sub-pixel variant too! > > Every FreeTypeChacheEntry also contains several Point instances so it is > a important source of them. Fortunately, this cache has mechanisms to limit > its size. This limit now set to expected size of 5MB, see FreeTypeChache>># > defaultMaximumSize. > > So in the end, probably no immediate fix is needed although this cache > deserves revision. > > IMO, the full FT implementation needs to be revisited, both in image side > and in vm sideâ¦ but this is no easy work, soâ¦ for when âthere is timeââ¦ > likely never ;) > > > > > And then we have a Text related memory leak that I will try to > investigate. This memory leak does not let to garbage collect any tool > window as soon as clipboard or history (undo) was used in it. > > keep pushing! :) > (I remember there was a problem with the find&replace dialog that was a > singleton and it was keeping strong references to the (rubric) editors that > called itâ¦ maybe the leak is someplace around... > https://pharo.fogbugz.com/f/cases/19704/Memory-leak-Clipboard-can-produce-m… And I have seen very similar memory leaks related to GLMHintableActionButtonBrick, undo managers a NECController. Maybe they are related but I need to find a way how to reproduce them. -- Pavel > > Esteban > > > > > Cheers, > > -- Pavel > > > > > > > > > > > > > > > > > > >

Feb. 14, 2017

Re: [Pharo-dev] Points, Free type fonts cache and memory leaks

by Esteban Lorenzano

> On 14 Feb 2017, at 14:54, Pavel Krivanek <pavel.krivanek(a)gmail.com> wrote: > > Hi, > > It is very common to see in actively used images that the number of instances of Point is very high. Sometimes in order of hundreds of thousands. There is several reasons for that: > > Pharo every few milliseconds checks for the new display size. That check generates two instances of Point that are not garbage collected in reasonably short time. So if you periodically check the result of "Point allInstances size", you will see that it is quickly growing. > On the other hand. That Point instances are not referenced by anyone so as soon as you do Smalltalk garbageCollect, the amount of Points in reduced. And they are correctly garbage collected by the VM after some time which is often longer than one minute. So no action needed. > > Then a lot of Points is stored in FreeTypeCache. That cache is a huge beast and it tries to store a small form for every new rendered character glyph. That means for every font, every pixel size etc. It sounds reasonable but the main reason why the cache can be very huge is in the fact that it stores a new glyph for every sub-pixel variant too! > Every FreeTypeChacheEntry also contains several Point instances so it is a important source of them. Fortunately, this cache has mechanisms to limit its size. This limit now set to expected size of 5MB, see FreeTypeChache>>#defaultMaximumSize. > So in the end, probably no immediate fix is needed although this cache deserves revision. IMO, the full FT implementation needs to be revisited, both in image side and in vm sideâ¦ but this is no easy work, soâ¦ for when âthere is timeââ¦ likely never ;) > > And then we have a Text related memory leak that I will try to investigate. This memory leak does not let to garbage collect any tool window as soon as clipboard or history (undo) was used in it. keep pushing! :) (I remember there was a problem with the find&replace dialog that was a singleton and it was keeping strong references to the (rubric) editors that called itâ¦ maybe the leak is someplace around... Esteban > > Cheers, > -- Pavel > > > > > > > >

Feb. 14, 2017

Re: [Pharo-dev] IPv6 support [VM]

by Pierce Ng

On Tue, Feb 14, 2017 at 01:35:14PM +0100, Esteban Lorenzano wrote: > there was some work but it was not finished, then is kind of aborted (for > now). if you want to take a look there are some primitives in the networking > pluginâ¦ never finished, never tested, never done :) Should be there in the VM/plugin. I got Unix domain socket working... and I should dust off the Fogbugz issue that I created and add some tests. Pierce

Feb. 14, 2017

[ANN] Pharo Association: Join or Renew now!

by Marcus Denker

Hi, The new Pharo Association website is up and running for some month. If you are a member, it will bug you once a year to renew. If you were a member in the past and have not been bugged by the system to renew until now, please subscribe now as a new member. (We added all active members, but as the old system was purely manual, people did not get asked to renew very consistently and you might not be in the system anymore) See the website for more information: https://association.pharo.org <https://association.pharo.org/> Marcus

Feb. 14, 2017

[pharo-project/pharo-core] 77ef45: 60390

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: 77ef450f7fa529837ed4c6b0bec5beba1e77d120 https://github.com/pharo-project/pharo-core/commit/77ef450f7fa529837ed4c6b0… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2017-02-14 (Tue, 14 Feb 2017) Changed paths: M Nautilus.package/AbstractNautilusUI.class/instance/build ui buttons/buildBrowseScopedButton.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60389.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60390.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60389.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60390.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Zinc-HTTP.package/ZnLogEvent.class/class/convenience/open.st Log Message: ----------- 60390 19698 Scoped button with a more meaningful tooltip https://pharo.fogbugz.com/f/cases/19698 18760 Failing test: WeakAnnouncerTest>>#testNoDeadWeakSubscriptions https://pharo.fogbugz.com/f/cases/18760 19183 ZnLogEvent open is broken https://pharo.fogbugz.com/f/cases/19183 http://files.pharo.org/image/60/60390.zip

Feb. 14, 2017

[pharo-project/pharo-core]

by GitHub

Branch: refs/tags/60390 Home: https://github.com/pharo-project/pharo-core

Feb. 14, 2017