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.
