RFR: 8259446: runtime/jni/checked/TestCheckedReleaseArrayElements.java fails with stderr not empty [v2]

David Holmes dholmes at openjdk.java.net
Fri Jan 8 22:18:14 UTC 2021


On Fri, 8 Jan 2021 15:07:13 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add comment as requested by Dan
>
> test/hotspot/jtreg/runtime/jni/checked/TestCheckedReleaseArrayElements.java line 51:
> 
>> 49:                                                       "-Djava.library.path=" + Utils.TEST_NATIVE_PATH,
>> 50:                                                       "TestCheckedReleaseArrayElements");
>> 51:             OutputAnalyzer output = ProcessTools.executeProcess(pb);
> 
> I would add a comment above L51:
> // Uses executeProcess() instead of executeTestJvm() to avoid passing options
> // that might generate output on stderr (which should be empty for this test).

Done.

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

PR: https://git.openjdk.java.net/jdk/pull/2001


More information about the hotspot-runtime-dev mailing list