On Wed, 25 Aug 2021 12:47:41 GMT, Andrey Turbanov <github.com+741251+turban...@openjdk.org> wrote:
>> src/java.base/share/classes/java/net/URLPermission.java line 222: >> >>> 220: >>> 221: List<String> l = normalizeMethods(methods); >>> 222: l.sort(null); >> >> I am not opposed to this change, but I find this is slightly more ugly than >> `Collections.sort(l)`; so I have to ask: Is there any noticeable benefit? > > Actually I agree with you. > One more point is that List.sort(null) doesn't check at compile time that > class implements `Comparable`. But Collections.sort have this check. > I will revert last commit. > @azvegint are you ok with that? No objections. ------------- PR: https://git.openjdk.java.net/jdk/pull/5229
