RFR 8240524: Removed warnings from test classes

Langer, Christoph christoph.langer at sap.com
Fri Mar 6 06:36:20 UTC 2020


Hi Vipin,

this all looks correct to me.

When changing the copyright headers, you have to keep the first (initial) year and only update the second year. If this doesn’t exist, you’ll have to add it, e.g.
Copyright (c) 1999, Oracle -> Copyright (c) 1999, 2020, Oracle

I can fix that for you though, when sponsoring, no need for new webrev.

Can I have a second review, please?

Thanks
Christoph



From: Vipin Sharma <vipinsharma85 at gmail.com>
Sent: Donnerstag, 5. März 2020 18:27
To: Langer, Christoph <christoph.langer at sap.com>; core-libs-dev at openjdk.java.net
Subject: RFR 8240524: Removed warnings from test classes

Hi All,

Please review patch to remove warnings from test classes.

Bug : https://bugs.openjdk.java.net/browse/JDK-8240524
Webrev : http://cr.openjdk.java.net/~clanger/webrevs/8240524.0/


Change description:

Class: test/jdk/java/lang/Boolean/GetBoolean.java
Fixed following warning:
1. "Exception 'java.lang.Exception' is never thrown in the method”

Class: test/jdk/java/lang/Boolean/MakeBooleanComparable.java
Fixed following warnings:
1. Explicit type argument Boolean can be replaced with <>
2. C-style array declaration of parameter 'args'

Class: test/jdk/java/lang/Boolean/ParseBoolean.java
Fixed following warning:
1. Exception 'java.lang.Exception' is never thrown in the method


Regards,
Vipin


More information about the core-libs-dev mailing list