[OpenJDK 2D-Dev] JDK 9 RFR of JDK-8034050: Fix serial lint warnings in javax.print
Jennifer Godinez
jennifer.godinez at oracle.com
Mon Feb 10 19:51:12 UTC 2014
Looks good. Approved.
- Jennifer
On 2/9/2014 9:12 PM, Joe Darcy wrote:
> Hello,
>
> Please review the proposed fix for
>
> JDK-8034050: Fix serial lint warnings in javax.print
> http://cr.openjdk.java.net/~darcy/8034050.0/
>
> I checked that the added serialVerionUID values match the serialver
> computations for those types on JDK 6 and 8.
>
> Thanks,
>
> -Joe
>
> --- old/src/share/classes/javax/print/DocFlavor.java 2014-02-09
> 21:09:29.000000000 -0800
> +++ new/src/share/classes/javax/print/DocFlavor.java 2014-02-09
> 21:09:29.000000000 -0800
> @@ -1045,6 +1045,7 @@
> * @author Alan Kaminsky
> */
> public static class URL extends DocFlavor {
> + private static final long serialVersionUID =
> 2936725788144902062L;
>
> /**
> * Constructs a new doc flavor with the given MIME type and a
> print
> --- old/src/share/classes/javax/print/PrintException.java 2014-02-09
> 21:09:30.000000000 -0800
> +++ new/src/share/classes/javax/print/PrintException.java 2014-02-09
> 21:09:29.000000000 -0800
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2000, Oracle and/or its affiliates. All rights
> reserved.
> + * Copyright (c) 2000, 2014, Oracle and/or its affiliates. All rights
> reserved.
> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> *
> * This code is free software; you can redistribute it and/or modify it
> @@ -33,7 +33,7 @@
> *
> */
> public class PrintException extends Exception {
> -
> + private static final long serialVersionUID = -5932531546705242471L;
>
> /**
> * Construct a print exception with no detail message.
> ---
> old/src/share/classes/javax/print/attribute/AttributeSetUtilities.java
> 2014-02-09 21:09:30.000000000 -0800
> +++
> new/src/share/classes/javax/print/attribute/AttributeSetUtilities.java
> 2014-02-09 21:09:30.000000000 -0800
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2000, 2004, Oracle and/or its affiliates. All rights
> reserved.
> + * Copyright (c) 2000, 2014, Oracle and/or its affiliates. All rights
> reserved.
> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> *
> * This code is free software; you can redistribute it and/or modify it
> @@ -77,6 +77,7 @@
> */
> private static class UnmodifiableAttributeSet
> implements AttributeSet, Serializable {
> + private static final long serialVersionUID =
> -6131802583863447813L;
>
> private AttributeSet attrset;
>
> @@ -147,6 +148,7 @@
> private static class UnmodifiableDocAttributeSet
> extends UnmodifiableAttributeSet
> implements DocAttributeSet, Serializable {
> + private static final long serialVersionUID =
> -6349408326066898956L;
>
> public UnmodifiableDocAttributeSet(DocAttributeSet
> attributeSet) {
>
> @@ -161,6 +163,7 @@
> extends UnmodifiableAttributeSet
> implements PrintRequestAttributeSet, Serializable
> {
> + private static final long serialVersionUID =
> 7799373532614825073L;
> public UnmodifiablePrintRequestAttributeSet
> (PrintRequestAttributeSet attributeSet) {
>
> @@ -175,6 +178,7 @@
> extends UnmodifiableAttributeSet
> implements PrintJobAttributeSet, Serializable
> {
> + private static final long serialVersionUID =
> -8002245296274522112L;
> public UnmodifiablePrintJobAttributeSet
> (PrintJobAttributeSet attributeSet) {
>
> @@ -189,6 +193,7 @@
> extends UnmodifiableAttributeSet
> implements PrintServiceAttributeSet, Serializable
> {
> + private static final long serialVersionUID =
> -7112165137107826819L;
> public UnmodifiablePrintServiceAttributeSet
> (PrintServiceAttributeSet attributeSet) {
>
> @@ -291,6 +296,7 @@
> */
> private static class SynchronizedAttributeSet
> implements AttributeSet, Serializable {
> + private static final long serialVersionUID =
> 8365731020128564925L;
>
> private AttributeSet attrset;
>
> @@ -357,6 +363,7 @@
> private static class SynchronizedDocAttributeSet
> extends SynchronizedAttributeSet
> implements DocAttributeSet, Serializable {
> + private static final long serialVersionUID =
> 6455869095246629354L;
>
> public SynchronizedDocAttributeSet(DocAttributeSet
> attributeSet) {
> super(attributeSet);
> @@ -369,6 +376,7 @@
> private static class SynchronizedPrintRequestAttributeSet
> extends SynchronizedAttributeSet
> implements PrintRequestAttributeSet, Serializable {
> + private static final long serialVersionUID =
> 5671237023971169027L;
>
> public SynchronizedPrintRequestAttributeSet
> (PrintRequestAttributeSet attributeSet) {
> @@ -382,6 +390,7 @@
> private static class SynchronizedPrintJobAttributeSet
> extends SynchronizedAttributeSet
> implements PrintJobAttributeSet, Serializable {
> + private static final long serialVersionUID =
> 2117188707856965749L;
>
> public SynchronizedPrintJobAttributeSet
> (PrintJobAttributeSet attributeSet) {
> @@ -395,6 +404,8 @@
> private static class SynchronizedPrintServiceAttributeSet
> extends SynchronizedAttributeSet
> implements PrintServiceAttributeSet, Serializable {
> + private static final long serialVersionUID =
> -2830705374001675073L;
> +
> public SynchronizedPrintServiceAttributeSet
> (PrintServiceAttributeSet attributeSet) {
> super(attributeSet);
> ---
> old/src/share/classes/javax/print/attribute/UnmodifiableSetException.java
> 2014-02-09 21:09:31.000000000 -0800
> +++
> new/src/share/classes/javax/print/attribute/UnmodifiableSetException.java
> 2014-02-09 21:09:30.000000000 -0800
> @@ -1,5 +1,5 @@
> /*
> - * Copyright (c) 2001, 2005, Oracle and/or its affiliates. All rights
> reserved.
> + * Copyright (c) 2001, 2014, Oracle and/or its affiliates. All rights
> reserved.
> * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> *
> * This code is free software; you can redistribute it and/or modify it
> @@ -33,6 +33,8 @@
> * @since 1.4
> */
> public class UnmodifiableSetException extends RuntimeException {
> + private static final long serialVersionUID = 2255250308571511731L;
> +
> /**
> * Constructs an UnsupportedOperationException with no detail
> message.
> */
>
More information about the 2d-dev
mailing list