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.