- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Pharo 5: browser windows are extremely resource hungry
by Max Leske July 17, 2015

July 17, 2015

Apparently this is a polling bug in the new FastTable. FastTable should not have been activated yet Esteban told me and in the newest build indeed FastTable has been deactivated again and the behaviour I described is no longer reproducible. Max > Hi > > I discovered that in Pharo 5, having two or more browser windows open significantly reduces responsiveness in the image (delayed clicks, ghosting when moving windows etc.). This is especially true if a method has been selected in two or more windows. The ghosting becomes much worse when you drag a window. I checked that the same does not happen in Pharo 4. > > I originally thought that there was a strong impact on CPU load too, but itâs actually not that bad. > > Has anybody seen this or even worked on a fix for it? If not, Iâll open a bug report. > > > Cheers, > Max

4 3

Seeking help on morphs : expander and scrolls
by Jigyasa Grover July 17, 2015

July 17, 2015

Hi I am using Expander Morphs to display some search results. I am finding it hard to insert a scroll bar in text displayed after expanding the title of the search result. The corresponding code snippet is as: *dialog newRow: {* *(* * (* * dialog newExpander: r title asString* * forAll: { dialog newText: r content asString . }* * )minWidth:600; minHeight: 10* *)cellInset: 0;* * borderStyle: (BorderStyle inset baseColor: dialog paneColor; width: 1)* * } * I basically need to add a scroll-able content in '*newText*:' portion. The entire code of my application an offline text search application namely searchQuick is available at github (https://github.com/jig08/sQuick_new) . You all are welcome to fork the code , try the infancy application and send in suggestions. Help Appreciated. Regards Jigyasa

2 1

libSDL2 not present in full download
by jannik laval July 17, 2015

July 17, 2015

Hi pharoers, I was trying to use OSWindow, but in a "full Download" (Image+VM), the plugin SDL2 is not present. Does this mean that the full download is based on an old VM ? It is the case for Pharo 4 and Pharo 5. Cheers, -- ~~Jannik Laval~~ Ãcole des Mines de Douai Enseignant-chercheur http://www.jannik-laval.eu http://www.phratch.com http://www.approchealpes.info http://car.mines-douai.fr/

4 5

Copy-on-write for a multithreaded VM
by Ben Coman July 17, 2015

July 17, 2015

I am curious what are the issues holding us back from a multithreaded VM being mainstream, since I had a passing thought about a strategy for a partial-multi-threaded-VM. I see multi-threading been prototyped before with RoarVM [1] & HydraVM [2], and the CogBlog project page [3] says "While multi-threading seems like an obvious and important direction to take the system in, making the VM multi-threaded per-se **does not provide any benefit before the Smalltalk image is made thread-safe** and that is probably more work than providing a multi-threaded VM. Hence a potentially more profitable approach is to concentrate on federating multiple VMs running multiple images, communicating through the threaded FFI." So I agree it would be a big job to make the whole Image thread-safe, but I wonder if a useful subset would be threads that are unable to disturb the system state? This might cover a lot of needs for parallelism, for example: * Complex force layouts with Roassal * Web server worker threads * Screen rendering by #fullDraw: & #drawOn: A copy-on-write facility might provide this, such that any required system state changes require manual coding in the parent-thread to process a result object returned from the child-thread when it exits. This might be implemented by combining Spur's new features for: * immutability * lazy become forwarding Implementation might require an extra object header bit WriteCopied and a new class CopyOnWriteThread known by the VM. Whenever a child-COWT reads an object, if WriteCopied is not set, set both it and the Immutable bits. Any thread writing to that object will trigger the existing immutability handler, which now additionally, when WriteCopied is set: 1. Creates a new WriteCopiedIndirection object holding a. the old object b. the newly written object c. the thread that performed the write. 2. Sets the old object's forward-pointer to that. Later the forward-following code observes the LocalCopy bit and based on matching the current thread with that stored in 1c. unrolls the correct object. The child-COWT leaves the WriteCopied bit set so as to avoid immutability being set again. Actually that precludes using real-immutability in a COWT, so my description above is not ideal - but hopefully is enough to show intent. Could this avoid needing to make the whole image thread safe? and thus facilitate running Smalltalk worker threads across multiple CPUs? Actually, this might even be useful without running across multiple CPUs, since the cause of several intermittent Red-Screen-Of-Deaths was due to using multiple green-threads to improve UI interactivity. (For example forking update of Monticello lists to allow typing into search boxes. Here #drawOn: had "aList size" returning different values halfway through the algorithm. Where execution forked to provide this useful feature was the best place for code readability and intuitive understanding, but it was long path to trace through execution to discover the coupling between there and the rendering code. Certainly the fix is less readable that the original.) [1] https://github.com/smarr/RoarVM [2] http://squeakvm.org/~sig/hydravm/devnotes.html [3] http://www.mirandabanda.org/cogblog/cog-projects/ cheers -ben

2 1

Re: [Pharo-dev] How to combine event listeners in Bloc?
by Stephan Eggermont July 16, 2015

July 16, 2015

On 16-07-15 11:41, Alain Plantec via Pharo-dev wrote: >i think it is better as specific feature of the pane event listener It feels a bit like feature envy. Too much poking around. BlWellPanelFlowEventListener>>PaneldropAccepted: anEvent |wellPanelFlow droppedMorph localPosition found| wellPanelFlow := anEvent handler. droppedMorph := anEvent contents asDragWell. localPosition := wellPanelFlow globalPointToLocal: anEvent position. wellPanelFlow hasSubmorphs ifFalse: [ wellPanelFlow addMorph: droppedMorph ]. found := wellPanelFlow morphsAt: localPosition. found ifNotNil: [ "We found at least one. The deepest is first, that's the one we want" found := found first. found bounds center x < localPosition x ifTrue: [ "to the right of its center, so after it?" wellPanelFlow addMorph: droppedMorph after: found ] ifFalse: [ wellPanelFlow addMorph: droppedMorph inFrontOf: found ] ] ifNil: [ found := self precedingMorphIn: wellPanelFlow submorphs at: localPosition. found ifNil: [ wellPanelFlow addMorphFront: droppedMorph ] ifNotNil: [ wellPanelFlow addMorph: droppedMorph after: found ] ] BlWellPanelFlowEventListener>>precedingMorphIn: aCollection at: localPosition |precedingMorph| precedingMorph := nil. aCollection do: [ :aMorph | aMorph bounds bottom < localPosition y ifTrue: [ "wrong row" precedingMorph := aMorph ] ifFalse: [ aMorph bounds right < localPosition x ifTrue: [ precedingMorph := aMorph] ifFalse: [ ^aMorph ]] ]. ^precedingMorph

1 1

[pharo-project/pharo-core]
by GitHub July 16, 2015

July 16, 2015

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

1 0

[pharo-project/pharo-core] 0b9a9b: 50174
by GitHub July 16, 2015

July 16, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 0b9a9ba2b14eb3fc549d4a666a7138feed9e088b https://github.com/pharo-project/pharo-core/commit/0b9a9ba2b14eb3fc549d4a66… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-07-16 (Thu, 16 Jul 2015) Changed paths: M Collections-Sequenceable.package/SparseLargeTable.class/instance/accessing/zapDefaultOnlyEntries.st R Collections-Sequenceable.package/SparseLargeTable.class/instance/private/findLastNonNilSubTable.st M Monticello.package/MCOrganizationDefinition.class/instance/accessing/description.st A Monticello.package/MCOrganizationDefinition.class/instance/private/fullDescription.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50173.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50174.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50173.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50174.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50174 14616 Package Tags causing merge conflict https://pharo.fogbugz.com/f/cases/14616 15987 SparseLargeTable>>zapDefaultOnlyEntries creates an unused copy https://pharo.fogbugz.com/f/cases/15987 http://files.pharo.org/image/50/50174.zip

1 0

[pharo-project/pharo-core]
by GitHub July 16, 2015

July 16, 2015

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

1 0

[pharo-project/pharo-core] a710ed: 50173
by GitHub July 16, 2015

July 16, 2015

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: a710ed58b1a2dc4e9d8d31ed7af96371366ae5bd https://github.com/pharo-project/pharo-core/commit/a710ed58b1a2dc4e9d8d31ed… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2015-07-16 (Thu, 16 Jul 2015) Changed paths: M ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/symbolic versions/stable_.st A ConfigurationOfCatalog.package/ConfigurationOfCatalog.class/instance/versions/version06_.st M Nautilus.package/AbstractNautilusUI.class/instance/build ui/addAll_.st M Nautilus.package/AbstractNautilusUI.class/instance/build ui/buildColumns_height_.st A Nautilus.package/AbstractNautilusUI.class/instance/build ui/navigationAreaProportion.st M NautilusCommon.package/NautilusWindow.class/instance/open%2Fclose/initialExtent.st M Reflectivity-Tests.package/MetaLinkTest.class/instance/tests - options/testLinkOption.st A Reflectivity-Tests.package/ReflectivityLinkTest.class/instance/tests - reflective disable/testUninstallLinkAndRunOptionNoJIT.st A Reflectivity.package/MetaLink.class/instance/accessing/option_.st A Reflectivity.package/MetaLink.class/instance/accessing/options_.st R Reflectivity.package/MetaLink.class/instance/as yet unclassified/option_.st A Reflectivity.package/ReflectiveMethod.class/instance/evaluation/compileAndInstallCompiledMethod.st M Reflectivity.package/ReflectiveMethod.class/instance/evaluation/run_with_in_.st R Reflectivity.package/ReflectiveMethod.class/instance/testing/installLink.st A Reflectivity.package/ReflectiveMethod.class/instance/testing/installLink_.st R Reflectivity.package/extension/CompiledMethod/instance/installLink.st A Reflectivity.package/extension/CompiledMethod/instance/installLink_.st M Reflectivity.package/extension/RBProgramNode/instance/link_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50172.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50173.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50172.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50173.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A Tool-Catalog.package/CatalogCommandLineHandler.class/README.md A Tool-Catalog.package/CatalogCommandLineHandler.class/class/accessing/commandName.st A Tool-Catalog.package/CatalogCommandLineHandler.class/class/accessing/description.st A Tool-Catalog.package/CatalogCommandLineHandler.class/definition.st A Tool-Catalog.package/CatalogCommandLineHandler.class/instance/accessing/projectName.st A Tool-Catalog.package/CatalogCommandLineHandler.class/instance/accessing/version.st A Tool-Catalog.package/CatalogCommandLineHandler.class/instance/activation/activate.st A Tool-Catalog.package/CatalogCommandLineHandler.class/instance/activation/getProject.st A Tool-Catalog.package/CatalogCommandLineHandler.class/instance/private/warning_.st A Tool-Catalog.package/CatalogProject.class/instance/installing/installVersion_.st Log Message: ----------- 50173 15985 Bottom Plugin takes too much space https://pharo.fogbugz.com/f/cases/15985 15986 implement optionCompileOnLinkInstallation https://pharo.fogbugz.com/f/cases/15986 15988 add catalog command line handler https://pharo.fogbugz.com/f/cases/15988 http://files.pharo.org/image/50/50173.zip

1 0

Re: [Pharo-dev] How to combine event listeners in Bloc?
by Stephan Eggermont July 16, 2015

July 16, 2015

Nice, looking good. Thank you. Making good progress. I can now drag-drop a color from one panel to another. In Morphic the layout would then be responsible for determining at which position in the submorph collection this would be added. In morphic that is done with LayoutPolicy>>indexForInserting: aMorph at: aPoint in: someMorph "Return the insertion index based on the layout strategy defined for some morph. Used for drop insertion." ^1 "front-most" which is only overridden in TableLayout. Is that a separate responsibility in Bloc or is that part of the responsibility of a LayoutStrategy (or even an EventListener)? Stephan

2 1