This change looks fine, I'll push it if there are no any objections.

BTW we have some other places where we use the similar approach:
"(int)Native.getLong()", I assume we need to update it also?

On 09/11/2018 04:01, Ichiroh Takiguchi wrote:
Hello,
IBM would like to contribute this patch to OpenJDK project.

Bug:    https://bugs.openjdk.java.net/browse/JDK-8213614
Change: https://cr.openjdk.java.net/~itakiguchi/8213614/webrev.00/

Issue:
This issue only happens on 64bit big endian CPU.
Keyboard's key data is 32bit, but current code picks up it for 64bit data
and casted for 32bit data.
This operation works on little endian CPU, but it does not work on BIG endian 
CPU.

Test instruction is in JDK-8213614.

I'd like to obtain a sponsor for this issue.

Thanks,
Ichiroh Takiguchi
IBM Japan, Ltd.



--
Best regards, Sergey.

Reply via email to