> Currently, `ParserVerifier#verifyConstantPool` uses a `switch` to produce a 
> `Runnable`, which is consumed by a `Consumer<Runnable>` (instantiated within 
> a loop) which runs the task inside if a `try`/`catch`. We can eliminate a 
> number of lambdas and method references, plus some allocation pressure, in 
> this code by simplifying it so that the `switch` is itself run directly 
> within the `try`/`catch`.

David M. Lloyd has updated the pull request incrementally with one additional 
commit since the last revision:

  Make sure that we record every error instead of stopping at the first error 
in a particular CPE

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/21209/files
  - new: https://git.openjdk.org/jdk/pull/21209/files/15b03b47..f1572865

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=21209&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21209&range=00-01

  Stats: 40 lines in 1 file changed: 32 ins; 0 del; 8 mod
  Patch: https://git.openjdk.org/jdk/pull/21209.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21209/head:pull/21209

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

Reply via email to