Thanks, Sergey!

One more thing, looks like I need a sponsor. I don’t have commit rights for the 
latest releases.

Best Regards,
Alexey

> On 13 Nov 2018, at 21:38, Sergey Bylokhov <sergey.bylok...@oracle.com> wrote:
> 
> Looks fine.
> 
> On 08/11/2018 01:40, Alexey Ushakov wrote:
>> I’ve checked the fix on 10.13. Looks like it works as well. So, here is 
>> updated webrev:
>> http://cr.openjdk.java.net/~avu/JDK-8213292/webrev.01
>> Best Regards,
>> Alexey
>>> On 8 Nov 2018, at 01:12, Sergey Bylokhov <sergey.bylok...@oracle.com 
>>> <mailto:sergey.bylok...@oracle.com>> wrote:
>>> 
>>>> I’ll check the fix on macOS 10.13 or lower but I prefer not to touch 
>>>> working code :), so decided to limit the fix impact to only 10.14
>>> The problem with such changes is that nobody will drop the OS-check, 
>>> because of the same reason, and it will be there forever.
>>> I suggest to recheck it on 10.13 and drop the OS-check if it works properly.
>>> 
>>> 
>>> -- 
>>> Best regards, Sergey.
> 
> 
> -- 
> Best regards, Sergey.

Reply via email to