The fix looks good to me.

 Thanks,
 Alexandr.

On 4/1/2015 1:31 PM, Semyon Sadetsky wrote:
Hi Sergey,

I extended the test to iterate through all available LnFs.
The updated webrev: http://cr.openjdk.java.net/~ssadetsky/8075314/webrev.01/

--Semyon




On 3/24/2015 2:35 PM, Sergey Bylokhov wrote:
Hi, Semyon.
The fix looks good. Since this is a jck issue I assume that this functionality was tested on all supported look and feels and the new test always passed. So I suggest to change the test and cover all l&f, to be sure that this bug will not be integrated back in some other/new looks and feels.

23.03.15 13:58, Semyon Sadetsky wrote:
Hello,

please review JDK9 fix
webrev: http://cr.openjdk.java.net/~alexsch/semyon-sadetsky/8075314/webrev.00/
bug: https://bugs.openjdk.java.net/browse/JDK-8075314

Thanks!
--Semyon




Reply via email to