RFR: 7903955: jtreg doesn't support .jasm in patch module

Dean Long dlong at openjdk.org
Fri Feb 28 19:31:08 UTC 2025


On Fri, 28 Feb 2025 10:20:24 GMT, Christian Stein <cstein at openjdk.org> wrote:

> Please review this change to support compilation of `.jasm` in patched modules.
> 
> Prior to this commit an extra check for `module != null` was the only difference when treating `.java` source files and JASM-related files. With that check removed, the handling of both types of source files could be merged.

src/share/classes/com/sun/javatest/regtest/exec/CompileAction.java line 213:

> 211:             if (currArg.endsWith(".java")) {
> 212:                 foundJavaFile = true;
> 213:             } else if (currArg.endsWith(".jasm") || currArg.endsWith("jcod")) {

Suggestion:

            } else if (currArg.endsWith(".jasm") || currArg.endsWith(".jcod")) {

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

PR Review Comment: https://git.openjdk.org/jtreg/pull/249#discussion_r1975916447


More information about the jtreg-dev mailing list