can you just put in in the issue tracker? I have no time right now… but mails are such bad TODO trackers…
On Dec 2, 2012, at 10:42 PM, Guillermo Polito <[email protected]> wrote: > 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 <[email protected]> > 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 <[email protected]> > 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 <[email protected]>: > >> perfect! can you open a bug report with this information? :) > > > > > > > <MessageNode.GuillermoPolito.cs>
