Looks fine Thanks, Alexander.
On 09/08/2017 07:19, Sergey Bylokhov wrote:
Hello, Please review the fix for jdk10. Bug: https://bugs.openjdk.java.net/browse/JDK-8153871 Webrev can be found at: http://cr.openjdk.java.net/~serb/8153871/webrev.00 We get a native error when we tries to initialize the "NSGraphicsContext" to pass it to the NSEvent. But according the specification [1] NSEvent.context is deprecated, and the method mouseEventWithType() which we use to construct the NSEvent highlights [2] that the context parameter is unused and the nil should be passed. In the fix we pass nil as suggested. Note that the test still fail time to time because of [3] [1] https://developer.apple.com/documentation/appkit/nsevent/1524291-context?language=objc [2] https://developer.apple.com/documentation/appkit/nsevent/1532495-mouseeventwithtype?language=objc [3] https://bugs.openjdk.java.net/browse/JDK-8186002