On 03/16/2018 10:08 AM, Sergey Bylokhov wrote:
On 15/03/2018 22:55, Philip Race wrote:
Looks OK .. can you confirm this builds without issue with VS2013 as well, and that some

I'll recheck that.

AWT tests that use the components touched are OK ?

I cannot do that because the build still fails at the end, even for openjdk and if "disable-warnings-as-errors" is passed.

Oh yes .. well I guess step 1 is we get these build issues fixed.

-phil.



-phil

On 3/15/18, 10:29 PM, Sergey Bylokhov wrote:
Hello.
Please review fix for jdk11.

Bug: https://bugs.openjdk.java.net/browse/JDK-8198895
Webrev can be found at: http://cr.openjdk.java.net/~serb/8198895/webrev.01

VS2017 produces the warnings when we use casts some integral types to pointers or use casts of pointers to types. The fix simply suppress such warnings using the intermediate casts to intptr_t(same approach as in JDK-8196884[1])

[1] https://bugs.openjdk.java.net/browse/JDK-8196884




Reply via email to