Re: Issue 2454 in pharo: decompiler fix

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

Re: Issue 2454 in pharo: decompiler fix

pharo

Comment #3 on issue 2454 by marcus.denker: decompiler fix
http://code.google.com/p/pharo/issues/detail?id=2454

Next action:

- take the code here and make a .cs or Slice out of it


Reply | Threaded
Open this post in threaded view
|

Re: Issue 2454 in pharo: decompiler fix

pharo

Comment #4 on issue 2454 by siguctua: decompiler fix
http://code.google.com/p/pharo/issues/detail?id=2454

Here the changeset with corresponding methods

Attachments:
        decompiler-fix.1.cs  1.6 KB


Reply | Threaded
Open this post in threaded view
|

Re: Issue 2454 in pharo: decompiler fix

pharo
Updates:
        Status: Fixed

Comment #5 on issue 2454 by stephane.ducasse: decompiler fix
http://code.google.com/p/pharo/issues/detail?id=2454

(No comment was entered for this change.)


Reply | Threaded
Open this post in threaded view
|

Re: Issue 2454 in pharo: decompiler fix

pharo
Updates:
        Status: Accepted
        Labels: -Milestone-1.2 Milestone-1.3

Comment #6 on issue 2454 by stephane.ducasse: decompiler fix
http://code.google.com/p/pharo/issues/detail?id=2454

In 12295 we tested with igor and without the fix in pharo we get the same  
as in Squeak:

test
         | one two |
        two := 2.
         ^{[one := 1].
        [ [one + two] value]}

is decompiled into

test
        | two one |
        two := 2.
        ^ {[one := 1]. [[one + two] value]}



Reply | Threaded
Open this post in threaded view
|

Re: Issue 2454 in pharo: decompiler fix

pharo
Updates:
        Labels: -Milestone-1.3

Comment #7 on issue 2454 by [hidden email]: decompiler fix
http://code.google.com/p/pharo/issues/detail?id=2454

(No comment was entered for this change.)