Hello Sergey.
Thank you for your review.

As you told me, I could find out some "(int)Native.getLong()".
I'm not sure, I checked them before or not.
I'll get back to you.

Thanks,

On 2018-11-14 05:43, Sergey Bylokhov wrote:
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.


Reply via email to