JDK 9 RFR of JDK-8035453: Fix serial lint warnings in com.sun.tools and elsewhere

Paul Sandoz paul.sandoz at oracle.com
Mon Feb 24 11:23:20 UTC 2014


Hi Joe,

On Feb 20, 2014, at 10:14 PM, Joe Darcy <joe.darcy at oracle.com> wrote:

> Hello,
> 
> Please review my proposed changes for:
> 
>    JDK-8035453: Fix serial lint warnings in com.sun.tools and elsewhere
>    http://cr.openjdk.java.net/~darcy/8035453.0/webrev/
> 

Looks good.

I agree with Paul B. that it seems a little odd to have to suppress serialization on an abstract class, especially one that implements Serializable (plus Connector.Argument also extends Serializable):


> --- old/src/share/classes/com/sun/tools/jdi/ConnectorImpl.java 2014-02-20 13:03:08.000000000 -0800
> +++ new/src/share/classes/com/sun/tools/jdi/ConnectorImpl.java 2014-02-20 13:03:08.000000000 -0800
> @@ -144,6 +144,7 @@
>         return string;
>     }
> 
> +    @SuppressWarnings("serial") // JDK implementation class
>     abstract class ArgumentImpl implements Connector.Argument, Cloneable, Serializable {
>         private String name;
>         private String label;


Paul.


More information about the core-libs-dev mailing list