JDK 14 RFR of JDK-8231202: Suppress warnings on non-serializable non-transient instance fields in serializable classes
Roger Riggs
Roger.Riggs at oracle.com
Wed Oct 9 17:13:59 UTC 2019
Looks fine.
Roger
On 10/9/19 1:10 PM, Joe Darcy wrote:
> Thanks for the reviews.
>
> Running some final checks, I'd also like to include two more
> @SuppressWarnings to clear the issues in java.base outside of
> java.util.concurrent; patch below.
>
> Thanks,
>
> -Joe
>
> diff -r 6e017b301287
> src/java.base/share/classes/sun/net/www/protocol/http/NegotiateAuthentication.java
> ---
> a/src/java.base/share/classes/sun/net/www/protocol/http/NegotiateAuthentication.java
> Wed Oct 09 09:57:41 2019 -0700
> +++
> b/src/java.base/share/classes/sun/net/www/protocol/http/NegotiateAuthentication.java
> Wed Oct 09 10:07:53 2019 -0700
> @@ -49,6 +49,7 @@
> private static final long serialVersionUID = 100L;
> private static final PlatformLogger logger =
> HttpURLConnection.getHttpLogger();
>
> + @SuppressWarnings("serial") // Not statically typed as Serializable
> private final HttpCallerInfo hci;
>
> // These maps are used to manage the GSS availability for diffrent
> @@ -67,6 +68,7 @@
> }
>
> // The HTTP Negotiate Helper
> + @SuppressWarnings("serial") // Not statically typed as Serializable
> private Negotiator negotiator = null;
>
> /**
>
> On 10/9/2019 7:28 AM, Roger Riggs wrote:
>> +1
>>
>> On 10/9/19 7:56 AM, Chris Hegarty wrote:
>>> Joe,
>>>
>>> On 05/10/2019 04:04, Joe Darcy wrote:
>>>> Hello,
>>>>
>>>> Please review the revised fix:
>>>>
>>>> http://cr.openjdk.java.net/~darcy/8231202.1/
>>>
>>> Given the prior discussion in this thread, then I think this version
>>> looks fine ( which consists solely of warning suppressions ).
>>>
>>> -Chris.
>>
More information about the core-libs-dev
mailing list