Hi, Eli,


commit 61da72 is different from commit c6ecda. We don't need to get IME status before setting IME status. If IME is the same status when setting it in the same status, IME will ignore the status changing. Emacs can be a litttle bit faster if get_ime_open_status_fn() is not invoked. 



And IME somtimes doesn't fully comply with Microsoft IME api, RIME for example. RIME which is a open source chinese ime doesn't return the status as Microsoft pinyin IME. When Microsoft pinyin IME is activated,  if Microsoft pinyin is in english mode, w32-get-ime-open-status() return nil.&nbsp;If Microsoft pinyin is in chinese mode, w32-get-ime-open-status() return t.&nbsp; We can press <SHIFT&gt; to switch between two modes by default.


When RIME is activated,  w32-get-ime-open-status() return t whenever IME is in english mode or chinese mode.&nbsp;


I modified code in w32-get-ime-open-status(). w32-get-ime-open-status() can get the status from IME.


$ git diff master -- w32fns.c
diff --git a/src/w32fns.c b/src/w32fns.c
index dddf3dc571..124b795119 100644
--- a/src/w32fns.c
+++ b/src/w32fns.c
@@ -3459,10 +3459,8 @@ w32_msg_pump (deferred_msg * msg_buf)
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (!context)
&nbsp;&nbsp;&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;&nbsp; BOOL new_status = (msg.wParam != 0);
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; BOOL ime_status = get_ime_open_status_fn (context);
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (new_status != ime_status)
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&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; BOOL wParam = (BOOL) msg.wParam;
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; set_ime_open_status_fn (context, wParam);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; release_ime_context_fn (focus_window, context);
&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; }
@@ -10261,7 +10259,7 @@ DEFUN ("w32-get-ime-open-status",
&nbsp;&nbsp;&nbsp;&nbsp; {
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; HWND current_window = FRAME_W32_WINDOW (sf);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; HIMC context = get_ime_context_fn (current_window);
-&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (!context)
+&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; if (context)
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; {
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; BOOL retval = get_ime_open_status_fn (context);
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; release_ime_context_fn (current_window, context);





------------------&nbsp;原始邮件&nbsp;------------------
发件人:&nbsp;"Eli Zaretskii"<eliz@gnu.org&gt;;
发送时间:&nbsp;2020年4月13日(星期一) 晚上9:44
收件人:&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



&gt; From: "Albert" <georgealbert@qq.com&gt;
&gt; Date: Mon, 13 Apr 2020 21:15:16 +0800
&gt; Cc: emacs-devel <emacs-devel@gnu.org&gt;
&gt; 
&gt; Thanks for your help. I will follow the emacs coding standard in the future.

Thanks, please try the latest master branch, I've modified the code a
little.&nbsp; In particular, w32-get-ime-open-status now returns nil or t,
not 0 or 1.