Hi Ujwal,

synchronized(li) {
        while (li.received < 1) {
                li.wait(100);
        }
}

I don't see why we need while loop ?

To me it looks like you could just do:

synchronized(li) {
        li.wait();
}

Since either we got notification and received must be bigger than 0.
Or jtreg timed out.

/Robbin ('r'eviewer)

On 11/04/2016 12:03 PM, Ujwal Vangapally wrote:
Please review this small change for the bug below

https://bugs.openjdk.java.net/browse/JDK-8168141

Webrev:
http://cr.openjdk.java.net/~asapre/sponsorships/Ujwal/JDK-8168141/webrev.01/


Thanks,
Ujwal.

Reply via email to