=?gb18030?B?u9i4tKO6ILvYuLSjuiC72Li0o7ogu9i4tKO6IFtQQVRDSF0gQWRkIElNRSBzdGF0dXMgY2hhbmdlIHN1cHBvcnQgb24gd2luZG93cyBuYXRpdmVseQ==?=

[=?gb18030?B?QWxiZXJ0?= <georgealbert@qq.com>]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="gb18030", Size: 2801 bytes --]

Hi, Eli,


Thanks. I'll try it.





------------------ Ô­Ê¼Óʼþ ------------------
·¢¼þÈË:&nbsp;"Eli Zaretskii"<eliz@gnu.org&gt;;
·¢ËÍʱ¼ä:&nbsp;2020Äê4ÔÂ13ÈÕ(ÐÇÆÚÒ») ÏÂÎç3:05
ÊÕ¼þÈË:&nbsp;"Albert"<georgealbert@qq.com&gt;;
³­ËÍ:&nbsp;"emacs-devel"<emacs-devel@gnu.org&gt;;
Ö÷Ìâ:&nbsp;Re: »Ø¸´£º »Ø¸´£º »Ø¸´£º [PATCH] Add IME status change support on windows natively



Here's what I propose.

Change w32-set-ime-open-status to do this:

DEFUN ("w32-set-ime-open-status",
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Fw32_set_ime_open_status, Sw32_set_ime_open_status,
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 1, 1, 0,
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; doc: /* Set emacs IME open status on Windows. */)
&nbsp; (Lisp_Object status)
{
&nbsp;&nbsp;&nbsp; unsigned ime_status;
&nbsp;&nbsp;&nbsp; if (NILP (status))
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ime_status = 0;
&nbsp;&nbsp;&nbsp; else
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; ime_status = 1;

&nbsp;&nbsp;&nbsp; PostThreadMessage (dwWindowsThreadId, WM_EMACS_IME_STATUS,
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; (WPARAM)ime_status, 0);

&nbsp;&nbsp;&nbsp; return Qnil;
}

Then in w32fns.c do this to handle the WM_EMACS_IME_STATUS message:

&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; case WM_EMACS_IME_STATUS:
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (!set_ime_open_status_fn)
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; break;
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; else
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
		&nbsp; HIMC context = get_ime_context_fn (w32_system_caret_hwnd);
		&nbsp; if (!context)
		&nbsp;&nbsp;&nbsp; break;
		&nbsp; BOOL ime_status = get_ime_open_status_fn (context);
		&nbsp; BOOL new_status = (wParam != 0);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (ime_status != new_status)
		&nbsp;&nbsp;&nbsp; set_ime_open_status_fn (context, new_status);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; release_ime_context_fn (w32_system_caret_hwnd, context);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; }
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; break;

Then you can get rid of the 2 static variables.

This is entirely untested, so please make sure it works as intended.

[-- Attachment #2: Type: text/html, Size: 3277 bytes --]