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