On Sat, 20 Aug 2022 16:37:29 GMT, liach <d...@openjdk.org> wrote: >> We can use `Comparator.naturalOrder()` for cases when a `TreeMap` instance >> is constructed without comparator. This allows to squash two branches in >> `TreeMap.get()` into one. >> >> P.S. I think the comment of `TreeMap.getEntryUsingComparator()` is outdated. >> Should we also change it? > > src/java.base/share/classes/java/util/TreeMap.java line 3318: > >> 3316: // Adapt or create a key-based comparator >> 3317: Comparator<? super K> treeComparator = tree.comparator; >> 3318: return Map.Entry.comparingByKey(treeComparator == null ? >> natural() : treeComparator); > > You can probably have: > > return treeComparator == nul ? > Map.Entry.comparingByKey() : > Map.Entry.comparingByKey(treeComparator); > > instead.
Nope, there'd be a compilation error ------------- PR: https://git.openjdk.org/jdk/pull/9901