[Help-smalltalk] Re: [feature] make TwistedPools the default pool search order

[Paolo Bonzini]

Issue status update for 
http://smalltalk.gnu.org/project/issue/206
Post a follow up: 
http://smalltalk.gnu.org/project/comments/add/206

Project:      GNU Smalltalk
Version:      <none>
Component:    STInST
Category:     feature requests
Priority:     normal
Assigned to:  Unassigned
Reported by:  S11001001
Updated by:   bonzinip
Status:       active
Attachment:   http://smalltalk.gnu.org/files/issues/gst-twistedpools.patch 
(21.92 KB)

The attached patch makes TwistedPools the default pool search order
outside the VM, and consequently changes your TwistedPools class to
just use the default pool search order.  All testcases still pass.

The code is heavily based on yours, with some refactoring because I
wasn't afraid of touching base classes :-) and because the search order
is implemented directly in Behavior (actually in Class).

I changed a couple of data structures.  For the set of superspaces of
this class and all the superclasses' environments, I used a Bag, which
makes it easy to account for namespaces that are present multiple
times.  For the topological sort, I used two IdentitySets (grey/white,
in three-color visit terminology) instead of a single dictionary.

I'll post the patch split in three to the ML too.