Make composition closing before text committing (fix rime#107)

WeaselTSF/Composition.cpp

@@ -280,7 +280,20 @@ void WeaselTSF::_UpdateComposition(ITfContext *pContext)
 
 	if (ok)
 	{
-		if (status.composing && !_IsComposing())
+		bool closed = _IsComposing();
+		if (!commit.empty())
+		{
+			// 修复顶字上屏的吞字问题：
+			// 顶字上屏（如五笔 4 码上屏时），当候选词数 > 1 时，
+			// 第 5 码输入时会将首选项顶屏。
+			// 此时由于第五码的输入，composition 应是开启的，同时也要在输入处插入顶字。
+			// 这里先关闭上一个字的 composition，然后为后续输入开启一个新 composition。
+			// 有点 dirty 但是 it works ...
+			_EndComposition(pContext);
+			_InsertText(pContext, commit);
+			closed = false;
+		}
+		if (status.composing && !closed)
 		{
 			if (!_fCUASWorkaroundTested)
 			{
@@ -294,14 +307,10 @@ void WeaselTSF::_UpdateComposition(ITfContext *pContext)
 			}
 			_StartComposition(pContext, _fCUASWorkaroundEnabled && !config.inline_preedit);
 		}
-		else if (!status.composing && _IsComposing())
+		else if (!status.composing && closed)
 			_EndComposition(pContext);
-		if (_IsComposing() && config.inline_preedit)
+		if (closed && config.inline_preedit)
 			_ShowInlinePreedit(pContext, context);
-		if (!commit.empty())
-		{
-			_InsertText(pContext, commit);
-		}
 	}
 }