RFR: 7904121: jasm usage in jtreg test leads to log messages on stdout of jtreg process

Jaikiran Pai jpai at openjdk.org
Thu Dec 11 06:43:47 UTC 2025


Can I please get a review of this change which proposes to address the issue noted in https://bugs.openjdk.org/browse/CODETOOLS-7904121.

The change here replaces the usage of the deprecated constructors in `org.openjdk.asmtools.jasm.Main` and `org.openjdk.asmtools.jcoder.Main` to their newer counterparts. This allows jtreg the control over where the stdout and stderr  generated by those tools gets written to. 

A new self test has been introduced to verify that the compilation of a jasm file in a jtreg test, generates the expected stdout/stderr sections in the test's report file.

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

Commit messages:
 - close the streams
 - 7904121: jasm usage in jtreg test leads to log messages on stdout of jtreg process

Changes: https://git.openjdk.org/jtreg/pull/301/files
  Webrev: https://webrevs.openjdk.org/?repo=jtreg&pr=301&range=00
  Issue: https://bugs.openjdk.org/browse/CODETOOLS-7904121
  Stats: 145 lines in 5 files changed: 133 ins; 2 del; 10 mod
  Patch: https://git.openjdk.org/jtreg/pull/301.diff
  Fetch: git fetch https://git.openjdk.org/jtreg.git pull/301/head:pull/301

PR: https://git.openjdk.org/jtreg/pull/301


More information about the jtreg-dev mailing list