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