On 20/06/2016 10:32, Hamlin Li wrote:

Would you please review the patch for following bug?

bug: https://bugs.openjdk.java.net/browse/JDK-8159879
webrev: http://cr.openjdk.java.net/~mli/8159879/
This looks okay except testServiceBindingWithConfigurations3 where the original comment is correct and just need to be re-ordered to:

Configuration cf1: [email protected] provides p.S
Test configuration:cf2: m1 uses p.S, [email protected] provides p.S
Test configuration:cf2: m1 uses p.S

-Alan

Reply via email to