Re: [cp-patches] FYI: Fix for BoxLayout

[Tom Tromey]

>>>>> "Roman" == Roman Kennke <address@hidden> writes:

Roman> I have optimized this. Now no new array is allocated at all. A List
Roman> implementation is returned instead that iterates over the components
Roman> and returns only the visible ones. The List itself is cached in a
Roman> WeakHashMap, so multiple layouts for one Container will reuse the same
Roman> List impl, still if the Component (more precisely: its children array)
Roman> is no longer in use, it can be garbage collected.

This might be heavier than the original.
Probably I shouldn't have even mentioned all this and we should just
have stuck with something simple and clear, and then worried about
optimization (if even needed) later on.  Sorry for the noise...

Tom