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

[bloc] feature envy -> move method close to data?
by stepharo Feb. 26, 2016

Feb. 26, 2016

Hi Blockers I do not really understand why drawOnAthensCanvas: is not defined on BlShape (especially when we see how many self space are defined in this method. And this is one of the few things I thought I understood from bloc (grouping the rendering in a separate object). drawOnAthensCanvas: aCanvas "Actually render receiver on aCanvas in local bounds. Override to customize. aCanvas is an instance of AthensCanvas aCanvas must not be nil" | pathTransform | pathTransform := aCanvas pathTransform. "First we fill" pathTransform restoreAfter: [ | path fillExtent fillTranslation | "We fill inside natural element's bounds" fillExtent := self shape path fillExtentFor: self shape inBounds: self localBounds. path := self shape path pathOn: aCanvas forExtent: fillExtent. "Translate fill if needed" fillTranslation := self shape path fillTranslationFor: self shape inBounds: self localBounds. pathTransform translateBy: fillTranslation. aCanvas setPaint: self shape fillPaint; drawShape: path ]. pathTransform restoreAfter: [ | path strokeExtent strokeTranslation | "We stroke inside natural element's bounds" strokeExtent := self shape path strokeExtentFor: self shape inBounds: self localBounds. path := self shape path pathOn: aCanvas forExtent: strokeExtent. "Translate stroke if needed" strokeTranslation := self shape path strokeTranslationFor: self shape inBounds: self localBounds. pathTransform translateBy: strokeTranslation. aCanvas setPaint: self shape strokePaint; drawShape: path ]

4 7

[pharo-project/pharo-core] c2b19a: 50611
by GitHub Feb. 25, 2016

Feb. 25, 2016

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: c2b19a15475f784e06cbcf617d7e304632e16de4 https://github.com/pharo-project/pharo-core/commit/c2b19a15475f784e06cbcf61… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-02-25 (Thu, 25 Feb 2016) Changed paths: M AST-Tests-Core.package/NumberParserTest.class/instance/tests - Float/testIntegerWithNegExponentIsAFloat.st A ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/baselines/baseline22_.st M ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/symbolic versions/development_.st M ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/symbolic versions/stable_.st A ConfigurationOfRubric.package/ConfigurationOfRubric.class/instance/versions/version28_.st A Kernel-Rules.package/FloatReferencesRule.class/README.md A Kernel-Rules.package/FloatReferencesRule.class/definition.st A Kernel-Rules.package/FloatReferencesRule.class/instance/accessing/group.st A Kernel-Rules.package/FloatReferencesRule.class/instance/accessing/name.st A Kernel-Rules.package/FloatReferencesRule.class/instance/accessing/rationale.st A Kernel-Rules.package/FloatReferencesRule.class/instance/running/basicCheck_.st A Kernel-Rules.package/FloatReferencesRule.class/instance/running/check_forCriticsDo_ifNone_.st A Kernel-Rules.package/FloatReferencesRule.class/instance/running/systemClassNames.st M NECompletion-Tests.package/NECContextTest.class/instance/tests/testReceiverConstant.st A Rubric-Rules.package/RbScriptingSetBeforeModel.class/README.md A Rubric-Rules.package/RbScriptingSetBeforeModel.class/definition.st A Rubric-Rules.package/RbScriptingSetBeforeModel.class/instance/accessing/group.st A Rubric-Rules.package/RbScriptingSetBeforeModel.class/instance/accessing/name.st A Rubric-Rules.package/RbScriptingSetBeforeModel.class/instance/running/basicCheck_.st A Rubric-Rules.package/RbScriptingSetBeforeModel.class/instance/running/check_forCriticsDo_ifNone_.st R Rubric.package/RubBasicSelectionDecorator.class/README.md R Rubric.package/RubBasicSelectionDecorator.class/class/querying/key.st R Rubric.package/RubBasicSelectionDecorator.class/definition.st R Rubric.package/RubBasicSelectionDecorator.class/instance/drawing/draw.st R Rubric.package/RubBasicSelectionDecorator.class/instance/drawing/drawOnAthensCanvas.st R Rubric.package/RubBasicSelectionDecorator.class/instance/drawing/handleDrawingOnAthensCanvas_bounds_.st R Rubric.package/RubBasicSelectionDecorator.class/instance/drawing/handleDrawingOn_using_at_.st M Rubric.package/RubCurrentLineBarDecorator.class/definition.st A Rubric.package/RubCurrentLineBarDecorator.class/instance/drawing/drawOnAthensCanvas_.st A Rubric.package/RubCurrentLineBarDecorator.class/instance/drawing/drawOn_.st M Rubric.package/RubExtraSelectionDecorator.class/definition.st M Rubric.package/RubExtraSelectionDecorator.class/instance/accessing/findRegex.st M Rubric.package/RubExtraSelectionDecorator.class/instance/accessing/primarySelectionInterval.st A Rubric.package/RubExtraSelectionDecorator.class/instance/computation/cleanUpText.st A Rubric.package/RubExtraSelectionDecorator.class/instance/computation/computeFindReplaceShapesFrom_to_.st R Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/cleanUpText.st R Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/computeFindReplaceShapesFrom_to_.st M Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/computeSecondarySelectionShapesFrom_to_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/computeSelectionShapesFor_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawAllSelectionShapesOnAthensCanvas_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawAllSelectionShapesOn_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawOnAthensCanvas_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawOn_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawSelectionShape_onAthensCanvas_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/drawing/drawSelectionShape_on_.st A Rubric.package/RubExtraSelectionDecorator.class/instance/testing/hasValidSelection.st A Rubric.package/RubOpeningClosingDelimiterDecorator.class/instance/drawing/drawOnAthensCanvas_.st A Rubric.package/RubOpeningClosingDelimiterDecorator.class/instance/drawing/drawOn_.st A Rubric.package/RubOpeningClosingDelimiterDecorator.class/instance/drawing/positionOfOppositeDelimiter_startingAt_in_.st M Rubric.package/RubParagraphDecorator.class/definition.st A Rubric.package/RubParagraphDecorator.class/instance/drawing/canDrawDecoratorsOn_.st A Rubric.package/RubParagraphDecorator.class/instance/drawing/drawOnAthensCanvas_.st M Rubric.package/RubParagraphDecorator.class/instance/drawing/drawOnAthensCanvas_bounds_color_.st A Rubric.package/RubParagraphDecorator.class/instance/drawing/drawOn_.st M Rubric.package/RubParagraphDecorator.class/instance/drawing/drawOn_using_at_.st A Rubric.package/RubParagraphDecorator.class/instance/instead of DNU/paragraph.st A Rubric.package/RubParagraphDecorator.class/instance/instead of DNU/text.st A Rubric.package/RubSmalltalkCodeMode.class/instance/formatting/maxFormatterLineLength.st M Rubric.package/RubSmalltalkCodeMode.class/instance/initialize-release/formatMethodCode.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50610.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50611.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50610.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50611.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Tools-Test.package/MessageTallyTest.class/instance/tallying/testTallySends.st Log Message: ----------- 50611 17686 Add rules about Rubric method precedence https://pharo.fogbugz.com/f/cases/17686 17689 BoxedFloat Fix remaining Tests https://pharo.fogbugz.com/f/cases/17689 17685 Add rules to check if Float system classes are referenced https://pharo.fogbugz.com/f/cases/17685 http://files.pharo.org/image/50/50611.zip

4 3

another difference since spur migration
by Nicolai Hess Feb. 25, 2016

Feb. 25, 2016

compare the bytecode of this code |outerContext local1 remote1 | outerContext := thisContext. local1 := 1. remote1 := 3. (1 to:1 ) do:[:index| | rlocal2 rlocal3 | rlocal2 := index. remote1:= rlocal2 / 3. rlocal3 := remote1. rlocal3 := outerContext]. local1 := remote1. thisContext method inspect. between pharo 50496 and pharo 50497 there is a minor difference for the numbering of temps for copying to the closure block and accessing indirect tempvar vector. I could not find the code change that may be responsible for this change. But maybe this is the reason why this test fails: 17702 failting test: OCClosureCompilerTest>>#testDebuggerTempAccess nicolai

1 0

[Bloc] MDLColor and new gradient
by stepharo Feb. 25, 2016

Feb. 25, 2016

Hi Guillaume Larcheveque in Telescope added support for MDLColor http://www.materialui.co/colors and nice gradient building. I asked him if we could extract this functionality and move it into Pharo because it will be usefull for Pharo, Bloc, and Roassal. A first step could be to have MDLColor subclass as it is now but the second step would be to rethink Color and palette. May be we should have some Palette And we could have (brainstorming) (ColorBuilder from: MDLPalette) -> aColorBuilder aColorBuilder red -> .... Stef

5 7

Is there a plan to have refreshing values in debugger?
by stepharo Feb. 25, 2016

Feb. 25, 2016

Hi Is there a plan to have refreshed values (instance variables) in debugger and inspector? This is super annoying to have to refresh the complete list in the debugger to see new values of instance variables. Why we cannot put back the old pulling behavior and in the future use reflectivity? Stef

3 3

ZTimestamp improvement
by Blondeau Vincent Feb. 25, 2016

Feb. 25, 2016

Hello, I use Ztimestamp to parse a date but the time zone offset is wrongly set: (DateAndTime readTimezoneOffsetFrom: '+0530' readStream) printString gives '0:05:00:00' instead of '0:05:30:00'. See https://pharo.fogbugz.com/f/cases/17693/Time-offset-parsing-is-false for the DateAndTime issue. As the bug is in DateAndTime>>readOptionalSeparatorFrom: and ZTimestamp reuses the same code, I propose to apply the same patch to it. Can I have the commit access to the repo (http://smalltalkhub.com/#!/~SvenVanCaekenberghe/Neo) to submit the patch? Cheers, Vincent ________________________________ Ce message et les pi?ces jointes sont confidentiels et r?serv?s ? l'usage exclusif de ses destinataires. Il peut ?galement ?tre prot?g? par le secret professionnel. Si vous recevez ce message par erreur, merci d'en avertir imm?diatement l'exp?diteur et de le d?truire. L'int?grit? du message ne pouvant ?tre assur?e sur Internet, la responsabilit? de Worldline ne pourra ?tre recherch?e quant au contenu de ce message. Bien que les meilleurs efforts soient faits pour maintenir cette transmission exempte de tout virus, l'exp?diteur ne donne aucune garantie ? cet ?gard et sa responsabilit? ne saurait ?tre recherch?e pour tout dommage r?sultant d'un virus transmis. This e-mail and the documents attached are confidential and intended solely for the addressee; it may also be privileged. If you receive this e-mail in error, please notify the sender immediately and destroy it. As its integrity cannot be secured on the Internet, the Worldline liability cannot be triggered for the message content. Although the sender endeavours to maintain a computer virus-free network, the sender does not warrant that this transmission is virus-free and will not be liable for any damages resulting from any virus transmitted.

3 4

Process>>restart
by Ben Coman Feb. 25, 2016

Feb. 25, 2016

Just sharing a passing thought so I don't forget it before I have time to think more deeply on it. Several parts of Pharo (like SHTextStyler) are designed to have a background process running, and when new input arrives that process is killed so another can be immediately started - and sometimes we seem to end up with large amounts of zombie processes. As I poke around Process>>terminate, I wonder if a different approach would be to have a Process>>restart. I see there is Context>>restart. The starting the background process might look like... stylingProcess := [ [ "do stuff with nextData". self suspend ] repeat ] forkAt: 30. and the code in the main morphic priority 40 thread might look like... sylingProcess suspend. nextData := 'blah'. sylingProcess restart. anyway, back to the grind for me, cheers -ben

3 4

debugger versus primitive 19 "simulation guard"
by Ben Coman Feb. 25, 2016

Feb. 25, 2016

Trying to debug step two times into... [ 'a' ] newProcess ...hangs the image. It works okay with... Pharo5.0 update: #50495 NBCoInterpreter NativeBoost-CogPlugin-EstebanLorenzano.21 uuid: 4d9b9bdf-2dfa-4c0b-99eb-5b110dadc697 Apr 2 2015 By: Esteban Lorenzano <estebanlm(a)gmail.com> Jenkins build #14904 Unix built on Apr 2 2015 00:51:53 Compiler: 4.6.3 but not with... Pharo5.0 update: #50496 CoInterpreter VMMaker.oscog-eem.1630 uuid: 2ed025ea-f400-4440-8e8b-5aa46d06c9ab Jan 14 2016 21ec004cce7d26010c18d357c805a0e1a4ffe376 Date: 2016-01-14 11:42:33 +0100 By: Esteban Lorenzano <estebanlm(a)gmail.com> Jenkins build #498 Unix built on Jan 14 2016 11:50:34 Compiler: 4.6.3 btw, the same case on Squeak-4.6-All-In-one and Squeak-5-All-In-One both produce "Code simulation error." Is it maybe related to... "another endless debugger loop" https://pharo.fogbugz.com/default.asp?16877 cheers -ben

2 1

[pharo-project/pharo-core] f90141: 50613
by GitHub Feb. 25, 2016

Feb. 25, 2016

Branch: refs/heads/5.0 Home: https://github.com/pharo-project/pharo-core Commit: f9014105d5211bb4d78aabd1278956e126bb00c3 https://github.com/pharo-project/pharo-core/commit/f9014105d5211bb4d78aabd1… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-02-25 (Thu, 25 Feb 2016) Changed paths: A Deprecated50.package/NBExternalAddress.class/README.md A Deprecated50.package/NBExternalAddress.class/definition.st A Deprecated50.package/NBExternalArray.class/README.md A Deprecated50.package/NBExternalArray.class/definition.st A Deprecated50.package/NBExternalEnumeration.class/README.md A Deprecated50.package/NBExternalEnumeration.class/definition.st A Deprecated50.package/NBExternalObject.class/README.md A Deprecated50.package/NBExternalObject.class/definition.st A Deprecated50.package/NBExternalStructure.class/README.md A Deprecated50.package/NBExternalStructure.class/definition.st A Deprecated50.package/NBExternalStructureFields.class/README.md A Deprecated50.package/NBExternalStructureFields.class/definition.st A Deprecated50.package/NBExternalTypeValue.class/README.md A Deprecated50.package/NBExternalTypeValue.class/definition.st A Deprecated50.package/NBExternalUnion.class/README.md A Deprecated50.package/NBExternalUnion.class/definition.st A Deprecated50.package/NBFFICallback.class/README.md A Deprecated50.package/NBFFICallback.class/definition.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50612.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - scripts/script50613.st R ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50612.st A ScriptLoader50.package/ScriptLoader.class/instance/pharo - updates/update50613.st M ScriptLoader50.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 50613 17700 add stub NB classes to easy migrations https://pharo.fogbugz.com/f/cases/17700 http://files.pharo.org/image/50/50613.zip

1 0

[pharo-project/pharo-core]
by GitHub Feb. 25, 2016

Feb. 25, 2016

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

1 0