Hello all,<div><br></div><div>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.</div>
<div><br></div><div>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.</div>
<div><br></div><div>Methods changed:</div><div><br></div><div>Text>>askIfAddStyle:req: (Collections)</div><div>Compiler>>format:in:notifying:contentsSymbol: (Compiler)</div><div>SHTextStylerST80>>initializePreferences (ShoutCore ...this is ironic...)</div>
<div><br></div><div>There were a lot of changes in the System category:</div><div><br></div><div>Preferences>></div><div>colorWhenPrettyPrinting (removed)</div><div>defaultValueTableForCurrentRelease</div><div>smallLandPreferences</div>
<div><br></div><div>SystemDictionary>></div><div>testFormatter</div><div>testFormatter2</div><div><br></div><div>And yet more in tools:</div><div><br></div><div>ChangeSorter>>setContents</div><div><br></div><div>
CodeHolder>></div><div>colorPrintString (removed)</div><div>contentsSymbol</div><div>showingColorPrint (removed)</div><div>sourceStringPrettifiedAndDiffed</div><div>toggleColorPrint (removed)</div><div><br></div><div>
FileContentsBrowser>>selectedMessage</div><div><br></div><div>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:</div>
<div><br></div><div>Compiler>></div><div>format:in:notifying:contentsSymbol: (removed)</div><div>format:in:notifying:contentsSymbol: (removed)</div><div>format:in:notifying: (added)</div><div>format:in:notifying: (added)</div>
<div><br></div><div>In addition to applying the above, I had to do:</div><div><br></div><div>Preferences removePreference: #colorWhenPrettyPrinting</div><div><br></div><div>... to make the actual preference go away.</div>
<div><br></div><div>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.</div><div><br></div><div>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:</div>
<div><br></div><div>Collections-rss.118</div><div>Compiler-rss.72</div><div>Tools-rss.110</div><div>System-rss.134</div><div><br></div><div> - Ron</div>