RFR: JDK-8168141: javax/management/remote/mandatory/notif/EmptyDomainNotificationTest.java: No notif received!

Robbin Ehn robbin.ehn at oracle.com
Tue Nov 8 10:44:34 UTC 2016


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.


More information about the serviceability-dev mailing list