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

[pharo-project/pharo-core]

by GitHub

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

Feb. 14, 2014

[pharo-project/pharo-core] d45a12: 30761

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: d45a1290215271045c4da922bb155c4d43fc624a https://github.com/pharo-project/pharo-core/commit/d45a1290215271045c4da922… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-02-14 (Fri, 14 Feb 2014) Changed paths: M Athens-Cairo.package/CairoFontMetricsProvider.class/README.md M Athens-Cairo.package/CairoFontRenderer.class/README.md M Athens-CairoPools.package/AthensCairoDefs.class/README.md M Athens-Examples.package/AthensBalloonSurfaceExamples.class/README.md R Athens-Examples.package/AthensBalloonSurfaceExamples.class/class/as yet unclassified/newSurface_.st A Athens-Examples.package/AthensBalloonSurfaceExamples.class/class/instance creation/newSurface_.st M Athens-Examples.package/AthensMorphScene.class/README.md M Athens-Examples.package/AthensSimpleTreeNode.class/README.md M Athens-Examples.package/AthensTigerShape.class/README.md M Athens-Examples.package/AthensTreeView.class/README.md R Athens-Examples.package/AthensTreeView.class/class/as yet unclassified/openOn_extentBlock_childsBlock_.st A Athens-Examples.package/AthensTreeView.class/class/examples/example1.st A Athens-Examples.package/AthensTreeView.class/class/instance creation/openOn_extentBlock_childsBlock_.st M Athens-Examples.package/CurveWorkshop.class/README.md M Athens-Text.package/AthensTextBackground.class/README.md M Athens-Text.package/AthensTextComposer.class/README.md M Athens-Text.package/AthensTextRenderTest.class/README.md M Athens-Text.package/GlyphRenderer.class/README.md M Athens-Text.package/SimpleTextGlyphInfo.class/README.md R Athens-Text.package/SimpleTextGlyphInfo.class/class/as yet unclassified/fromFace_.st A Athens-Text.package/SimpleTextGlyphInfo.class/class/instance-creation/fromFace_.st R Athens-Text.package/SimpleTextGlyphInfo.class/instance/as yet unclassified/loadFrom_.st A Athens-Text.package/SimpleTextGlyphInfo.class/instance/initialization/loadFrom_.st A GroupManagerUI.package/extension/TBehavior/instance/prettyName.st R GroupManagerUI.package/extension/TClass/instance/prettyName.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script414.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30761.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st M Spec-Core.package/MenuModel.class/instance/initialize/initialize.st M Spec-MorphicAdapters.package/MorphicMenuAdapter.class/instance/private/buildWidget.st R Spec-Tools.package/ChangeSorterApplication.class/class/menu/changeSetMenu1NewGeneration_.st M Spec-Tools.package/ChangeSorterApplication.class/class/menu/changeSetMenu1_.st M Spec-Tools.package/ChangeSorterApplication.class/class/menu/changeSetMenu2_.st M Spec-Tools.package/ChangeSorterApplication.class/class/menu/classListMenu_.st M Spec-Tools.package/ChangeSorterApplication.class/class/menu/messageListMenu_.st R Spec-Tools.package/ChangeSorterApplication.class/instance/menu/changeSetMenu1NewGeneration_.st M Spec-Tools.package/ChangeSorterApplication.class/instance/menu/changeSetMenu1_shifted_.st M Spec-Tools.package/ChangeSorterApplication.class/instance/menu/changeSetMenu2_shifted_.st M Spec-Tools.package/ChangeSorterApplication.class/instance/menu/changeSetMenu_shifted_.st M Spec-Tools.package/ChangeSorterApplication.class/instance/menu/classMenu_shifted_.st M Spec-Tools.package/ChangeSorterApplication.class/instance/menu/messageMenu_shifted_.st M Spec-Tools.package/DualChangeSorterApplication.class/class/menu/changesListMenu_.st M Spec-Tools.package/DualChangeSorterApplication.class/instance/menu/changesMenu_shifted_from_to_.st M Spec-Tools.package/DualChangeSorterApplication.class/instance/menu/classMenu_shifted_from_to_.st M Spec-Tools.package/DualChangeSorterApplication.class/instance/menu/messageMenu_shifted_from_to_.st Log Message: ----------- 30761 12862 Class comments missing in Athens package https://pharo.fogbugz.com/f/cases/12862 12773 Adding packages to group raises DNU Metaclass>>prettyName: https://pharo.fogbugz.com/f/cases/12773 12741 [Dual]ChangeSorter is badly broken https://pharo.fogbugz.com/f/cases/12741 http://files.pharo.org/image/30/30761.zip

Feb. 14, 2014

Re: [Pharo-dev] Bugfix to '/foo:bar' asUrl

by Johan Brichau

On 14 Feb 2014, at 11:57, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > Yes, I see the function tests, but not the functional-ajaxified ones... Sorry, I now realize this is present in the non-ajaxified too. So, you can test it there. If you enter a text with a '+' inside, you will see it gets replaced with a space too. Johan

Feb. 14, 2014

Re: [Pharo-dev] Versionner Suggestions

by Sean P. DeNigris

Christophe Demarey wrote > Done in Versionner repo. Will be integrated soon. Thanks, Christophe! ----- Cheers, Sean -- View this message in context: http://forum.world.st/Versionner-Suggestions-tp4743390p4743673.html Sent from the Pharo Smalltalk Developers mailing list archive at Nabble.com.

Feb. 14, 2014

Amber Suggested Use

by J.F. Rick

It would be cool to use Pharo / Amber to do something like the following: http://flapmmo.com Cheers, Jeff -- Jochen "Jeff" Rick, Ph.D. http://www.je77.com/ Skype ID: jochenrick

Feb. 14, 2014

Re: [Pharo-dev] [gsoc-mentors] Re: GSoC: call for ideas

by Nicolas Petton

Paolo Bonzini writes: > Il 11/02/2014 10:42, Damien Cassou ha scritto: >> Hi fellow Pharo hackers, >> >> ESUG, the European Smalltalk User Group, is applying for this >> year's Google Summer of Code. As you probably know, the Summer >> of Code provides the opportunity to fund students to work during >> the summer on Pharo. Please reply to this >> email (be sure to use "Reply to all") if you have ideas you >> would like to propose. >> >> Please include a summary of the project and links to web pages >> that can help prospective students to write their application. >> Please also include the following information: >> >> - if applicable, other dialects that you would be willing to >> mentor this project for >> >> - the skill level >> >> - name of the mentor(s), email addresses, and possibly any IRC >> network/channel/nickname where they can be found. >> >> Thanks for contributing to ESUG's Summer of Code application! >> > > Just a note that I haven't yet processed this thread, but our ideas page > is already looking pretty well (and Pharo section is big) so I'll > probably do it in the weekend or next Monday. I can do it if you want. Cheers, Nico

Feb. 14, 2014

Re: [Pharo-dev] Issue 12832 OpalCompiler / Debugger

by Camille Teruel

On 14 fÃ©vr. 2014, at 00:45, Nicolai Hess <nicolaihess(a)web.de> wrote: > Thank you camille, > I still don't fully understand, why it is so hard to make it work for all use cases. > It looks like a simple thing, make a sourceNode for the current context! > > So for me it looks like the main problem is to distinguish these cases: > Create a SourceNode for > a) Method or > b) Block or > c) MethodContext or > d) BlockContext > and at the moment we are mixing case c/d/b It's easy for method and block. For MethodContext, it's harder. I would love if I could write: MethodContext>>sourceNode ^ self isBlockContext ifTrue: [ closureOrNil sourceNode ] ifFalse: [ method sourceNode ]. but it doesn't work because inlined blocks don't create a new context: 1 to: 1 do: [ :i | ^ thisContext isBlockContext ] returns false. So instead we have to rely on pc->IR nodes and IR nodes->AST nodes mappings. But everything is not mapped as you would expect yet in these two mappings. I found that: - the (pushLiteral: nil) bytecodes used to mark the number of block temporaries have corresponding IR nodes but these node are mapped with the AST block node instead of the AST sequence node. - the (pushTemp: n) bytecodes used to push the copied values of blocks don't have corresponding IR nodes so no AST node is associated with them. - the block return bytecodes are mapped to the block node instead of the sequence. Fixing these mapping is not easy, and it's too big a change for Pharo 3. So we have to deal with the current mapping for now. > > Thank you ben, for finding/looking at the failing tests. > > (I wished we would have an improved debugger, showing > sourcecode, AST, IR and bytecode linked together :-) > > > > 2014-02-13 16:27 GMT+01:00 Camille Teruel <camille.teruel(a)gmail.com>: > Hi Nicolai, > > I opened the issue 12875 and committed a fix. > Can you look at it? > > > On 11 fÃ©vr. 2014, at 11:54, Nicolai Hess <nicolaihess(a)web.de> wrote: > > > Ben, Camille, can you take a look at this code > > |a b| > > a:='String'. > > b:='t'. > > a collectWithIndex:[:c :i | c < b] > > > > Debug this code and step over until you reach the > > collectWithIndex: method, the context inspector will show > > a,b,c,i even though only a and b are in the current scope. > > And it'll show the wrong values > > a->'String' > > b->'String' > > c->'String' > > i->'t' > > > > (This does not happen with old compiler and it does not > > happen in 307052) > >

Feb. 14, 2014

[pharo-project/pharo-core] e97ba0: 30760

by GitHub

Branch: refs/heads/3.0 Home: https://github.com/pharo-project/pharo-core Commit: e97ba0b74972f88f605f450012dc76fb6a503329 https://github.com/pharo-project/pharo-core/commit/e97ba0b74972f88f605f4500… Author: Jenkins Build Server <board(a)pharo-project.org> Date: 2014-02-14 (Fri, 14 Feb 2014) Changed paths: M RPackage-Core.package/extension/TBehavior/instance/originalName.st M Ring-Core-Containers.package/RGOrganization.class/README.md M Ring-Core-Kernel.package/RGClassInstanceVariableDefinition.class/README.md M Ring-Core-Kernel.package/RGClassVariableDefinition.class/README.md M Ring-Core-Kernel.package/RGDefinition.class/README.md M Ring-Core-Kernel.package/RGGlobalVariableDefinition.class/README.md M Ring-Core-Kernel.package/RGInstanceVariableDefinition.class/README.md M Ring-Core-Kernel.package/RGPoolVariableDefinition.class/README.md M Ring-Core-Kernel.package/RGVariableDefinition.class/README.md M Ring-Tests-Containers.package/RGContainerTest.class/README.md M Ring-Tests-Containers.package/RGNamespaceTest.class/README.md M Ring-Tests-Containers.package/RGPackageTest.class/README.md M Ring-Tests-Containers.package/RGSliceTest.class/README.md M Ring-Tests-Kernel.package/RGClassDefinitionTest.class/README.md M Ring-Tests-Kernel.package/RGCommentDefinitionTest.class/README.md M Ring-Tests-Kernel.package/RGGlobalDefinitionTest.class/README.md M Ring-Tests-Kernel.package/RGMethodDefinitionTest.class/README.md M Ring-Tests-Kernel.package/RGTraitDefinitionTest.class/README.md M Ring-Tests-Kernel.package/RGVariableDefinitionTest.class/README.md M Ring-Tests-Monticello.package/RGMonticelloTest.class/README.md A ScriptLoader30.package/ScriptLoader.class/instance/pharo - scripts/script413.st A ScriptLoader30.package/ScriptLoader.class/instance/pharo - updates/update30760.st M ScriptLoader30.package/ScriptLoader.class/instance/public/commentForCurrentUpdate.st Log Message: ----------- 30760 12874 Error on TBehavior>>originalName when the class is not in system dictionary but also does not start with AnObsolete https://pharo.fogbugz.com/f/cases/12874 12868 Class comments missing for Ring package https://pharo.fogbugz.com/f/cases/12868 http://files.pharo.org/image/30/30760.zip

Feb. 14, 2014

[pharo-project/pharo-core]

by GitHub

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

Feb. 14, 2014

Re: [Pharo-dev] Bugfix to '/foo:bar' asUrl

by Sven Van Caekenberghe

I think we are getting somewhere (it is about time). $+ is handled in any usage of ZnPercentEncoder, independent of context and safeSet. That is not right, it is only valid in the context of application/x-www-form-urlencoded get/post requests: http://en.wikipedia.org/wiki/Percent-encoding#The_application.2Fx-www-form-… Outside of that specific use case, $+ becomes space, should not be done, I think. OK, now I have something concrete to work with. I still would like to be able to run the Seaside Ajax functional test though. On 14 Feb 2014, at 11:42, Johan Brichau <johan(a)inceptive.be> wrote: > > On 14 Feb 2014, at 10:44, Sven Van Caekenberghe <sven(a)stfx.eu> wrote: > >> So any percent-encoded character is accepted [ but being encoded means that its meaning as structuring element is lost, consider $/ ], not necessarily encoded on output. > > Right, but here the exact opposite happens. > A reserved character (+) was included as an encoded character (%2B), gets correctly decoded by Zinc but then never gets encoded again when printing the url, hence become a structuring element representing a space in the printed url. > > Because a WAUrl is constructed from incoming ZnUrl's printString, the WAUrl gets to read the '+' instead of the '%2B'. > > The following code snippet is now true, but should be false imho: > > (ZnUrl fromString: (ZnUrl fromString: 'http://localhost/?param=foo%2Bbar') printString) printString = 'http://localhost/?param=foo%20bar' > > Johan

Feb. 14, 2014