Re: [9] Review request for 8139394: more detailed message for "Could not initialize shell library"

2016-10-17 Thread Semyon Sadetsky
On 10/12/2016 8:45 PM, Sergey Bylokhov wrote: What kind of information these new error messages will contain(Will it contain paths, username, etc)? It is obvious from the the fix that it will contain the library name which loading was failed and the OS error message. On 07.10.16 13:10, Semyon

Re: RfR JDK-8167213 Move include/bridge/AccessBridgeCalls.c to the source directory

2016-10-17 Thread Magnus Ihse Bursie
On 2016-10-14 17:51, Pete Brunet wrote: Please review the following. The .h files and .c file provided to allow Assistive Technology to interface to the Java Access Bridge API are being removed from the built JRE/JDK images. They are not used much and they can be obtained online via the OpenJDK

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Alexander Zvegintsev
Looks good. On 10/7/16 4:21 PM, Sergey Bylokhov wrote: On 07.10.16 10:06, Semyon Sadetsky wrote: Hi Sergey, After applying the patch I found 72 usages of the Event class. Why they are not replaced? By the same reason why InputEvent.getModifiers() was not replaced by InputEvent.getModifiers

Re: [9] Review request for 8167176 Exported elements referring to inaccessible types in java.desktop

2016-10-17 Thread Alexandr Scherbatiy
Hello, Could you review the updated fix: http://cr.openjdk.java.net/~alexsch/8167176/webrev.01 MetalBorders.bumps and MetalScrollBarUI.bumps fields are nor marked for removal any more. Thanks, Alexandr. On 10/14/2016 3:23 PM, Sergey Bylokhov wrote: Is it necessary to remove te

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Semyon Sadetsky
On 10/7/2016 4:21 PM, Sergey Bylokhov wrote: On 07.10.16 10:06, Semyon Sadetsky wrote: Hi Sergey, After applying the patch I found 72 usages of the Event class. Why they are not replaced? By the same reason why InputEvent.getModifiers() was not replaced by InputEvent.getModifiersEx(): >>>

[9] Review request for 8168077: Solaris build failed: gtk2_interface.h typedef redeclared: GThreadFunctions

2016-10-17 Thread Semyon Sadetsky
--- old/src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.h 2016-10-17 16:31:34.889162611 +0300 +++ new/src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.h 2016-10-17 16:31:34.809162613 +0300 @@ -351,9 +351,6 @@ guint ellipsize : 3; }; - -typedef struct _GThreadFunctions

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Sergey Bylokhov
On 17.10.16 15:16, Semyon Sadetsky wrote: >>> We can replace old constants by the new one in the whole jdk, but I >>> updated only the code in InputEvent to make change smaller and >>> safer. So at least the new code in jdk and the code in applications >>> will start to use the ne

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Semyon Sadetsky
On 17.10.2016 17:19, Sergey Bylokhov wrote: On 17.10.16 15:16, Semyon Sadetsky wrote: >>> We can replace old constants by the new one in the whole jdk, but I >>> updated only the code in InputEvent to make change smaller and >>> safer. So at least the new code in jdk and the code i

Re: [9] Review request for 8168077: Solaris build failed: gtk2_interface.h typedef redeclared: GThreadFunctions

2016-10-17 Thread Philip Race
+1 assuming a JPRT build passes. I'll send you the right incantation off-list -phil. On 10/17/16, 6:43 AM, Semyon Sadetsky wrote: --- old/src/java.desktop/unix/native/libawt_xawt/awt/gtk2_interface.h 2016-10-17 16:31:34.889162611 +0300 +++ new/src/java.desktop/unix/native/libawt_xawt/awt/gtk2

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Sergey Bylokhov
On 17.10.16 17:39, Semyon Sadetsky wrote: On 17.10.2016 17:19, Sergey Bylokhov wrote: On 17.10.16 15:16, Semyon Sadetsky wrote: >>> We can replace old constants by the new one in the whole jdk, but I >>> updated only the code in InputEvent to make change smaller and >>> safer. So

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Semyon Sadetsky
On 17.10.2016 18:37, Sergey Bylokhov wrote: On 17.10.16 17:39, Semyon Sadetsky wrote: On 17.10.2016 17:19, Sergey Bylokhov wrote: On 17.10.16 15:16, Semyon Sadetsky wrote: >>> We can replace old constants by the new one in the whole jdk, but I >>> updated only the code in InputEven

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Sergey Bylokhov
On 17.10.16 19:01, Semyon Sadetsky wrote: How it could be safe? both are a different constants which should be used in pair with different methods? Then why do you add in java doc for those constants: @deprecated It is recommended that *_DOWN_MASK be used instead This recommendation was there

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Semyon Sadetsky
On 10/17/2016 7:35 PM, Sergey Bylokhov wrote: On 17.10.16 19:01, Semyon Sadetsky wrote: How it could be safe? both are a different constants which should be used in pair with different methods? Then why do you add in java doc for those constants: @deprecated It is recommended that *_DOWN_MASK

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Sergey Bylokhov
On 17.10.16 21:14, Semyon Sadetsky wrote: Then this explanation should be added to the javadoc deprecation note because currently it states that those constants can be replaced with the new ones. But actually they are related to different APIs and cannot simply substitute each other. It can be

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Semyon Sadetsky
On 10/17/2016 9:23 PM, Sergey Bylokhov wrote: On 17.10.16 21:14, Semyon Sadetsky wrote: Then this explanation should be added to the javadoc deprecation note because currently it states that those constants can be replaced with the new ones. But actually they are related to different APIs and

Re: [9] Review Request: 8143077 Deprecate InputEvent._MASK in favor of InputEvent._DOWN_MASK

2016-10-17 Thread Sergey Bylokhov
On 17.10.16 21:47, Semyon Sadetsky wrote: It can be replaced if it will be used in pair with getModifiersEx(). The old getModifiers() is also deprecated. And javadoc for getModifiersEx() describes what and how constants should be used. Can you add link to getModifiersEx() to all those constants'

Re: [9] Review request for 8167176 Exported elements referring to inaccessible types in java.desktop

2016-10-17 Thread Philip Race
Hi, First note that any of the alternatives here require an approved CCC before pushing. As I wrote in the bug the fields in JRootPane are not used. Making it a public supertype is no more useful than just deleting it. This like hiding the peers which was a much bigger change so please just