On Fri, 6 Oct 2023 23:31:57 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:
>> Leonid Mesnik has updated the pull request incrementally with one additional >> commit since the last revision: >> >> fixed JInfoTest.java > > test/jdk/sun/tools/jinfo/JInfoTest.java line 41: > >> 39: * @summary Unit test for jinfo utility >> 40: * >> 41: * @requires vm.flagless > > I'm not so sure about this one. There are a number of things that change in > the output based on JVM args. It would seem we would want to test this. updated the test ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/16078#discussion_r1353279685