Hello,
I've adjusted the patch based on changes to the client repo in the interim:
http://cr.openjdk.java.net/~darcy/8033908.1/
Please review.
Thanks,
-Joe
On 02/07/2014 11:53 AM, Joe Darcy wrote:
On 02/07/2014 01:33 AM, Alan Bateman wrote:
On 07/02/2014 07:53, Joe Darcy wrote:
Hello,
Please review the webrev to address
JDK-8033908: Fix serial lint warnings in com.sun.java.swing.plaf
http://cr.openjdk.java.net/~darcy/8033908.0/
This looks fine although for cases where you comment that the
supertypes aren't serial compatible across versions then I didn't
check that the supertypes document it (I assume they do).
Right, the supertypes in question are generally those covered in
8032627: Add @SuppressWarnings("serial") to appropriate
javax.swing classes
which are the Swing type which say "I'm serializable on the same
version, but not across versions." Some of the types in question are
subtypes of types in com.sun.* which directly document this constraint
too.
Thanks for the review,
-Joe
