The fix looks good to me.

  Thanks,
  Alexandr.

On 7/29/2015 2:09 PM, Alexander Stepanov wrote:
Hello Alexander,

Thank you for the review; could you please look at the updated version (sorry):
http://cr.openjdk.java.net/~avstepan/8132237/webrev.01/

the only difference - missing propertyChange event set was added (so, JDK-8132153 is not an issue).

Thanks,
Alexander


On 29.07.2015 12:35, Alexander Scherbatiy wrote:


  The fix looks good to me.

  Thanks,
  Alexandr.

On 7/23/2015 2:02 PM, alexander stepanov wrote:
Could you please review the following fix:
http://cr.openjdk.java.net/~avstepan/8132237/webrev.00/
for the test bug
https://bugs.openjdk.java.net/browse/JDK-8132237

Just a single regression test for JDK-8131342, JDK-8131939, JDK-8132153 (now failing because of them).

Thanks,
Alexander



Reply via email to