- Pharo-dev - lists.pharo.org

Re: [Pharo-dev] methodDict versus methodDictionary

by StÃ©phane Ducasse

>> but I have 103 send of methodDict in my image >> > Yes, I am slowly removing them, but that takes time (I removed quite a lot over the last months! A *lot*!) Ok excellent I did not know that. > I even added a lint rule detecting calls to #methodDict. > >> So I do not get it and personally I would not deprecate methodDictionary and I would remove >> methodDict because it sucks! > > methodDict should *never* be used from the outside. Itâs far too easy to get problems because you modify the dictionary > without e.g. invalidating the VM cache or telling the system. > > And it gets *much* easier to read: > > methodDict values â> methods > methodDict keys â> selectors > methodDict size > 0 â> #hasMethods > and so onâ¦ itâs amazing how much more readable code gets. > > So why do we provide a âniceâ selector âmethodDictionaryâ if this is a private accessor for internal state? > If we do that it communicates âuse meâ! And you should not. If you want to use it, you know what you are doing > and you know #methodDict. I disagree here. In that case we should start using vv jyv jvzxxx. We should put the method in a private category and we should add a comment. > And if people use it, the #methodDict is so deep in the brain of people that they never used #methoDictionary > (there where maybe 5 sendersâ¦). Still I would rename it. Stef

Dec. 15, 2013

[pharo-project/pharo-core]

by GitHub

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

Dec. 15, 2013

[pharo-project/pharo-core] ad29ad: 30645

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: ad29adb38aacff04fe1b5ab6e9b6a1854d744eca https://github.com/pharo-project/pharo-core/commit/ad29adb38aacff04fe1b5ab6… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2013-12-15 (Sun, 15 Dec 2013) Changed paths: A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script298.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30645.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st A Spec-Inspector.package/extension/SmallDictionary/class/inspectorClass.st R Spec-Inspector.package/extension/SmallDictionary/instance/inspectorClass.st A Transcript.package/ThreadSafeTranscript.class/class/declare/installThreadSafeAsSTranscript.st Log Message: ----------- 30645 12395 ThreadSafeTranscript examples do not work due to a missing method https://pharo.fogbugz.com/f/cases/12395 12428 Error while printing element https://pharo.fogbugz.com/f/cases/12428 http://files.pharo.org/image/30/30645.zip

Dec. 15, 2013

Re: [Pharo-dev] [ANN] traversal-enabled objects with DeepTraverser

by Tudor Girba

Hi, On Sat, Dec 14, 2013 at 3:44 PM, Mariano Martinez Peck < marianopeck(a)gmail.com> wrote: > > > > On Wed, Dec 11, 2013 at 12:21 PM, Tudor Girba <tudor(a)tudorgirba.com>wrote: > >> Hi, >> >> I put together a little implementation for traversals. It is inspired by >> an original implementation from Mariano. >> > > Wow... my memory is sooo bad I don't remember ;) > I asked once for something that can compute the size of an object, and you sent me a GraphTraverser class. I extended this concept and generalized it a bit. > > >> Using DeepTraverser, you can traverse arbitrary graphs by describing the >> traversal and by specifying the actions you want to perform on the nodes >> and relations. >> >> > Once (or even more times) we thought with Martin 2 things: > > 1) Reify the traversal in Fuel so that we could plug a different one. > 2) Use Fuel only for traversing and allowing user a hook to plug the what > to do with each node. > 3) Use the same traversal of 2) for deepCopying > 4) Use the same traversal of 2) to get an approximate memory consumption > of the transitive closure of an object > This would be great. > But as always happened we run out of time. > > Anyway...I think this is a nice area to explore. There are many uses for a > nice traverser. > Agreed. It would be great to build on DeepTraverser and make it more powerful. Also, did you see we wrote a Rossal extension to print the fuel traversal? > > http://rmod.lille.inria.fr/web/pier/software/Fuel/Version1.9/Documentation/… > Yes. That is cool :) Doru > The code is available in the Moose image as part of the MooseAlgos >> subproject, but can also be loaded separately in a Pharo image via: >> >> Gofer new >> url: 'http://www.smalltalkhub.com/mc/Moose/DeepTraverser/main'; >> package: âConfigurationOfDeepTraverserâ; >> load. >> (Smalltalk globals at: #ConfigurationOfDeepTraverser) loadDevelopment. >> >> >> Just to give you an idea, here are some representative examples: >> >> Number >> deep: #subclasses >> do: [:each | Transcript show: each; cr]. >> >> Number deepCollect: #subclasses. >> >> Number >> deep: #subclasses >> collect: #name. >> >> Number >> deep: #subclasses >> do: [:each | Transcript show: each; cr] >> relationDo: [ :from :to | Transcript show: from; show: ' <-- '; show: >> to; cr ]. >> >> >> >> More details about the usages and a little description of the >> implementation can be found here: >> http://www.humane-assessment.com/blog/traversal-enabled-pharo-objects/ >> >> >> Cheers, >> Doru >> >> >> >> -- >> www.tudorgirba.com >> >> "Every thing has its own flow" >> > > > > -- > Mariano > http://marianopeck.wordpress.com > -- www.tudorgirba.com "Every thing has its own flow"

Dec. 15, 2013

[pharo-project/pharo-core]

by GitHub

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

Dec. 15, 2013

[pharo-project/pharo-core] 5bca99: 30644

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: 5bca99a63da818c2369bdbead2ccf51144b48875 https://github.com/pharo-project/pharo-core/commit/5bca99a63da818c2369bdbea… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2013-12-15 (Sun, 15 Dec 2013) Changed paths: M KernelTests.package/BlockClosureTest.class/instance/tests - on-fork/testOnForkErrorTakesLessThanOneSecond.st R KernelTests.package/SizeInMemoryTest.class/instance/tests/testSizeInMemoryClasses.st R Komitter.package/Komitter.class/class/icon/iconData.st M Komitter.package/Komitter.class/class/icon/taskbarIcon.st M Komitter.package/KomitterUI.class/class/icons/historyIcon.st A Komitter.package/extension/EclipseUIThemeIcons/instance/historyIcon.st A Komitter.package/extension/EclipseUIThemeIcons/instance/historyIconContents.st A Komitter.package/extension/EclipseUIThemeIcons/instance/komitterIcon.st A Komitter.package/extension/EclipseUIThemeIcons/instance/komitterIconContents.st A Komitter.package/extension/ThemeIcons/instance/historyIcon.st A Komitter.package/extension/ThemeIcons/instance/komitterIcon.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script297.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30644.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 30644 11983 testOnForkErrorTakesLessThanOneSecond failing https://pharo.fogbugz.com/f/cases/11983 12424 testSizeInMemoryClasses is buggy https://pharo.fogbugz.com/f/cases/12424 12422 move icons from komitter to theme icons class https://pharo.fogbugz.com/f/cases/12422 http://files.pharo.org/image/30/30644.zip

Dec. 15, 2013

[pharo-project/pharo-core]

by GitHub

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

Dec. 15, 2013

[pharo-project/pharo-core] e439a6: 30643

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: e439a622234c592a87f897d7e5d0a7bcc79a1609 https://github.com/pharo-project/pharo-core/commit/e439a622234c592a87f897d7… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2013-12-15 (Sun, 15 Dec 2013) Changed paths: M Kernel.package/UndefinedObject.class/instance/class hierarchy/subclass_instanceVariableNames_classVariableNames_poolDictionaries_category_.st M KernelTests.package/DateAndTimeTest.class/instance/tests - under design/testNotSymmetricWithString.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script296.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30641.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30642.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30643.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Spec-Examples.package/DynamicSpecExample.class/instance/accessing/object.st M Spec-Examples.package/DynamicSpecExample.class/instance/accessing/object_.st M Spec-Examples.package/DynamicSpecExample.class/instance/protocol/open.st M Spec-Examples.package/DynamicSpecExample.class/instance/protocol/openOnInteger.st M Spec-Examples.package/DynamicSpecExample.class/instance/protocol/openOnString.st R Tests.package/ReleaseTest.class/instance/testing/testObsoleteBehaviors.st Log Message: ----------- 30643 12421 Fix DateAndTimeTest >> #testNotSymmetricWithString https://pharo.fogbugz.com/f/cases/12421 12420 DynamicSpecExample uses deprecated messages https://pharo.fogbugz.com/f/cases/12420 12423 Subclass of undefined classes should be subclasses of Object and not ProtoObject https://pharo.fogbugz.com/f/cases/12423 12165 #testObsoleteBehaviors failing after running native boost tests https://pharo.fogbugz.com/f/cases/12165 http://files.pharo.org/image/30/30643.zip

Dec. 15, 2013

Re: [Pharo-dev] Is it possible to load and display pdfs inside pharo ?

by kilon alios

Is there any plan to display Pier files inside Pharo via Athens or Spec/Morphic ? On Fri, Dec 13, 2013 at 5:51 PM, StÃ©phane Ducasse <stephane.ducasse(a)inria.fr > wrote: > So far I do not think so but I know that the open-source library of pdf > developed by christian Haider on VW > can do it. > > Stef > > > I was wondering if there is a library to load and display or at least > parse pdf files for pharo. > > >

Dec. 15, 2013

Re: [Pharo-dev] strange bug when opening the morph menu

by Tudor Girba

Exactly, Nicolai! Thanks for reproducing. I added your description to the issue. Essentially, I just wanted to get the halo menu in order to export a debugger window to png. Doru On Sat, Dec 14, 2013 at 10:26 PM, Nicolai Hess <nicolaihess(a)web.de> wrote: > If you press and hold the mousebutton the menu is visible. > If you just click the mousebutton the menu flashes for a short time > and the window is in "grab" mode. > Actually the grab-menu item is selected, because the menu > is such wide, that it does not open beneath the mouse pointer, > and so the mouse pointer is always over some menu item. > (at leas with on windows) > > If you have a normal workspace title bar, the menu stays on even if > you just click the mousebutton. > > nicolai > > > > 2013/12/14 StÃ©phane Ducasse <stephane.ducasse(a)inria.fr> > >> >> >> Hi, >> >> Try this: >> >> 1. Execute >> Workspace open >> 2. Try to get the menu of the window >> ==> it opens just fine >> >> 1. Execute >> Workspace open label: (1 to: 100) asOrderedCollection asString >> 2. Try to get the menu of the window >> ==> it does not open properly (it opens large and it closes quickly) and >> the mouse remains in dragging mode >> >> >> I do not get this problem. >> When I want to edit the title it just takes the complete screen width. >> >> >> I opened a bug report: >> >> https://pharo.fogbugz.com/f/cases/12426/Morph-menu-has-strange-behavior-whe… >> >> Does anyone have any idea of why it happens? >> >> Cheers, >> Doru >> >> >> -- >> www.tudorgirba.com >> >> "Every thing has its own flow." >> >> >> > -- www.tudorgirba.com "Every thing has its own flow"

Dec. 15, 2013