Hi!

I've hacked a fix, but I don't know if it is correct... can somebody check it so I make a Slice?

Guille

On Sun, Dec 2, 2012 at 8:56 PM, Guillermo Polito <guillermopolito@gmail.com> wrote:

Ok, I've found the cause :).

From update 20043:

 Issue 5744:    Clean up Temp Embedding 2): DoIts
        http://code.google.com/p/pharo/issues/detail?id=5744


- Doits are now compiled with source embedded
- embedding sources for doits pretty prints from AST as doits are compiled specially (with a return added)

CompiledMethod:

- remove blockExtendsToTempsMap: Not needed anymore, all calls go the MethodNode
- remove #copyWitTempsFromMethodNode: All senders use #copyWithSource
- remove #holdsTempNames. Temp name embedding not needed
- add isDoit
- schematicTempNamesString always returns the schematic string representation of all temps
- tempNames forwards to methodNode
- remove tempNamesString. Concatenate yourself if you need that.

Debugger>>selectedMessage asks the selectedContext directly for the source. No need for the methodMap.

DebuggerMethodMap

- no special handling of methods that encode temps. Just as for methodNode always.

MethodNode

- generateWithSource. embedd pretty printed code in case of doit
- remove generateWithTempNames

In general: rewrite code to use #generateWithSource

On Sat, Dec 1, 2012 at 6:24 PM, St�phane Ducasse <stephane.ducasse@inria.fr> wrote:

Sebastian

I can tell you that what you did is **useful** because we are all working like mad
and such kind of help is really welcome.

Stef

On Dec 1, 2012, at 3:24 PM, Sebastian Nozzi wrote:

> It's the #7076.
>
> 2012/11/29 Camillo Bruni <camillobruni@gmail.com>:
>> perfect! can you open a bug report with this information? :)
>

<MessageNode.GuillermoPolito.cs>