<div dir="ltr">Hi All,<div><br></div><div>    another change planned for Squeak 6 is eliminating ContextPart & MethodContext in favor of Context.  In reviewing the (very few) changes necessary I came across the following:</div><div><br></div><div>MethodContext>>printString</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">        </span>"Answer an emphasized string in case of a breakpoint method"</div><div><br></div><div><span class="gmail-Apple-tab-span" style="white-space:pre">        </span>^(self method notNil and: [self method hasBreakpoint])</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">         </span>ifTrue:[(super printString , ' [break]') asText allBold]</div><div><span class="gmail-Apple-tab-span" style="white-space:pre">               </span>ifFalse:[super printString] </div><div><br></div><div>Seriously?  I don't object to modifying [Method]Conext>>printOn: to include [break] but having printString answer a text seems completely broken to me, and overriding printString is a hack; printOn: being the operative method for producing self-descriptions.  Should we nuke it and add [break] to printOn:? Or...?</div><div><br></div><div><div class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div>
</div></div>