RFR: JDK-8298528: Clean up raw type warnings in base in bindings and collections packages

John Hendrikx jhendrikx at openjdk.org
Mon Dec 26 14:41:53 UTC 2022


On Mon, 19 Dec 2022 18:00:30 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

> I spot-checked it and it seems OK, but I'll let Nir and Ambarish review it in more detail. One question I had was around the changes to remove one of the overloads of `mergeSort` (the one without a `Comparator` arg) from `SortHelper.java`: I presume you have verified that you won't ever get an NPE due to a null `Comparator`?

I checked all the call sites, and they never call it with a `null` as they check either just before or in the previous method if the comparator is `null` before going into that path, so the code that was removed wasn't (currently) in use (perhaps it was in the past).

The caller is now responsible for calling either a version that uses an explicit comparator (with the appropriate generic signature), or one that relies on the natural order of elements (ie. the elements implement `Comparable`).  I documented the methods to make sure (future) callers know what to expect.

-------------

PR: https://git.openjdk.org/jfx/pull/972


More information about the openjfx-dev mailing list