Just nitpicks - your change is approved!
while ((s = lines.readLine()) != null) {
I think it's better style to localize
for (String s; (s = lines.readLine()) != null; ) {
121 String s = lines.readLine();
I think it's better style to write to a local that's never used.
In my old test TimedAcquireLeak I synchronize with child by
reading/writing just one byte.
public static void main(String[] args) throws Throwable {
// Synchronize with parent process, so that jps can find us
sendByte(System.out);
System.in.read();
On Tue, Feb 23, 2016 at 2:30 PM, Roger Riggs <roger.ri...@oracle.com> wrote:
> Please review a test fix to resolve a timing race with the start time of a
> spawned process.
>
> Webrev:
> http://cr.openjdk.java.net/~rriggs/webrev-start-8150346/
>
> Jira:
> https://bugs.openjdk.java.net/browse/JDK-8150346
>
> Thanks, Roger
>
