Hi All,
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:
MethodContext>>printString
"Answer an emphasized string in case of a breakpoint method"
^(self method notNil and: [self method hasBreakpoint])
ifTrue:[(super printString , ' [break]') asText allBold]
ifFalse:[super printString]
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...?
_,,,^..^,,,_
best, Eliot