RFR: 8356441: IllegalStateException in RichDiagnosticFormatter after JDK-8355065 [v2]

Liam Miller-Cushon cushon at openjdk.org
Thu May 8 17:51:13 UTC 2025


> [JDK-8355065](https://bugs.openjdk.org/browse/JDK-8355065) fixed an issue where if RichDiagnosticFormatter was used recursively, it would overwrite instance state. That fix added guards to prevent the logic from being used recursively. As reported in [JDK-8356441](https://bugs.openjdk.org/browse/JDK-8356441) there are other situations where RichDiagnosticFormatter is used recursively. This fixes updates it to support being used recursively, by saving and restoring instance state on recursive calls.

Liam Miller-Cushon has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:

 - Avoid asserting on localized text
 - Merge remote-tracking branch 'origin/master' into b416188790
 - Imports
 - 8356441: IllegalStateException in RichDiagnosticFormatter after JDK-8355065

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25105/files
  - new: https://git.openjdk.org/jdk/pull/25105/files/d7238c1d..7babdb1e

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

  Stats: 733 lines in 20 files changed: 568 ins; 85 del; 80 mod
  Patch: https://git.openjdk.org/jdk/pull/25105.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25105/head:pull/25105

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


More information about the compiler-dev mailing list