RFR: JDK-8241310 Fix warnings in jdk buildtools
Erik Joelsson
erik.joelsson at oracle.com
Thu Mar 19 17:54:17 UTC 2020
Looks good to me.
I love the WrapperGenerator using Vector and Hashtable!
/Erik
On 2020-03-19 09:53, Magnus Ihse Bursie wrote:
> The buildtools (java tools needed to be run during the build) have
> long been plagued by warnings, includuing deprecations and unchecked
> warnings, which cannot be silenced during the build.
>
> This patch fixes all buildtool warnings. Most of the warnings are
> fixed properly, but a few have had their warnings suppressed locally.
>
> For two tools, cldrconverter and tzdb, I gave up to get them fully
> fixed, and instead suppressed warnings in some places. Common for both
> these tools were that they used collections of some kind, with a mixed
> bag of data types (e.g. a Map from String to either String, HashMap,
> ArrayList and String[]). I'm frankly not sure how this could ever have
> worked :-) but I assume that the data files being parsed has a
> structure that "guarantees" that this works.
>
> Two files in generatecharacter were missing a proper copyright header.
> I noticed this when I were about to update the copyright year, and
> when I checked the other files I noted another one without header.
> While I did not need to change this file, I thought it was suitable to
> fix the missing header for both files.
>
> I have verified that the code generated by the build is identical with
> and without this patch.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8241310
> WebRev:
> http://cr.openjdk.java.net/~ihse/JDK-8241310-fix-warnings-in-buildtools/webrev.01
>
> /Magnus
More information about the build-dev
mailing list