RFR: 8303133: Update ProcessTools.startProcess(...) to exit early if process exit before linePredicate is printed. [v4]
Leonid Mesnik
lmesnik at openjdk.org
Mon Feb 27 23:11:40 UTC 2023
On Mon, 27 Feb 2023 22:48:46 GMT, David Holmes <dholmes at 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
More information about the core-libs-dev
mailing list