[squeak-dev] new 4.2 cleanUp method

Chris Muller asqueaker at gmail.com
Thu Jan 6 23:14:59 UTC 2011


An expert review of the new 4.2 cleanUp process is needed.  Andreas, I
noticed that ReleaseBuilderTrunk class>>#prepareNewBuild for 4.1 did
not do a

  Smalltalk cleanUp: true

but that it had code that was duplicated in many of the class-side
cleanUp: methods.  However, the time-stamp on those methods was just a
few minutes after #prepareNewBuild, so I thought maybe you started
there but then decided to delegate to classes, but never actually
finished with a call to them?

I removed all of the duplicate code from the 4.1 version of
#prepareNewBuild methods and put in:

  Smalltalk cleanUp: true

instead.  Since that enumerates all classes in the system and calls
cleanUp: on them, it's safe to say it's a more aggressive "clean-up"
for 4.2 it was for 4.1.

But maybe too aggressive?

I think this is why Bernhards tests are mysteriously passing..



More information about the Squeak-dev mailing list