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