This update results in a new test failure in ClosureCompilerTest>>testDecompiledDoitMethodTempNames.
This is probably just a test issue that can be fixed with an update to
ClosureCompilerTest>>closureCases, but I am not an expert here.

Eliot, Nicolas - could one of you please check this and fix the test if
that is what is required?

Also, a new test to document the problem being fixed by this update would
be a good thing.

Thanks,
Dave


On Tue, Nov 06, 2012 at 01:03:30PM +0100, Nicolas Cellier wrote:
> Some kind of Heisenberg principle, you cannot observe a node without
> some sort of interaction ;).
> We should create a price for the best anti-pattern of the month.
> It's invaluable for teaching.
> 
> Nicolas
> 
> 2012/11/6  <commits at source.squeak.org>:
> > Eliot Miranda uploaded a new version of Compiler to project The Trunk:
> > http://source.squeak.org/trunk/Compiler-eem.239.mcz
> >
> > ==================== Summary ====================
> >
> > Name: Compiler-eem.239
> > Author: eem
> > Time: 5 November 2012, 3:09:13.298 pm
> > UUID: 074c58f2-bc92-4c3f-adff-56c47d3884ef
> > Ancestors: Compiler-eem.238
> >
> > Eliminate horrible side-effect in printing whileTrue/whileFalse
> > nodes that undid the tranformation to whileTrue:/whileFalse:.
> >
> > =============== Diff against Compiler-eem.238 ===============
> >
> > Item was changed:
> >   ----- Method: MessageNode>>printWhileOn:indent: (in category 'printing') -----
> >   printWhileOn: aStream indent: level
> > -
> >         self printReceiver: receiver on: aStream indent: level.
> > +       self
> > +               printKeywords: originalSelector
> > +               arguments: originalArguments
> > -       (arguments isEmpty not
> > -        and: [arguments first isJust: NodeNil]) ifTrue:
> > -                       [selector := SelectorNode new
> > -                                                       key:
> > -                                                                       (selector key == #whileTrue:
> > -                                                                               ifTrue: [#whileTrue]
> > -                                                                               ifFalse: [#whileFalse])
> > -                                                       code: #macro.
> > -                       arguments := Array new].
> > -       self printKeywords: selector key arguments: arguments
> >                 on: aStream indent: level!
> >
> >