[squeak-smalltalk/squeak-object-memory] Remove unused globalSourceRanges from Encoder via MethodMassage (Issue #157)

LinqLover created an issue (squeak-smalltalk/squeak-object-memory#157)

Quoting from the version message of Compiler-eem.504:

> globalSourceRanges appears to be unused. It has been removed from the code, but cannot be removed from Encoder because class shape changes do not recompile code atomically and so the specific BytecodeEncoder subclass is broken when attempting to recompile itself without the globalSourceRanges inst var. The fix would be to use bytecode scanning to revise methods when class shapes change (a la VisualWorks).

With the recent introduction of MethodMassage into the trunk, this should be easier to be done now. The result of this ticket would be a Monticello version that removes the instance variable via a preamble script, deprecates the accessor, and formally undeclares the variable from the class definition as a regular monticello operation.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/squeak-smalltalk/squeak-object-memory/issues/157
You are receiving this because you are subscribed to this thread.

Message ID: <squeak-smalltalk/squeak-object-memory/issues/[email protected]>

Squeak-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]