Looks fine.
Thanks, Artem On 10/2/2013 2:34 PM, 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