Re: JDK 9 RFR of JDK-8037174: Fix serial lint warnings in sun.applet

2014-03-12 Thread Alan Bateman
cc'ing awt-dev as this where the Applet code is maintained. In any case, the suppressing of serial warnings looks okay to me, assuming of course they don't leak into the serialized stream of a supported type. -Alan. On 12/03/2014 06:18, Joe Darcy wrote: Hello, Please review the patch belo

Re: [9] Review Request: 8036103 Cleanup of java.awt and java.awt.peer packages

2014-03-12 Thread Oleg Pekhovskiy
Hi Sergey, the fix looks good. Regards, Oleg. On 03.03.2014 23:26, Sergey Bylokhov wrote: Hello. Please review a "weekend cleanup" in jdk 9. typos,final,replacement of old tags, unused imports. Bug: https://bugs.openjdk.java.net/browse/JDK-8036103 Webrev can be found at: http://cr.openjdk.jav

Re: JDK 9 RFR of JDK-8037174: Fix serial lint warnings in sun.applet

2014-03-12 Thread Petr Pchelko
Hello, Joe. The fix looks good. > assuming of course they don't leak into the serialized stream of a supported > type. These classes are related to the AppletViewer, so they should not. With best regards. Petr. On 12.03.2014, at 11:29, Alan Bateman wrote: > > cc'ing awt-dev as this where th

[9] Review request for 8035069 [macosx] Loading resolution variants by demand

2014-03-12 Thread Alexander Scherbatiy
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8035069 webrev: http://cr.openjdk.java.net/~alexsch/8035069/webrev.00 Image resolution variants are generated by request and cached in the ImageCache. - ImageCache is refactored to store different type of