- Pharo-dev - lists.pharo.org

[pharo-project/pharo-core]
by GitHub Jan. 30, 2016

Jan. 30, 2016

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

1 0

ProcessTest>>testHighPriorityOverridesWaitTime intermittent failures
by Ben Coman Jan. 30, 2016

Jan. 30, 2016

Case 17373 had an erroneous failure from ProcessTest>>testHighPriorityOverridesWaitTime and several others seem also to have suffered... * https://pharo.fogbugz.com/default.asp?17373 * https://pharo.fogbugz.com/default.asp?17496 * https://pharo.fogbugz.com/default.asp?17445 * https://pharo.fogbugz.com/default.asp?7444 These have comments that the problem can't be reproduced, which is common of occasional race conditions. So rather than just try to solve this on the issue tracker, I thought I'd share a tip how to isolate and squash such intermittent failures, essentially by stressing that single test a large number of times like this... 1 to: 100 do: [ :n | Transcript crShow: 'Try ' , n printString. ProcessTest debug: #testHighPriorityOverridesWaitTime. ] Since the debugger only focusses on a point in time snapshot of the running system, some Transcript instrumentation was useful to overview the dynamic behaviour, like this... ProcessTest>>testHighPriorityOverridesWaitTime | lowerPriorityWaitingLonger higherPriorityWaitingLess nextReadyProcess | lowerPriorityWaitingLonger := [ ] forkAt: 11 higherPriorityWaitingLess := [ ] forkAt: 12. nextReadyProcess := Processor nextReadyProcess. Transcript cr; tab;show: (lowerPriorityWaitingLonger priority -> lowerPriorityWaitingLonger identityHash); cr; tab;show: (higherPriorityWaitingLess priority -> higherPriorityWaitingLess identityHash); cr; tab;show: (nextReadyProcess priority -> nextReadyProcess identityHash). self assert: nextReadyProcess equals: higherPriorityWaitingLess. which produces... Try 1 11->343137024 12->40659200 12->40659200 Try 2 11->461794304 12->374996992 12->40659200 Observe on the second try nextReadyProcess refers to same process as from the first try. This can be cleaned up just prior to the asset check by adding ... lowerPriorityWaitingLonger terminate. higherPriorityWaitingLess terminate. which produces... Try 1 11->1005688064 12->833273600 12->833273600 ... Try 100 11->764450816 12->807973888 12->807973888 However this is still a little fragile, since some process with priority greater than the running test could slip in before the call to nextReadyProcess, as can be seen by pushing the #to:do: loop out to 100000, which produces... Try 32044 11->784361216 12->431928576 40->397533952 or wrapping it by [ ]forkAt:39, which produces... Try 1 11->431897088 12->400972288 30->338363648 After several awkward failed attempts to fix this, it seemed best to observe actual behaviour rather than peeking into the dynamic data structures. Thus I propose... ProcessTest>>testHighPriorityOverridesWaitTime "The first process to resume will pass straight through the gate while the other waits for the assert to whichRan." | gate checkAssert priorityRan | gate := Semaphore new signal. checkAssert := Semaphore new. [ gate wait. priorityRan := 11. checkAssert signal ] forkAt: 11. [ gate wait. priorityRan := 12. checkAssert signal ] forkAt: 12. checkAssert wait. self assert: priorityRan=12 description: 'Second scheduled but higher priority should run first'. gate signal. checkAssert wait. self assert: whichRan=11 description: 'First scheduled but lower priority should run after'. https://pharo.fogbugz.com/default.asp?17501

2 1

GTDebugger feedback - Setup/TearDown tabs for test cases
by Ben Coman Jan. 30, 2016

Jan. 30, 2016

I really like the SetUp/TearDown tabs when debugging a failed test, however it seems a waste to devote a whole half of window to them causing lines in the Source tab to wrap unnecessarily. Could they be moved adjacent to the Source tab? I guess the Browse button would equally apply to #setUp and #tearDown methods. Also, you can edit the SetUp code but changes don't seem to be saved through. So can this either be fixed (including when the method doesn't exist) or there be some hint it can't be saved: * tab says [SetUp (read-only)] * in an fake comment above its code put "This method is read only" (maybe in light grey) * Instead of the yellow/orange "modified" triangle appearing in the top-right of the pane, make it another colour - maybe black to mean you can't save - I think red is reserved for meaning the pane was edited elsewhere? cheers -ben P.S. I'd like to add my voice to those commenting that the stepping buttons are too far away. Glancing back and forth between them and the code I feel like I'm watching a tennis game. P.P.S The icons on those buttons are real nice.

1 0

Use cases for methods with optional parameters
by Damien Cassou Jan. 30, 2016

Jan. 30, 2016

Hi, I would like to study the impact of adding optional parameters to keyword methods in Pharo. The goal of optional parameters is to facilitate the implementation of methods where some parameters are optional. For example, Seaside has: WAComponent>>request: aRequestString label: aLabelString default: aDefaultString onAnswer: aBlock ... This method of 4 arguments has only 1 required argument (aRequestString) and 3 optional ones. In the current implementation, this results in 7 additional methods that only delegate directly or indirectly to the one above: - request: - request:default: - request:default:onAnswer: - request:label: - request:label:default: - request:label:onAnswer: - request:onAnswer: Before starting to implement anything, I need to know if it makes sense. If you want to help me, please send me all the use cases you have by giving me: - the library name (and URL if not in Catalog), or "pharo" if it's in Pharo - the class where this happens - the range of selectors that make sense For the example above, this would be: - Seaside - WAComponent - request:* Thank you very much -- Damien Cassou http://damiencassou.seasidehosting.st "Success is the ability to go from one failure to another without losing enthusiasm." --Winston Churchill

6 7

[pharo-project/pharo-core] 075ec6: 50550
by GitHub Jan. 29, 2016

Jan. 29, 2016

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 075ec66df6ea08e7be71d5275b08ca095432aa2b https://github.com/pharo-project/pharo-core/commit/075ec66df6ea08e7be71d527… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-01-27 (Wed, 27 Jan 2016) Changed paths: M Alien.package/Alien.class/class/class initialization/ensureInSpecialObjectsArray.st M Alien.package/Alien.class/class/class initialization/initialize.st M Alien.package/Alien.class/class/examples/libcName.st M Alien.package/Alien.class/class/instance creation/atAddress_.st M Alien.package/Alien.class/class/instance creation/atAddress_dataSize_.st M Alien.package/Alien.class/class/instance creation/forPointer_.st M Alien.package/Alien.class/class/instance creation/newC_.st M Alien.package/Alien.class/class/instance creation/newGC_.st M Alien.package/Alien.class/class/instance creation/new_.st M Alien.package/Alien.class/class/instance creation/rawNewC_.st M Alien.package/Alien.class/class/system startup/startUp_.st M Alien.package/Alien.class/definition.st R Alien.package/Alien.class/instance/as yet unclassified/isPointer.st M Alien.package/Alien.class/instance/printing/storeOn_.st A Alien.package/Alien.class/instance/testing/isPointer.st M Alien.package/AlienSunit.class/instance/testing/testLongLong.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/README.md R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/accessing/project.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/catalog/catalogContactInfo.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/catalog/catalogDescription.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/catalog/catalogKeywords.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/loading/load.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/metacello tool support/isMetacelloConfig.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/metacello tool support/lastMetacelloVersionLoad.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/metacello tool support/metacelloVersion_loads_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/private/bootstrapPackage_from_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/class/private/ensureMetacello.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/definition.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/accessing/project.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/baselines/baseline10_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/baselines/baseline12_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/baselines/baseline13_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/baselines/baseline14_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/baselines/baseline15_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/doits/ffiExampleInitialization10.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/symbolic versions/development_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/symbolic versions/stable_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version1%5F10%5F1_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version1%5F10_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version1%5F9%5F1_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version10_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version11_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version12_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version13_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version14_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version15_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version16_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version17_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version18_.st R ConfigurationOfFFI.package/ConfigurationOfFFI.class/instance/versions/version19_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/README.md R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/accessing/project.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/catalog/catalogContactInfo.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/catalog/catalogDescription.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/catalog/catalogKeywords.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/development support/DevelopmentSupport.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/development support/validate.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/loading/load.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/loading/loadBleedingEdge.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/loading/loadDevelopment.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/metacello tool support/isMetacelloConfig.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/private/baseConfigurationClassIfAbsent_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/private/ensureMetacello.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/class/private/ensureMetacelloBaseConfiguration.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/definition.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/accessing/project.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/baselines/baseline10_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/symbolic versions/stable_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/versions/version1%5F2_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/versions/version10_.st R ConfigurationOfOldAlien.package/ConfigurationOfOldAlien.class/instance/versions/version11_.st A ConfigurationOfUnifiedFFI.package/ConfigurationOfUnifiedFFI.class/instance/baselines/baseline0%5F11_.st M ConfigurationOfUnifiedFFI.package/ConfigurationOfUnifiedFFI.class/instance/symbolic versions/development_.st M ConfigurationOfUnifiedFFI.package/ConfigurationOfUnifiedFFI.class/instance/symbolic versions/stable_.st A ConfigurationOfUnifiedFFI.package/ConfigurationOfUnifiedFFI.class/instance/versions/v0%5F11%5F2_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50549.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50550.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50549.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50550.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50550 17482 load UnifiedFFI 0.11.2 https://pharo.fogbugz.com/f/cases/17482 17479 Interval class comment mixes step/stop when explaining Instance Variables https://pharo.fogbugz.com/f/cases/17479 http://files.pharo.org/image/50/50550.zip

2 1

[pharo-project/pharo-core]
by GitHub Jan. 29, 2016

Jan. 29, 2016

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

1 0

[pharo-project/pharo-core] 03f485: 50559
by GitHub Jan. 29, 2016

Jan. 29, 2016

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: 03f48517e360f34d4c97b75a5f3a77f32988a4de https://github.com/pharo-project/pharo-core/commit/03f48517e360f34d4c97b75a… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-01-29 (Fri, 29 Jan 2016) Changed paths: R DebuggerActions.package/PeelToFirstDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/RestartDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/ResumeDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/ReturnValueDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/RunToSelectionDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/StepIntoDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/StepOverDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerActions.package/StepThroughDebugAction.class/instance/testing/appliesToDebugger_.st R DebuggerModel.package/DebugSession.class/instance/testing/isInterruptedContextPostMortem.st R GT-BytecodeDebugger.package/GTStepToBytecodeDebugAction.class/instance/testing/appliesToDebugger_.st R Glamour-Tests-Morphic.package/GLMMorphicExamplesTest.class/instance/tests/testSmoke.st M Nautilus.package/Nautilus.class/instance/accessing/selectedPackage_.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50558.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50559.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50558.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50559.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50559 17489 remove testSmoke https://pharo.fogbugz.com/f/cases/17489 17463 NautilusUI>>#updatePackages MessageNotUnderstood: RPackage>>item https://pharo.fogbugz.com/f/cases/17463 17459 #isInterruptedContextPostMortem should not be used https://pharo.fogbugz.com/f/cases/17459 http://files.pharo.org/image/50/50559.zip

1 0

[pharo-project/pharo-core]
by GitHub Jan. 29, 2016

Jan. 29, 2016

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

1 0

[pharo-project/pharo-core] c99e24: 50558
by GitHub Jan. 29, 2016

Jan. 29, 2016

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: c99e24992162ba83eaacf95176c92ec9338c70c6 https://github.com/pharo-project/pharo-core/commit/c99e24992162ba83eaacf951… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-01-29 (Fri, 29 Jan 2016) Changed paths: R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50557.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50558.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50557.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50558.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50558 16735 Replace Startup/Shutdown list with SessionManager https://pharo.fogbugz.com/f/cases/16735 http://files.pharo.org/image/50/50558.zip

1 0

Color yzw
by stepharo Jan. 29, 2016

Jan. 29, 2016

Hi In the past in Pharo when a message was not matching an existing selector the selector was turned into red So we could see immediately that you typed an error. Right now, the behavior only exists for Class so Po is italiczec Pox is red I played with the blueStyleTable but I cannot get it working on selector. Does anybody know? Such kind of glitch is slowing me when shooting new pharo videos. Stef

7 20