Hi, Anthony.
The fix looks good.

On 02.10.2013 14:34, Anthony Petrov wrote:
Hello,

Please review a long-awaited back-port of a fix for https://bugs.openjdk.java.net/browse/JDK-7174704 at:

http://cr.openjdk.java.net/~anthony/8-61-headlessPrinting-7174704.0/

It is not a direct back-port because the code in JDK8 slightly differs from 7u, but the basic idea is the same: we always load the lwawt dynamic library on Mac, regardless of the headless/headful mode.

Also with this fix I remove support for the undocumented AWT_TOOLKIT environment variable. It's become obsolete with removal of the MToolkit from JDK7. For selecting custom toolkits users still can use the documented -Dawt.toolkit=<class-name> system property.

I've tested the fix with a test mentioned in the bug report, and it passes well.

--
best regards,
Anthony


--
Best regards, Sergey.

Reply via email to