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

Reply via email to