> `java.util.concurrent.ConcurrentHashMap` is relatively old and has not been 
> updated to reflect the current state of Java and can be modernized: 
> 
> * Add `@Serial` annotations 
> * Seal classes and restrict subclassing for internal classes 
> * Use pattern matching for instance 
> * Remove redundant modifiers (such as some final declarations) 
> * Use Objects.requireNonNull for invariant checking 
> * Use diamond operators instead of explicit types 
> * Simplify expressions using Math::max

Per Minborg has updated the pull request incrementally with one additional 
commit since the last revision:

  Remove redundant line

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/11924/files
  - new: https://git.openjdk.org/jdk/pull/11924/files/b2dbdd8f..cd4b1365

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=11924&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=11924&range=02-03

  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/11924.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/11924/head:pull/11924

PR: https://git.openjdk.org/jdk/pull/11924

Reply via email to