- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] Building .deb with Pharo VM

by Damien Cassou

On Fri, Sep 26, 2014 at 9:42 AM, Jan Vrany <jan.vrany(a)fit.cvut.cz> wrote: > However, I already exhausted my free time. I'll try again later and > get back to you. I created a job that builds a nice archive with pre-patched sources: http://files.pharo.org/vm/src/vm-unix-sources/blessed. -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." Winston Churchill

Sept. 26, 2014

Re: [Pharo-dev] Some methods do not belong to their origin

by Damien Cassou

On Fri, Sep 26, 2014 at 5:01 PM, Damien Cassou <damien.cassou(a)gmail.com> wrote: > https://pharo.fogbugz.com/f/cases/14091 no need to click on this one, sorry -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." Winston Churchill

Sept. 26, 2014

[pharo-project/pharo-core] 12d0ee: 40261

by GitHub

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: 12d0ee50942d265284f62231d9377bce1c143842 https://github.com/pharo-project/pharo-core/commit/12d0ee50942d265284f62231… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-09-26 (Fri, 26 Sep 2014) Changed paths: M FreeType.package/FreeTypeFontProvider.class/instance/loading and updating/updateFromFile_.st M Fuel.package/FLRectangleCluster.class/instance/serialize%2Fmaterialize/materializeReferencesOf_with_.st R Kernel.package/DegeneratedRectangle.class/README.md R Kernel.package/DegeneratedRectangle.class/definition.st R Kernel.package/Rectangle.class/instance/private/setOrigin_corner_.st M Morphic-Core.package/DamageRecorder.class/instance/recording/recordInvalidRect_.st M Morphic-Core.package/MouseButtonEvent.class/instance/accessing/blueButtonChanged.st M Morphic-Core.package/MouseButtonEvent.class/instance/accessing/redButtonChanged.st M Morphic-Core.package/MouseButtonEvent.class/instance/accessing/yellowButtonChanged.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script261.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40261.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Text-Scanning.package/CharacterBlock.class/instance/private/stringIndex_text_topLeft_extent_.st Log Message: ----------- 40261 14054 Cant load font from a path with accented characters https://pharo.fogbugz.com/f/cases/14054 11021 MouseEvent>>blueButtonChanged https://pharo.fogbugz.com/f/cases/11021 14075 Halt once in Rectangle>>setOrigin:corner: https://pharo.fogbugz.com/f/cases/14075 http://files.pharo.org/image/40/40261.zip

Sept. 26, 2014

[pharo-project/pharo-core]

by GitHub

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

Sept. 26, 2014

Some methods do not belong to their origin

by Damien Cassou

https://pharo.fogbugz.com/f/cases/14091 -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." Winston Churchill

Sept. 26, 2014

Re: [Pharo-dev] [Vm-dev] Re: catching FFI errors

by Ben Coman

Eliot Miranda wrote: > > Hi Ben, > > On Sep 25, 2014, at 5:50 PM, Ben Coman <btc(a)openInWorld.com> wrote: > >> I had a brief moment wondering about the general problem we have when calling out through FFI that we expose ourselves to bugs in the external library that crash the Image. I wonder... >> >> 1. Debuggers like gdb can run a program that crashes, without themselves itself crashing. Could our FFI calls be wrapped in similar mechanism? >> >> 2. Languages have exception handling, so that errors like divide by zero can be caught. Could our FFI calls be wrapped in similar mechanism? >> cheers -ben > > one thing that is pretty straight-forward to do is to catch and report errors during FFI calls as primitive failures with error codes. I added this to the VisualWorks VM a while ago It works like this: > > 1. The VM installs a first-level exception handler (windows) or signal handlers for SIGSEGV et al (unix). (This is something the VM already does). > > 2. When an FFI call out is made the VM can find the activation they made an FDI call out. It must be able to do this if it is to handle a callback. > > 3. An FFI call out is modified to set a flag, eg inFFICall, that is tested by the first-level exception handler. The flag is cleared when the FFI call completes or cleared when taking a callback and restored when the callback returns. > > 4. When the first level exception/signal handler is invoked it tests the inFFICall flag. If unset it does what it always did (exit, do nothing, etc). If the flag is set it collects the error information, located the FFI activation for the FFI call, does a long jump that returns from the FFI call as a primitive failure, supplying the error information. > > Now this is useful because the VM doesn't crash (provided the error was localized in its effects and didnt eg corrupt the heap), and hence easy for the developer to find out which FFI call provoked the error, but not as useful as catching the error in gdb. The exception info is typically an exception code and an address, which may be difficult for the programmer to decode. Hence the mechanism needs to be made optional, typically in by default, and then disabled to investigate under gdb. > > Does this sound useful? > > Eliot (phone) Its good to know how that works, and that it is standard already. Now I wonder, (though it might not be portable) would it be feasible to use something like mprotect to make the Image memory read-only for the duration of the FFI call? Maybe even use a separate segment just for FFI calls (is it even possible for processes to have additional segments than the usual heap/stack/? Thanks for the detailed reply Eliot, but note (and I should have said already), these are idle thoughts. I'll file answers for later reference, but its not something I'm working on right now. cheers -ben

Sept. 26, 2014

[pharo-project/pharo-core]

by GitHub

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

Sept. 26, 2014

[pharo-project/pharo-core] 42a25b: 40260

by GitHub

Branch: refs/heads/4.0 Home: https://github.com/pharo-project/pharo-core Commit: 42a25b86ccb92e42989a04cf08898403c2a280d8 https://github.com/pharo-project/pharo-core/commit/42a25b86ccb92e42989a04cf… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-09-26 (Fri, 26 Sep 2014) Changed paths: A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/README.md A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/class/instance creation/on_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/definition.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/balloonText.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/helpText.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/helpText_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/offImageSelector.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/offImageSelector_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/onImageSelector.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/onImageSelector_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/pressedImageSelector.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/pressedImageSelector_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/stateSelector.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/stateSelector_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/accessing/target_.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateOffImage.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateOnImage.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/updating/updatePressedImage.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateState.st A Morphic-Widgets-Basic.package/PluggableThreePhaseButtonMorph.class/instance/updating/update_.st R Morphic-Widgets-Windows.package/StandardWindow.class/instance/accessing/open.st R Morphic-Widgets-Windows.package/StandardWindow.class/instance/initialization/defaultColor.st R Morphic-Widgets-Windows.package/StandardWindow.class/instance/initialization/initialize.st A Morphic-Widgets-Windows.package/StandardWindow.class/instance/initialize/defaultColor.st A Morphic-Widgets-Windows.package/StandardWindow.class/instance/initialize/initialize.st A Morphic-Widgets-Windows.package/StandardWindow.class/instance/initialize/withText_label_.st R Morphic-Widgets-Windows.package/StandardWindow.class/instance/macpal/flash.st A Morphic-Widgets-Windows.package/StandardWindow.class/instance/open%2Fclose/open.st A Morphic-Widgets-Windows.package/StandardWindow.class/instance/operations/flash.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/README.md R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/class/instance creation/on_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/definition.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/balloonText.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/helpText.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/helpText_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/offImageSelector.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/offImageSelector_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/onImageSelector.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/onImageSelector_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/pressedImageSelector.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/pressedImageSelector_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/stateSelector.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/stateSelector_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/accessing/target_.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateOffImage.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateOnImage.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/updating/updatePressedImage.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/updating/updateState.st R NautilusCommon.package/PluggableThreePhaseButtonMorph.class/instance/updating/update_.st A NewValueHolder.package/NewValueHolder.class/instance/accessing/rawValue_.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - scripts/script260.st A ScriptLoader40.package/ScriptLoader.class/instance/pharo - updates/update40260.st M ScriptLoader40.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st R Spec-Extensions.package/extension/NewValueHolder/instance/rawValue_.st R System-FilePackage.package/ClassDiffBuilder.class/README.md R System-FilePackage.package/ClassDiffBuilder.class/definition.st R System-FilePackage.package/ClassDiffBuilder.class/instance/private/print_withAttributes_on_.st R System-FilePackage.package/ClassDiffBuilder.class/instance/private/split_.st R System-FilePackage.package/DiffElement.class/README.md R System-FilePackage.package/DiffElement.class/class/instance creation/string_.st R System-FilePackage.package/DiffElement.class/definition.st R System-FilePackage.package/DiffElement.class/instance/accessing/match.st R System-FilePackage.package/DiffElement.class/instance/accessing/match_.st R System-FilePackage.package/DiffElement.class/instance/accessing/matches_.st R System-FilePackage.package/DiffElement.class/instance/accessing/string.st R System-FilePackage.package/DiffElement.class/instance/accessing/string_.st R System-FilePackage.package/DiffElement.class/instance/comparing/=.st R System-FilePackage.package/DiffElement.class/instance/comparing/hash.st R System-FilePackage.package/DiffElement.class/instance/printing/printOn_.st R System-FilePackage.package/DiffElement.class/instance/testing/hasMatch.st R System-FilePackage.package/PrettyTextDiffBuilder.class/README.md R System-FilePackage.package/PrettyTextDiffBuilder.class/class/instance creation/from_to_inClass_.st R System-FilePackage.package/PrettyTextDiffBuilder.class/definition.st R System-FilePackage.package/PrettyTextDiffBuilder.class/instance/initialize/sourceClass_.st R System-FilePackage.package/PrettyTextDiffBuilder.class/instance/initialize/split_.st R System-FilePackage.package/TextDiffBuilder.class/README.md R System-FilePackage.package/TextDiffBuilder.class/class/class initialization/initialize.st R System-FilePackage.package/TextDiffBuilder.class/class/class initialization/initializeTextAttributes.st R System-FilePackage.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_.st R System-FilePackage.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_inClass_.st R System-FilePackage.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_inClass_prettyDiffs_.st R System-FilePackage.package/TextDiffBuilder.class/class/instance creation/from_to_.st R System-FilePackage.package/TextDiffBuilder.class/class/instance creation/from_to_inClass_.st R System-FilePackage.package/TextDiffBuilder.class/class/settings/differatorSettingsOn_.st R System-FilePackage.package/TextDiffBuilder.class/class/settings/diffsWithPrettyPrint.st R System-FilePackage.package/TextDiffBuilder.class/class/settings/diffsWithPrettyPrint_.st R System-FilePackage.package/TextDiffBuilder.class/class/settings/ignoreLineEndings.st R System-FilePackage.package/TextDiffBuilder.class/class/settings/ignoreLineEndings_.st R System-FilePackage.package/TextDiffBuilder.class/definition.st R System-FilePackage.package/TextDiffBuilder.class/instance/creating patches/buildDisplayPatch.st R System-FilePackage.package/TextDiffBuilder.class/instance/creating patches/buildPatchSequence.st R System-FilePackage.package/TextDiffBuilder.class/instance/creating patches/patchSequenceDoIfMatch_ifInsert_ifRemove_.st R System-FilePackage.package/TextDiffBuilder.class/instance/initialize/from_to_.st R System-FilePackage.package/TextDiffBuilder.class/instance/private/findMatches.st R System-FilePackage.package/TextDiffBuilder.class/instance/private/lcsFor_and_.st R System-FilePackage.package/TextDiffBuilder.class/instance/private/print_withAttributes_on_.st R System-FilePackage.package/TextDiffBuilder.class/instance/private/split_.st A Text-Diff.package/ClassDiffBuilder.class/README.md A Text-Diff.package/ClassDiffBuilder.class/definition.st A Text-Diff.package/ClassDiffBuilder.class/instance/private/print_withAttributes_on_.st A Text-Diff.package/ClassDiffBuilder.class/instance/private/split_.st A Text-Diff.package/DiffElement.class/README.md A Text-Diff.package/DiffElement.class/class/instance creation/string_.st A Text-Diff.package/DiffElement.class/definition.st A Text-Diff.package/DiffElement.class/instance/accessing/match.st A Text-Diff.package/DiffElement.class/instance/accessing/match_.st A Text-Diff.package/DiffElement.class/instance/accessing/matches_.st A Text-Diff.package/DiffElement.class/instance/accessing/string.st A Text-Diff.package/DiffElement.class/instance/accessing/string_.st A Text-Diff.package/DiffElement.class/instance/comparing/=.st A Text-Diff.package/DiffElement.class/instance/comparing/hash.st A Text-Diff.package/DiffElement.class/instance/printing/printOn_.st A Text-Diff.package/DiffElement.class/instance/testing/hasMatch.st A Text-Diff.package/PrettyTextDiffBuilder.class/README.md A Text-Diff.package/PrettyTextDiffBuilder.class/class/instance creation/from_to_inClass_.st A Text-Diff.package/PrettyTextDiffBuilder.class/definition.st A Text-Diff.package/PrettyTextDiffBuilder.class/instance/initialize/sourceClass_.st A Text-Diff.package/PrettyTextDiffBuilder.class/instance/initialize/split_.st A Text-Diff.package/TextDiffBuilder.class/README.md A Text-Diff.package/TextDiffBuilder.class/class/class initialization/initialize.st A Text-Diff.package/TextDiffBuilder.class/class/class initialization/initializeTextAttributes.st A Text-Diff.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_.st A Text-Diff.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_inClass_.st A Text-Diff.package/TextDiffBuilder.class/class/instance creation/buildDisplayPatchFrom_to_inClass_prettyDiffs_.st A Text-Diff.package/TextDiffBuilder.class/class/instance creation/from_to_.st A Text-Diff.package/TextDiffBuilder.class/class/instance creation/from_to_inClass_.st A Text-Diff.package/TextDiffBuilder.class/class/settings/differatorSettingsOn_.st A Text-Diff.package/TextDiffBuilder.class/class/settings/diffsWithPrettyPrint.st A Text-Diff.package/TextDiffBuilder.class/class/settings/diffsWithPrettyPrint_.st A Text-Diff.package/TextDiffBuilder.class/class/settings/ignoreLineEndings.st A Text-Diff.package/TextDiffBuilder.class/class/settings/ignoreLineEndings_.st A Text-Diff.package/TextDiffBuilder.class/definition.st A Text-Diff.package/TextDiffBuilder.class/instance/creating patches/buildDisplayPatch.st A Text-Diff.package/TextDiffBuilder.class/instance/creating patches/buildPatchSequence.st A Text-Diff.package/TextDiffBuilder.class/instance/creating patches/patchSequenceDoIfMatch_ifInsert_ifRemove_.st A Text-Diff.package/TextDiffBuilder.class/instance/initialize/from_to_.st A Text-Diff.package/TextDiffBuilder.class/instance/private/findMatches.st A Text-Diff.package/TextDiffBuilder.class/instance/private/lcsFor_and_.st A Text-Diff.package/TextDiffBuilder.class/instance/private/print_withAttributes_on_.st A Text-Diff.package/TextDiffBuilder.class/instance/private/split_.st M Tool-Profilers.package/MessageTally.class/class/spying/spyAllOn_cutoff_.st M Tool-Profilers.package/MessageTally.class/class/spying/spyOnProcess_forMilliseconds_reportOtherProcesses_.st M Tool-Profilers.package/MessageTally.class/class/spying/spyOn_reportOtherProcesses_cutoff_openResultWindow_closeAfter_.st M Tool-Profilers.package/MessageTally.class/class/spying/tallySendsTo_inBlock_showTree_closeAfter_openResultWindow_.st M Tool-Profilers.package/SpaceTally.class/instance/class analysis/compareTallyIn_to_.st A Tool-Profilers.package/SpaceTally.class/instance/fileout/computeSpaceForClass_.st A Tool-Profilers.package/SpaceTally.class/instance/fileout/printHeader_.st M Tool-Profilers.package/SpaceTally.class/instance/fileout/printSpaceAnalysis_on_.st M Tool-Profilers.package/SpaceTally.class/instance/fileout/printSpaceDifferenceFrom_to_.st A Tool-Profilers.package/SpaceTally.class/instance/fileout/printSpaceTallyItem_percent_on_.st Log Message: ----------- 40260 14028 PluggableThreePhaseButtonMorph should be packaged with Morphic-Widgets-Basic https://pharo.fogbugz.com/f/cases/14028 13668 remove the last StringHolder references https://pharo.fogbugz.com/f/cases/13668 13635 Package Spec-Extensions just contains one accessor --> can be moved to the class https://pharo.fogbugz.com/f/cases/13635 5625 TextDiffBuilder in System-FilePackage ? https://pharo.fogbugz.com/f/cases/5625 http://files.pharo.org/image/40/40260.zip

Sept. 26, 2014

Re: [Pharo-dev] showing decompiled code in the browser (was "Nautilus roadmap / icons")

by ClÃ©ment Bera

2014-09-26 13:58 GMT+02:00 Henrik Johansen <henrik.s.johansen(a)veloxit.no>: > > On 26 Sep 2014, at 9:08 , Esteban Lorenzano <estebanlm(a)gmail.com> wrote: > > > > > On 26 Sep 2014, at 09:04, Craig Latta <craig(a)netjam.org> wrote: > > > >> > >> Hi Marcus-- > >> > >>> Honestly, I do not get why [showing decompiled code in the browser] > >>> was useful, ever. > >> > >> Some of my debugging sessions require using the virtual machine > >> simulator. I find browser-presented decompilation useful for finding my > >> place in a method I'm running that way. It's also useful for learning > >> about how the compiler and virtual machine work. > > > > so is a good functionality for putting in plugin, not a standard feature > of nautilus: if working with vm simulator, if debugging it, ifâ¦ > > > > Esteban > > > I've also found it useful when doing optimization work on normal code. > If you're not entirely certain what the compiler is capable of, it helps > you determine things like wether temps are automatically moved to the > smallest enclosing scope, whether a certain function gets inlined, etc. > I imagine with Sista, it becomes an even more essential tool for checking > what the compiler actually produces. Building and debugging something like > that without a decompiler also seems like a whole lot of pain... > Hello, Two decompilers are available in Pharo 3, one for each compiler. They both can decompile 99% of Pharo methods. I enhanced one of them to decompile simple sista methods. However, the optimizer may generate code that cannot be expressed at source level, so some method cannot be decompiled. I am not sure the right place for decompiled sista method is in nautilus. In nautilus the user now sees the source code of the non optimized method, and if he edits it and compiles it, the optimized method is removed from method dictionary. It would be strange to have decompiled code not matching the source code. However, I have my own tools using the decompiler to debug sista. > > Cheers, > Henry >

Sept. 26, 2014

Re: [Pharo-dev] Copy class needs more information to do a good job

by Camille Teruel

Another annoying thing is that when you copy a hierarchy you have to change the subclass relationships manually. For example, if you select two classes -- Superclass Subclass -- and copy them you end up with NewSubclass as a subclass of Superclass instead of NewSuperclass. When copying a big hierarchy it's tiresome. On 26 sept. 2014, at 14:08, Stephan Eggermont <stephan(a)stack.nl> wrote: > Copy class needs more information to do a good job > > At the moment, the copy class command only asks for the name of the new class. That is not enough information. There have been a number of issues posted about copying (or not) of the class comment, Issue 13247, 10039, 12294 and 12265. > > Other information that can be helpful is the package where the new class should be. I nearly never need a copy in the package where the original was. > > Cheers, > Stephan > > https://pharo.fogbugz.com/default.asp?14084

Sept. 26, 2014