<AWT Dev> <i18n dev> The input method handling

Sergey Malenkov malenkov at gmail.com
Thu Dec 11 11:15:03 UTC 2014


Hi Naoto,

I think it may affect some Chinese symbols, which are represented by 2
Java-characters, but I don't have a corresponding keyboard to check. I
could modify the fix: create jstring before the if-statement and check
for its length. This looks safe and helps to determine a complex (in
Java terms) Unicode value. What do you think?




On Wed, Dec 10, 2014 at 10:21 PM, Naoto Sato <naoto.sato at oracle.com> wrote:
> Hi Sergey,
>
> It seems that the comment specifically mentions to go this route "when the
> string to insert is a complex Unicode value." and your change would break
> it. Have you investigated what it means?
>
> Also, if the change turns OK, you probably would want to change the variable
> name from "utf8Length" to something more appropriate.
>
> Naoto
>
>
> On 12/10/14, 9:56 AM, Sergey Malenkov wrote:
>>
>> Hi all,
>>
>> In Oracle JDK the input method handling is slightly different from
>> Apple JDK. And this is a reason of some issues in IntelliJ IDEA. I
>> found out that the insertText method in the following file uses
>> lengthOfBytesUsingEncoding instead of length in characters:
>>
>>
>> http://hg.openjdk.java.net/jdk9/dev/jdk/file/e5b66323ae45/src/java.desktop/macosx/native/libawt_lwawt/awt/AWTView.m
>>
>> This means that almost every Alt+Key combination generates
>> InputMethodEvent instead of pair of KeyEvent for PRESSED and TYPED. It
>> makes impossible to support custom shortcuts for all Alt+Key and
>> Shift+Alt+Key combinations on Mac, what is one of the reasons why
>> users do not want to migrate from Apple JDK to Oracle JDK on Mac.
>>
>> I suggest to apply the following patch to the AWTView.m file:
>>
>> --- a/src/macosx/native/sun/awt/AWTView.m Wed Dec 10 17:20:48 2014 +0400
>> +++ b/src/macosx/native/sun/awt/AWTView.m Wed Dec 10 19:51:56 2014 +0300
>> @@ -889,7 +889,7 @@
>>       // text, or 'text in progress'.  We also need to send the event
>> if we get an insert text out of the blue!
>>       // (i.e., when the user uses the Character palette or Inkwell),
>> or when the string to insert is a complex
>>       // Unicode value.
>> -    NSUInteger utf8Length = [aString
>> lengthOfBytesUsingEncoding:NSUTF8StringEncoding];
>> +    NSUInteger utf8Length = [aString length];
>>
>>       if ([self hasMarkedText] || !fProcessingKeystroke || (utf8Length >
>> 1)) {
>>           JNIEnv *env = [ThreadUtilities getJNIEnv];
>>
>> It makes the input method handling  more compatible with Apple JDK and
>> allows us to provide users the same behavior with Oracle JDK.
>>
>> Do you have any objections?
>> Are there some hidden pitfalls?
>>
>



-- 
Best regards,
Sergey A. Malenkov


More information about the awt-dev mailing list