Hi Roman,
The patch looks great. However swing already has a few image caches.
It seems swing has at least two:
sun.swing.CachedPainter used at least for Windows l&f and gtk (synth) l&f
and
com.sun.java.swing.plaf.nimbus.ImageCache (in 6u10 source tree. Has not
been checked into jdk7 yet)
There were discussions on unifying them. Would not it be great to have a
single image cache used throughout swing sources (gray filtered images
included)?
Thanks,
Igor
Roman Kennke wrote:
Hi,
I'm not a swing developer, but I'd suggest to not use static field for
cache, but have per-AppContext cache instead (due to security reasons)
Whoops. I have to get used to that .. Improved patch attached.
No more opinions on that?
Cheers, Roman
