On Mon, 27 Feb 2023 22:48:46 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   added some extra time to dead process.
>
> test/lib/jdk/test/lib/process/ProcessTools.java line 227:
> 
>> 225:                         // Give some extra time for the StreamPumper to 
>> run after the process completed
>> 226:                         Thread.sleep(1000);
>> 227:                         if (latch.getCount() > 0) {
> 
> Nit: we use > 0 here but != 0 above.

let use > in all places

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

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

Reply via email to