On 29.6.2015 12:09, Alan Bateman wrote:


On 29/06/2015 10:10, Jan Lahoda wrote:

Thanks for the comment - done that. Updated webrev:
http://cr.openjdk.java.net/~jlahoda/8080679/webrev.03/full/

Delta against the previous iteration:
http://cr.openjdk.java.net/~jlahoda/8080679/webrev.03/delta/

How does this look? Feedback is appreciated.

Nearly there :-)

I don't think initIDs needs to return a jboolean because it throws an
exception when it fails.

Also is there any reason to do this each time that a WindowsTerminal is
created. A static initializer would be more normal here.

The library is Windows-only, but the WindowsTerminal (or its subclasses) are registered on all platforms using "WindowsTerminal.class". While this does not cause initialization, it seemed safer to ensure the library is only loaded when needed.

Thanks,
    Jan


-Alan.

Reply via email to