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

[pharo-project/pharo-core] 164590: 60208

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: 164590c10b2d7f32b006db92dcad5c2134dedcf0 https://github.com/pharo-project/pharo-core/commit/164590c10b2d7f32b006db92… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-09-07 (Wed, 07 Sep 2016) Changed paths: M ConfigurationOfFlatQA.package/ConfigurationOfFlatQA.class/instance/versions/versionv3%5F2%5F7_.st M Renraku.package/ReAbstractCritique.class/instance/gt-extension/gtInspectorSourceCodeIn_.st A Renraku.package/ReIntervalSourceAnchor.class/instance/gt-inspector-extension/gtInspectorAnchorIn_.st R Renraku.package/ReIntervalSourceAnchor.class/instance/gt-inspector-extension/gtInspectorSourceCodeIn_.st M Renraku.package/ReSearchStringSourceAnchor.class/instance/accessing/interval.st A Renraku.package/ReSearchStringSourceAnchor.class/instance/gt-inspector-extension/gtInspectorAnchorIn_.st R Renraku.package/ReSearchStringSourceAnchor.class/instance/gt-inspector-extension/gtInspectorSourceCodeIn_.st A Renraku.package/ReSourceAnchor.class/instance/gt-inspector-extension/gtInspectorAnchorIn_.st R Renraku.package/ReSourceAnchor.class/instance/gt-inspector-extension/gtInspectorSourceCodeIn_.st A Renraku.package/ReVarSearchSourceAnchor.class/README.md A Renraku.package/ReVarSearchSourceAnchor.class/definition.st A Renraku.package/ReVarSearchSourceAnchor.class/instance/as yet unclassified/interval.st A Renraku.package/ReVarSearchSourceAnchor.class/instance/as yet unclassified/searchString.st A Renraku.package/extension/CompiledMethod/instance/glmCritiqueSourceIn_.st R Renraku.package/extension/CompiledMethod/instance/glmSourcePresentationIn_.st A Renraku.package/extension/Object/instance/glmCritiqueSourceIn_.st A Renraku.package/extension/RBProgramNode/instance/glmCritiqueSourceIn_.st R Renraku.package/extension/RBProgramNode/instance/glmSourcePresentationIn_.st A Renraku.package/extension/TBehavior/instance/glmCritiqueSourceIn_.st R Renraku.package/extension/TBehavior/instance/glmSourcePresentationIn_.st M Rubric-Styling.package/RubShoutStylerDecorator.class/instance/editing/style_.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60207.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60208.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60207.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60208.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 60208 19052 rub shoutdecorator should not raise a debugger https://pharo.fogbugz.com/f/cases/19052 19054 QA 3.2.7 https://pharo.fogbugz.com/f/cases/19054 http://files.pharo.org/image/60/60208.zip

Sept. 7, 2016

Re: [Pharo-dev] FogBugz (Case [Issue]19057) CI and Bugtracker - Failing test: SHParserST80Test.testNumberHierarchy

by Marcus Denker

> On 07 Sep 2016, at 13:37, Nicolai Hess <nicolaihess(a)gmail.com> wrote: > > Fraction>>round: numberOfWishedDecimal > "Round the decimal part of the receiver to be limited to the number of wished decimal. Only leave a fixed amount of decimal" > < expr: (1/3 round: 2) result: 0.33 > > < expr: (111/100 round: 2) result: 1.11 > > > ^self asFloat round: numberOfWishedDecimal > > > this method was changed for case 19034 <https://pharo.fogbugz.com/f/cases/19034/Improve-comment-of-round> Improve comment of round: > upsâ¦ I did not review correctly. I thought that only the comment changed. Pragmas are not comments... > But I think > < expr: (111/100 round: 2) result: 1.11 > > is actually not a valid pragma expression. (Squeak compiler does not allow this). > > 1. Do we want to allow this syntax ( I don't want) No! This would make the system quite unreadable if you add lots of those everywhere. Marcus

Sept. 7, 2016

Re: [Pharo-dev] [Ann] Remote debugging tools

by Denis Kudriashov

Hi Norbert. 2016-07-29 10:45 GMT+02:00 Norbert Hartl <norbert(a)hartl.name>: > > I start implement optimized protocol and I got really nice results. I > reduce size of packet in my example to 16 bytes. I not measure performance > yet but it should be better too. Also new protocol will allow application > specific optimisations. > So soon we will have really usable tools for your case with slow network. > > > That sounds wonderful! Thank you very much! Is the new code online? > So you can try new version now

Sept. 7, 2016

Fwd: FogBugz (Case [Issue]19057) CI and Bugtracker - Failing test: SHParserST80Test.testNumberHierarchy

by Nicolai Hess

Fraction>>round: numberOfWishedDecimal "Round the decimal part of the receiver to be limited to the number of wished decimal. Only leave a fixed amount of decimal" < expr: (1/3 round: 2) result: 0.33 > < expr: (111/100 round: 2) result: 1.11 > ^self asFloat round: numberOfWishedDecimal this method was changed for case 19034 <https://pharo.fogbugz.com/f/cases/19034/Improve-comment-of-round> Improve comment of round: But I think < expr: (111/100 round: 2) result: 1.11 > is actually not a valid pragma expression. (Squeak compiler does not allow this). 1. Do we want to allow this syntax ( I don't want) 2. how to express this "example-pragma", use literal arrays instead: < expr: #(1/3 round: 2) result: 0.33 > or <expr: #(1 #/ 3 #round: 2) result: 0.33> nicolai ---------- Forwarded message ---------- From: Pharo Issue Tracker <do-not-reply(a)pharo.fogbugz.com> Date: 2016-09-07 10:45 GMT+02:00 Subject: FogBugz (Case [Issue]19057) CI and Bugtracker - Failing test: SHParserST80Test.testNumberHierarchy To: "nicolaihess(a)web.de" <nicolaihess(a)web.de> [image: avatar] *Marcus Denker* opened *Case 19057*: Failing test: SHParserST80Test. testNumberHierarchy <https://pharo.fogbugz.com/f/cases/19057> and assigned it to *Everyone*: *Bug* in [image: Project:] CI and Bugtracker: 1. Pharo Image â¢ You are subscribed to this case testNumberHierarchy is Failing [image: Priority] *Priority*: 5 â Fix If Time [image: Status] *Status*: Work Needed [image: Assigned To] *Assigned to*: Everyone [image: Milestone] *Milestone*: Pharo6.0 Go to Case <https://pharo.fogbugz.com/f/cases/19057> No longer need updates? Unsubscribe from this case. <https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comma…> Don't want FogBugz notifications anymore? Update your preferences. <https://pharo.fogbugz.com/f/userPrefs> [image: FogBugz] A FogBugz case was opened by Marcus Denker. Case ID: 19057 Title: Failing test: SHParserST80Test.testNumberHierarchy Status: Work Needed Category: Bug Project: CI and Bugtracker Area: 1. Pharo Image Priority: 5 - Fix If Time Milestone: Pharo6.0: 3/31/2017 Assigned To: Everyone URL: https://pharo.fogbugz.com/f/cases/19057 testNumberHierarchy is Failing You are subscribed to this case. If you do not want to receive automatic notifications in the future, unsubscribe (https://pharo.fogbugz.com/default.asp?pre=preUnsubscribe&pg=pgEditBug&comma…) from this case.

Sept. 7, 2016

Planning: Pharo Sprints Lille for the rest 2016

by Marcus Denker

Hi Here locally we try to organise one âSprintâ per month were we meet to work on boring issue tracker entries together. (external visitors are welcome!) In the past people organised local sprints at other places at the sane time (e.g. Santiago/Chile). The next dates are: 30 September 28 October 25 November 16 December. Pharo Christmas Party! Marcus

Sept. 7, 2016

Re: [Pharo-dev] SmallLintRule RBGuardingClauseRule and RBGuardClauseRule

by Yuriy Tymchuk

Hi, Iâm moving rules anyway. I will just remove those two and create one that highlights code and transforms. Uko > On 07 Sep 2016, at 10:33, Marcus Denker <marcus.denker(a)inria.fr> wrote: > >> >> On 05 Sep 2016, at 09:15, Nicolai Hess <nicolaihess(a)gmail.com> wrote: >> >> We have to similar rules >> RBGuardClauseRule and RBGuardingClauseRule >> One is a ParseTreeLintRule, the other one is a TransformationRule. >> >> I don't know why we have to at all. >> One rule was "disabled" (or not chown) in the CriticsBrowser because it >> was removed from the set of all rules before opening CriticsBrowser >> (see method #allGoodRules). >> >> In the past, monty fixed the class comment and #rational for class >> RBGuardClauseRule to correctly explain what this rule is about. >> This of course now differes from the similar rule RBGuardingClauseRule. >> >> 1. Why do we have both (ok, on can only select the rule violation, whereas the other on >> does not select it but shows the possible transformation >> 2. why is the second one now in the set of all rules used by CriticsBrowser >> 3. what to do ? Remove one, fix the comment and #rational for the second one? >> > > Yes, we should clean upâ¦ I would remove one. > > Marcus

Sept. 7, 2016

Re: [Pharo-dev] OCCompilerASTPlugin

by Marcus Denker

> On 04 Sep 2016, at 20:48, Serge Stinckwich <serge.stinckwich(a)gmail.com> wrote: > > I have one question about Opal Compiler. > > I would like do an AST transformation before using the compiler. If I > understand correctly, I have to built a subclass of > OCCompilerASTPlugin ? Is it correct ? Yes. See the example ASTPluginMeaningOfLife Plugins are not active by default, you need to compile with a compiler that includes your plugin. e.g. add on the class side compiler "the example plugin is active for this class" ^super compiler addPlugin: ASTPluginMeaningOfLife. to enable it for this hierarchy. ( see ASTTransformExamplePluginActive ) > Any documentation more update about Opal than this : > https://github.com/SquareBracketAssociates/PharoLimbo/tree/master/Opal There is the paper: https://marcusdenker.de/publications/Bera13a-OpalIWST.pdf But it is not a tutorial. Marcus

Sept. 7, 2016

Re: [Pharo-dev] SmallLintRule RBGuardingClauseRule and RBGuardClauseRule

by Marcus Denker

> On 05 Sep 2016, at 09:15, Nicolai Hess <nicolaihess(a)gmail.com> wrote: > > We have to similar rules > RBGuardClauseRule and RBGuardingClauseRule > One is a ParseTreeLintRule, the other one is a TransformationRule. > > I don't know why we have to at all. > One rule was "disabled" (or not chown) in the CriticsBrowser because it > was removed from the set of all rules before opening CriticsBrowser > (see method #allGoodRules). > > In the past, monty fixed the class comment and #rational for class > RBGuardClauseRule to correctly explain what this rule is about. > This of course now differes from the similar rule RBGuardingClauseRule. > > 1. Why do we have both (ok, on can only select the rule violation, whereas the other on > does not select it but shows the possible transformation > 2. why is the second one now in the set of all rules used by CriticsBrowser > 3. what to do ? Remove one, fix the comment and #rational for the second one? > Yes, we should clean upâ¦ I would remove one. Marcus

Sept. 7, 2016

[pharo-project/pharo-core]

by GitHub

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

Sept. 7, 2016

[pharo-project/pharo-core] 5917ae: 60207

by GitHub

Branch: refs/heads/6.0 Home: https://github.com/pharo-project/pharo-core Commit: 5917aebcf7d2b406f2ed921c967f85c613748ad9 https://github.com/pharo-project/pharo-core/commit/5917aebcf7d2b406f2ed921c… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2016-09-07 (Wed, 07 Sep 2016) Changed paths: M ConfigurationOfFlatQA.package/ConfigurationOfFlatQA.class/instance/symbolic versions/stable_.st A ConfigurationOfFlatQA.package/ConfigurationOfFlatQA.class/instance/versions/versionv3%5F2%5F7_.st M DebuggerActions.package/BrowseDebugAction.class/class/actions creation/debugActionsFor_.st M DebuggerActions.package/DebugAction.class/definition.st M DebuggerActions.package/DebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/DebugAction.class/instance/accessing/defaultKeymap.st M DebuggerActions.package/DebugAction.class/instance/accessing/keyText.st M DebuggerActions.package/DebugAction.class/instance/accessing/keyText_.st A DebuggerActions.package/DebugAction.class/instance/accessing/keymap.st A DebuggerActions.package/DebugAction.class/instance/accessing/keymap_.st R DebuggerActions.package/RestartDebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/RestartDebugAction.class/instance/accessing/defaultKeymap.st R DebuggerActions.package/ResumeDebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/ResumeDebugAction.class/instance/accessing/defaultKeymap.st R DebuggerActions.package/StepIntoDebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/StepIntoDebugAction.class/instance/accessing/defaultKeymap.st R DebuggerActions.package/StepOverDebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/StepOverDebugAction.class/instance/accessing/defaultKeymap.st R DebuggerActions.package/StepThroughDebugAction.class/instance/accessing/defaultKeyText.st A DebuggerActions.package/StepThroughDebugAction.class/instance/accessing/defaultKeymap.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60206.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - scripts/script60207.st R ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60206.st A ScriptLoader60.package/ScriptLoader.class/instance/pharo - updates/update60207.st M ScriptLoader60.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Spec-Debugger.package/SpecDebuggerStack.class/instance/updating/updateDebuggingActions.st R Spec-Debugger.package/extension/DebugAction/instance/shortcutCommand.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - code search/browseClassReferencesShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - code search/browseMethodHierarchyShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - debugger/restartExecutionShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - debugger/resumeExecutionShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - debugger/stepIntoShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - debugger/stepOverShortcut.st A Tool-Base.package/PharoShortcuts.class/instance/keymaps - debugger/stepThroughShortcut.st M Traits.package/TClass.class/instance/copying/duplicateClassWithNewName_.st Log Message: ----------- 60207 19054 QA 3.2.7 https://pharo.fogbugz.com/f/cases/19054 19050 Copying a class doesnt replicate the pool dictionary definitions https://pharo.fogbugz.com/f/cases/19050 19044 DebugAction should use key-mappings and not characters to indicate shortcuts https://pharo.fogbugz.com/f/cases/19044 http://files.pharo.org/image/60/60207.zip

Sept. 7, 2016