RFR: 8272992: Replace usages of Collections.sort with List.sort call in jdk.* modules [v2]

Jonathan Gibbons jjg at openjdk.java.net
Tue Sep 21 01:27:58 UTC 2021


On Tue, 14 Sep 2021 07:46:12 GMT, Andrey Turbanov <github.com+741251+turbanoff at openjdk.org> wrote:

>> Collections.sort is just a wrapper, so it is better to use an instance method directly.
>
> Andrey Turbanov has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8272992: Replace usages of Collections.sort with List.sort call in jdk.* modules

I've looked at the javadoc changes.
In general, it would be better to split a review like this into separate ones for separate components, but in this case, I guess it's innocuous enough.

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

PR: https://git.openjdk.java.net/jdk/pull/5230


More information about the serviceability-dev mailing list