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 <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> 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.
