Hello all,
I spent some time today looking into what cruft might be left behind after
sloughing colorPrint out of the system browser in favor of Shout. What I
found was: somewhat tangled and messy dependencies.

My main goal for the day was cleanly removing the preference
#colorWhenPrettyPrinting. Note that I may have removed more than what was
actually necessary, e.g., ShoutCore seems to disable the
colorWhenPrettyPrinting preference on install, and I removed that line,
since the preference would be gone.

Methods changed:

Text>>askIfAddStyle:req: (Collections)
Compiler>>format:in:notifying:contentsSymbol: (Compiler)
SHTextStylerST80>>initializePreferences (ShoutCore ...this is ironic...)

There were a lot of changes in the System category:

Preferences>>
colorWhenPrettyPrinting (removed)
defaultValueTableForCurrentRelease
smallLandPreferences

SystemDictionary>>
testFormatter
testFormatter2

And yet more in tools:

ChangeSorter>>setContents

CodeHolder>>
colorPrintString (removed)
contentsSymbol
showingColorPrint (removed)
sourceStringPrettifiedAndDiffed
toggleColorPrint (removed)

FileContentsBrowser>>selectedMessage

The most disturbing thing that I saw on my Sunday drive was the method
#format:in:notifying:contentsSymbol: in Compiler (on both class and instance
sides). It looks as though the argument aSymbol is checked for equality to
#colorPrint, colorizing text the old way if so, and that's the only thing
it's used for. Thus, the method signature could stand to be simplified, if
it weren't for the fact that this one has other senders, meaning yet more
code churn; I'm hesitant to mess with the signature of a Compiler method at
this point, but doing so anyway for at least the sake of experiment, I've
changed:

Compiler>>
format:in:notifying:contentsSymbol: (removed)
format:in:notifying:contentsSymbol: (removed)
format:in:notifying: (added)
format:in:notifying: (added)

In addition to applying the above, I had to do:

Preferences removePreference: #colorWhenPrettyPrinting

... to make the actual preference go away.

So far I haven't seen anything explode yet, but this is a pretty ambitious
change for me, considering that I've only been Squeaking for a little over a
year.

Would the good people of squeak-dev be so kind as to look over the changes
I've placed in the Inbox for obvious errors? Here they are:

Collections-rss.118
Compiler-rss.72
Tools-rss.110
System-rss.134

 - Ron
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.squeakfoundation.org/pipermail/squeak-dev/attachments/20090823/77f9bfb0/attachment.htm