[ENH][Refactoring] InstructionPrinter ([approved] dvf
Daniel Vainsencher
danielv at netvision.net.il
Wed Apr 30 11:16:07 UTC 2003
I see. In the previous state, each of two classes held a meaningless
subset of the variable references...
Yuck. This changeset cleans up plenty of cruft.
Daniel
Marcus <marcus at ira.uka.de> wrote:
> On Wed, Apr 30, 2003 at 12:00:13PM +0200, Daniel Vainsencher wrote:
> > I'm reviewing this now for approval.
> >
> > I didn't understand the use of the new variable "bingo" which seems to
> > be initialized, returned, but never updated?
> >
> > Can either of you explain it?
> >
> This variable was pushed down from AbtractIntructionPrinter, together
> with the Method that returns it:
>
> interpretNextInstructionUsing: aScanner
>
> bingo _ false.
> aScanner interpretNextInstructionFor: self.
> ^bingo
>
>
> So this simply interprets all bytecodes, sets bingo to "true"
> when seeing an IntVar reference, e.g.
>
> pushReceiverVariable: offset
>
> bingo _ true
>
> (this method in not in the changeset, it's allready there)
>
> Marcus
More information about the Squeak-dev
mailing list
|