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

Re: [Pharo-dev] missing variable names in context inspector (from issue 12800?)

by Sven Van Caekenberghe

And there is an obsolete instance ... On 05 Feb 2014, at 15:03, Nicolai Hess <nicolaihess(a)web.de> wrote: > Since the update 30742 , the context inspector (in a Debugger window) > shows only a list of numbers instead of that contexts variable names. > > > Nicolai

Feb. 5, 2014

missing variable names in context inspector (from issue 12800?)

by Nicolai Hess

Since the update 30742 , the context inspector (in a Debugger window) shows only a list of numbers instead of that contexts variable names. Nicolai

Feb. 5, 2014

Re: [Pharo-dev] Shortcuts

by Sven Van Caekenberghe

Hi Diego, What platform are you on ? In what tool do the shortcuts fail ? Everything seems to work as expected for me on Mac OS X, #30742. Sven On 05 Feb 2014, at 14:26, Diego Lont <diego.lont(a)delware.nl> wrote: > Hi all, > > I am probably a stupid user, but I cannot use the new short-cuts that are default everywhere. Because currently when I press crtl-x or crtl-f, nothing happens, so I assume they are broken. > > I have learned that when I type a short-cut, this should give visual feedback. So when I press crtl-x (or cmd-x on my mac), I would expect a visual aid that tells me what I can delete. class xxx, method yyy, etc. This way I hope I can find my shortcuts again. > > Diego

Feb. 5, 2014

Shortcuts

by Diego Lont

Hi all, I am probably a stupid user, but I cannot use the new short-cuts that are default everywhere. Because currently when I press crtl-x or crtl-f, nothing happens, so I assume they are broken. I have learned that when I type a short-cut, this should give visual feedback. So when I press crtl-x (or cmd-x on my mac), I would expect a visual aid that tells me what I can delete. class xxx, method yyy, etc. This way I hope I can find my shortcuts again. Diego

Feb. 5, 2014

Re: [Pharo-dev] test failures in downloaded image 30733

by Goubier Thierry

Le 05/02/2014 12:05, Usman Bhatti a Ã©crit : > I see and I've been achieving it in another way. > > aClass >> export > fileRef := UITheme builder > fileSave: 'Choose a file' > extensions: #('txt') > path: nil. > fileRef isNil ifFalse: [ > self exportOn: fileRef writeStream > ] > > Instead of testing the method export, I test exportOn: with a stream as > a parameter. > What I am thinking is that, with this approach, we are not testing all > the execution paths in the method export. But then, with a Shim, we will > not be testing that path either. No, it would be different with a Shim: you could test two times this code, one with a nil answer, and another one with a fileReference answer. You would do something like: {nil. 'aPath'} do: [:d | [ aShim setAsUITheme. aShim preload: #fileSave:extension:path: withAnswer: d. aClass export ] ensure: [ aShim reset ] ] And hop, your paths are covered. Thierry -- Thierry Goubier CEA list Laboratoire des Fondations des SystÃ¨mes Temps RÃ©el EmbarquÃ©s 91191 Gif sur Yvette Cedex France Phone/Fax: +33 (0) 1 69 08 32 92 / 83 95

Feb. 5, 2014

Re: [Pharo-dev] test failures in downloaded image 30733

by Usman Bhatti

On Tue, Feb 4, 2014 at 10:01 AM, Goubier Thierry <thierry.goubier(a)cea.fr>wrote: > > > Le 04/02/2014 09:48, Usman Bhatti a Ã©crit : > >> >> >> >> On Tue, Feb 4, 2014 at 9:29 AM, Goubier Thierry <thierry.goubier(a)cea.fr >> <mailto:thierry.goubier@cea.fr>> wrote: >> >> >> >> Le 04/02/2014 09:23, Usman Bhatti a Ã©crit : >> >> >> >> >> Yes, I use that already with Glamour and Roassal, and other morph >> widgets that I have. >> The harder part for me so far is responding to dialog boxes e.g. >> answer >> the user questions automatically and once I have that, I'd be >> able to >> completely test my current set of UIs automatically. >> >> >> What about writing a Shim over the dialog creator with a catalog of >> predefined answers that could be picked by each dialog creation? >> When calling a dialog, we're usually going through a factory >> interface usually. >> >> >> I do not know what is a Shim. >> > > From Wikipedia: > > In computer programming, a shim (from shim) or shiv is a small library > that transparently intercepts an API and changes the parameters passed, > handles the operation itself, or redirects the operation elsewhere. Tx for this info. I was not aware of it and it is surprising for me because half of my doctoral thesis was about AOP and related concepts, and Shim seems to be a very similar concept, in its definition, to AOP. > > > Sometimes, we would like to provide our own answers to the dialogue to >> see if the tests pass in different configurations (e.g. when we have two >> or more checkboxes, to see if we get different results selecting >> different boxes). >> >> Or may be we can start with a simpler solution and then evolve towards a >> completely customizable one. >> > > My simple approach would be : > > in normal code, instead of calling UIManager directly as in: > > UIManager default chooseFrom: aCollection title: aString withCRs. > > use a shim (self class myUIManager ....). In the test case, setup so that: > > `self class myUIManager` returns a shim over UIManager capturing the > chooseFrom:title: and containing a dictionary with 'title' -> value to > choose. You preload the shim with the right answers and the keys for the > shim to find to which dialog this refers (it could be ordered: answer for > first dialog, answer for second dialog, etc...). The shim, instead of > creating the dialog, returns the value it already has (or an error if it > hasn't, which will help you fill the needed values !). > > For stuff not related to dialogs you are using the shim for, you can just > do a passthrough (i.e. UIManager default perform: ) which can even be > handled automagically via doesNotUnderstand:. > > As you can see, there isn't much to it. I see and I've been achieving it in another way. aClass >> export fileRef := UITheme builder fileSave: 'Choose a file' extensions: #('txt') path: nil. fileRef isNil ifFalse: [ self exportOn: fileRef writeStream ] Instead of testing the method export, I test exportOn: with a stream as a parameter. What I am thinking is that, with this approach, we are not testing all the execution paths in the method export. But then, with a Shim, we will not be testing that path either. Usman > > > Thierry > -- > Thierry Goubier > CEA list > Laboratoire des Fondations des SystÃ¨mes Temps RÃ©el EmbarquÃ©s > 91191 Gif sur Yvette Cedex > France > Phone/Fax: +33 (0) 1 69 08 32 92 / 83 95 > >

Feb. 5, 2014

Re: [Pharo-dev] comments :)

by Torsten Bergmann

Still 1017 to go. Feel free to help: Â Â Â Smalltalk allClasses select: [:each | each comment includesSubstring: 'xxxx' ] Â External Package owners should also reintegrate too so these do not get lost when a new version of external packages get integrated. (for instance https://pharo.fogbugz.com/f/cases/12812 for Spec) step by step it goes ... Â Bye T. Â Gesendet:Â Mittwoch, 05. Februar 2014 um 08:36 Uhr Von:Â Pharo4Stef <pharo4Stef(a)free.fr> An:Â "Pharo Development List" <pharo-dev(a)lists.pharo.org> Betreff:Â [Pharo-dev] comments :) Thanks for all these comments! Â On 05 Feb 2014, at 00:27, GitHub <noreply(a)github.com> wrote:Â 30742 12794 Class comments missing in UpdateStreamer package https://pharo.fogbugz.com/f/cases/12794[https://pharo.fogbugz.com/f/cases/1… 12802 Class comments missing for Kernel package https://pharo.fogbugz.com/f/cases/12802[https://pharo.fogbugz.com/f/cases/1… 12804 Class comment missing in AST packages https://pharo.fogbugz.com/f/cases/12804[https://pharo.fogbugz.com/f/cases/1… 12797 Class comments missing for tools package https://pharo.fogbugz.com/f/cases/12797[https://pharo.fogbugz.com/f/cases/1… 12796 Class comments missing for Traits package https://pharo.fogbugz.com/f/cases/12796[https://pharo.fogbugz.com/f/cases/1… 12798 Class comments missing in GroupManager package https://pharo.fogbugz.com/f/cases/12798[https://pharo.fogbugz.com/f/cases/1… 12806 Class comment missing in package FreeType + categorizing for FreeTypeCacheTest https://pharo.fogbugz.com/f/cases/12806[https://pharo.fogbugz.com/f/cases/1… 12800 EyeContextInspector should be renamed to EyeMethodContextInspector https://pharo.fogbugz.com/f/cases/12800[https://pharo.fogbugz.com/f/cases/1…

Feb. 5, 2014

comments :)

by Pharo4Stef

Thanks for all these comments! On 05 Feb 2014, at 00:27, GitHub <noreply(a)github.com> wrote: > 30742 > 12794 Class comments missing in UpdateStreamer package > https://pharo.fogbugz.com/f/cases/12794 > > 12802 Class comments missing for Kernel package > https://pharo.fogbugz.com/f/cases/12802 > > 12804 Class comment missing in AST packages > https://pharo.fogbugz.com/f/cases/12804 > > 12797 Class comments missing for tools package > https://pharo.fogbugz.com/f/cases/12797 > > 12796 Class comments missing for Traits package > https://pharo.fogbugz.com/f/cases/12796 > > 12798 Class comments missing in GroupManager package > https://pharo.fogbugz.com/f/cases/12798 > > 12806 Class comment missing in package FreeType + categorizing for FreeTypeCacheTest > https://pharo.fogbugz.com/f/cases/12806 > > 12800 EyeContextInspector should be renamed to EyeMethodContextInspector > https://pharo.fogbugz.com/f/cases/12800

Feb. 5, 2014

[pharo-project/pharo-core]

by GitHub

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

Feb. 5, 2014

[pharo-project/pharo-core] 58fe61: 30743

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: 58fe6166f72fb444ab6b53016f2c711d26aae014 https://github.com/pharo-project/pharo-core/commit/58fe6166f72fb444ab6b5301… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-02-04 (Tue, 04 Feb 2014) Changed paths: M AsmJit-StackManagement.package/AJCallArgument.class/README.md M AsmJit-StackManagement.package/AJCallCleanup.class/README.md M AsmJit-StackManagement.package/AJCallInfo.class/README.md M AsmJit-StackManagement.package/AJCdeclCallInfo.class/README.md M AsmJit-StackManagement.package/AJReleaseTemps.class/README.md M AsmJit-StackManagement.package/AJRoutineStackManager.class/README.md M AsmJit-StackManagement.package/AJStackInstruction.class/README.md M AsmJit-Tests.package/AJStackAlignmentTests.class/README.md M AsmJit-Tests.package/AJx64AssemblerTests.class/README.md M AsmJit-Tests.package/AJx86AssemblerTests.class/README.md M AsmJit-Tests.package/AJx86RegisterTests.class/README.md R Athens-Cairo.package/AthensCairoStrokePaint.class/instance/library path/nbLibraryNameOrHandle.st R Athens-Cairo.package/CairoFontMetricsProvider.class/instance/library path/nbLibraryNameOrHandle.st R Athens-Cairo.package/CairoFreetypeFontRenderer.class/instance/library path/nbLibraryNameOrHandle.st M ClassOrganizer-Tests.package/ClassOrganizationTest.class/README.md M HelpSystem-Core.package/HelpHowToHelpTopics.class/README.md M HelpSystem-Core.package/HelpHowToHelpTopicsFromCode.class/README.md M HelpSystem-Core.package/PackageAPIHelpBuilder.class/README.md M HelpSystem-Tests.package/ClassAPIHelpBuilderTest.class/README.md M HelpSystem-Tests.package/CustomHelpTest.class/README.md M HelpSystem-Tests.package/HelpBrowserTest.class/README.md M HelpSystem-Tests.package/HelpIconsTest.class/README.md M HelpSystem-Tests.package/HelpTopicListItemWrapperTest.class/README.md M HelpSystem-Tests.package/HelpTopicTest.class/README.md M HelpSystem-Tests.package/WikiStyleHelpBuilderTest.class/README.md M HelpSystem-Tests.package/WikiStyleHelpTest.class/README.md A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script396.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30743.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M System-FileRegistry.package/FileServices.class/README.md Log Message: ----------- 30743 12801 Class comments missing in AsmJIT package https://pharo.fogbugz.com/f/cases/12801 12805 Class comments missing in ClassOrganizer package https://pharo.fogbugz.com/f/cases/12805 12791 Failing test: #testLocalMethodsOfTheClassShouldNotBeRepeatedInItsTraits https://pharo.fogbugz.com/f/cases/12791 12795 Class comments missing for package System-FileRegistry https://pharo.fogbugz.com/f/cases/12795 12799 Class comments missing in Help package https://pharo.fogbugz.com/f/cases/12799 http://files.pharo.org/image/30/30743.zip

Feb. 5, 2014