On Tue, 15 Nov 2022 13:38:51 GMT, Christoph Langer <clan...@openjdk.org> wrote:

>> Lutz Schmidt has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   8296709: use multiple test ids
>
> test/jdk/tools/launcher/TestXcheckJNIWarnings.java line 31:
> 
>> 29:  * @test id=noagent
>> 30:  * @bug 8187442
>> 31:  * @summary Launching app with or without jdwp agent shouldn't produce 
>> any jni warnings.
> 
> You could keep the old text here.

Accepted.

> test/jdk/tools/launcher/TestXcheckJNIWarnings.java line 39:
> 
>> 37:  * @test id=jdwp-agent
>> 38:  * @bug 8187442
>> 39:  * @summary Launching app with or without jdwp agent shouldn't produce 
>> any jni warnings.
> 
> Here the text should be: Launching app with jdwp agent shouldn't produce any 
> jni warnings.

Accepted.

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

PR: https://git.openjdk.org/jdk/pull/11083

Reply via email to