From e4d9ac638e9611cb7f5c9a1a48ad3b16b4f27a30 Mon Sep 17 00:00:00 2001
From: fxliang <fx.liang@outlook.com>
Date: Sat, 9 Dec 2023 21:44:29 +0800
Subject: [PATCH] Update inline preedit option only when inline_preedit is
 changed

---
 RimeWithWeasel/RimeWithWeasel.cpp | 13 +++++++++----
 1 file changed, 9 insertions(+), 4 deletions(-)

diff --git a/RimeWithWeasel/RimeWithWeasel.cpp b/RimeWithWeasel/RimeWithWeasel.cpp
index b57a0383a..18d0d7340 100644
--- a/RimeWithWeasel/RimeWithWeasel.cpp
+++ b/RimeWithWeasel/RimeWithWeasel.cpp
@@ -491,20 +491,22 @@ void RimeWithWeaselHandler::_LoadAppInlinePreeditSet(UINT session_id, bool ignor
 	if(!ignore_app_name && m_last_app_name == app_name)
 		return;
 	m_last_app_name = app_name;
+	bool inline_preedit = m_ui->style().inline_preedit;
 	if (!app_name.empty())
 	{
 		if (m_app_options.find(app_name) != m_app_options.end())
 		{
 			AppOptions& options(m_app_options[app_name]);
 			auto pfind = std::make_shared<bool>(false);
-			std::for_each(options.begin(), options.end(), [session_id, app_name, pfind, this](std::pair<const std::string, bool> &pair)
+			std::for_each(options.begin(), options.end(), [session_id, app_name, pfind, inline_preedit, this](std::pair<const std::string, bool> &pair)
 			{
 			if(pair.first == "inline_preedit")
 			{
 				*pfind = true;
 				RimeSetOption(session_id, pair.first.c_str(), Bool(pair.second));
 				m_ui->style().inline_preedit = Bool(pair.second);
-				_UpdateInlinePreeditStatus(session_id);
+				if(m_ui->style().inline_preedit != inline_preedit)
+					_UpdateInlinePreeditStatus(session_id);
 			}
 			});
 			if (!(*pfind))
@@ -527,7 +529,8 @@ void RimeWithWeaselHandler::_LoadAppInlinePreeditSet(UINT session_id, bool ignor
 					m_ui->style().inline_preedit = !!inline_preedit;
 				RimeConfigClose(&config);
 				RimeFreeStatus(&status);
-				_UpdateInlinePreeditStatus(session_id);
+				if(m_ui->style().inline_preedit != inline_preedit)
+					_UpdateInlinePreeditStatus(session_id);
 			}
 		}
 	}
@@ -1124,9 +1127,11 @@ void RimeWithWeaselHandler::_GetStatus(Status & stat, UINT session_id, Context&
 			m_last_schema_id = schema_id;
 			if(schema_id != ".default") {						// don't load for schema select menu
 				RimeSetOption(session_id, "__synced", false); // Sync new schema options with front end
+				bool inline_preedit = m_ui->style().inline_preedit;
 				_LoadSchemaSpecificSettings(schema_id);
 				_LoadAppInlinePreeditSet(session_id, true);
-				_UpdateInlinePreeditStatus(session_id);			// in case of inline_preedit set in schema
+				if(m_ui->style().inline_preedit != inline_preedit)
+					_UpdateInlinePreeditStatus(session_id);			// in case of inline_preedit set in schema
 				_RefreshTrayIcon(session_id, _UpdateUICallback);	// refresh icon after schema changed
 				if (m_show_notifications_when & SHOWN_SCHEMA) {
 					ctx.aux.str = stat.schema_name;