RFR: 7903808: IOOBE in compile action when processing class files only

[Christian Stein]

On Fri, 30 Aug 2024 19:28:22 GMT, Christian Stein <cstein at openjdk.org> wrote:

> Please review this fix preventing an `IndexOutOfBoundsException` in the compile action when processing class files only.
> 
> As an library used by a test might need `--enable-preview` it is possible that the `insertPos` variable is still assigned with the value `-1`: the `for (String currArg : args) { ... }` loop may exit normally without setting a valid insertion position.
> 
> This change checks for `-1` values and use `0` instead in order to prepend `--enable-preview` and `--source N` before any variadic arguments.

Sure. I added a self-test that used to fail with an `IIOBE`; based on [TypeToString.java](https://github.com/openjdk/jdk/blob/master/test/langtools/tools/javac/TypeToString.java).

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

PR Comment: https://git.openjdk.org/jtreg/pull/224#issuecomment-2337484820