>
>>
>>> But tempNames is fundamentally broken for closures.
>>>
>>
>> Just to avoid confusing, #tempNames itself looks correct:
>>
>> tempNames
>> "Answer a SequenceableCollection of the names of the receiver's
>> temporary
>> variables, which are strings."
>>
>> ^ self debuggerMap tempNamesForContext: self
>>
>
> Yes. That's for ContextPart. But it doesn't work for CompiledMethod,
> since temps may differ between a method and its blocks.
>
>
Right, but weren't we always talking about Contexts? Because #tempNames of
CompiledMethod is different than the thing of contexts, isn't it? It is
related to the method trailer. In fact, I have just checked Squeak 4.3 and
CompiledMethod does not even implement #tempNames. The implementation in
Pharo is:
CompiledMethod >> #tempNames
self holdsTempNames ifFalse: [^#()].
^self tempNamesString subStrings: ' '
how would it be the correct implementation then?
Thanks
--
Mariano
http://marianopeck.wordpress.com
