RFR: 8355065: ConcurrentModificationException in RichDiagnosticFormatter

Mon Apr 21 16:44:57 UTC 2025

On Sat, 19 Apr 2025 18:59:57 GMT, Liam Miller-Cushon <cushon at openjdk.org> wrote:

> Hi, please consider this fix for [JDK-8355065](https://bugs.openjdk.org/browse/JDK-8355065). RichDiagnosticFormatter is comparing type variables by their `toString` representation, and in this example the comparison fails due to type annotations. Replacing a call to `stripMetadataIfNeeded` with `stripMetadata` allows the comparison to succeed, and is consistent with the approach used in other parts of RichDiagnosticFormatter (e.g. for [the fix for JDK-8144580](https://github.com/openjdk/jdk/commit/570883e4dc5f769e17b3fcd8f2bbeb369dfe3a5d)).

This got me wondering whether we even need `stripMetadataIfNeeded` at this point, I filed https://bugs.openjdk.org/browse/JDK-8355204 to investigate removing it.

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

PR Comment: https://git.openjdk.org/jdk/pull/24769#issuecomment-2818991229