[Pharo 7.0-dev] Build #796: 21761-change-asDoItIn-to-asDoItForContext

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[Pharo 7.0-dev] Build #796: 21761-change-asDoItIn-to-asDoItForContext

ci-pharo-ci-jenkins2
There is a new Pharo build available!
       
The status of the build #796 was: SUCCESS.

The Pull Request #1241 was integrated: "21761-change-asDoItIn-to-asDoItForContext"
Pull request url: https://github.com/pharo-project/pharo/pull/1241

Issue Url: https://pharo.fogbugz.com/f/cases/21761
Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/796/
Reply | Threaded
Open this post in threaded view
|

Re: [Pharo 7.0-dev] Build #796: 21761-change-asDoItIn-to-asDoItForContext

Marcus Denker-4
For AST evaluation I continue to improve things step-by-step.

e.g. this PR

-  improves the method that wraps ASTs into doit-methods in the case where we want to execute for a certain context (#asDoItForContext: does all the magic of rewriting temps and adding a return)
-  OpalCompiler>>#parse was radically simplifies by using asDoItForContext: and #asDoit.
-  I started to rewrite all users of the #translate method of the compiler which will be deprecated later (as you can either use #parse or #compile).

More to come, step by step…


> On 23 Apr 2018, at 11:02, [hidden email] wrote:
>
> There is a new Pharo build available!
>
> The status of the build #796 was: SUCCESS.
>
> The Pull Request #1241 was integrated: "21761-change-asDoItIn-to-asDoItForContext"
> Pull request url: https://github.com/pharo-project/pharo/pull/1241
>
> Issue Url: https://pharo.fogbugz.com/f/cases/21761
> Build Url: https://ci.inria.fr/pharo-ci-jenkins2/job/Test%20pending%20pull%20request%20and%20branch%20Pipeline/job/development/796/