From f70e500ec2fd5c751e40833bdc4df586614bcff2 Mon Sep 17 00:00:00 2001 From: Sian Cao Date: Fri, 8 Dec 2017 14:47:32 +0800 Subject: [PATCH] fix: break forever loop for syncing backend data option setValue emits valueChanged, and handler for valueChanged emits setOption which requests backend to do doSetOption asynchronously which in turn calls option's setValue again. Change-Id: I5a6b5ccabfd29b26d3079779aad80461d7866d47 --- src/settings/backend/qsettingbackend.cpp | 1 - 1 file changed, 1 deletion(-) diff --git a/src/settings/backend/qsettingbackend.cpp b/src/settings/backend/qsettingbackend.cpp index fec9e4b..ae15b4f 100644 --- a/src/settings/backend/qsettingbackend.cpp +++ b/src/settings/backend/qsettingbackend.cpp @@ -72,7 +72,6 @@ void QSettingBackend::doSetOption(const QString &key, const QVariant &value) auto oldValue = d->settings->value("value"); if (oldValue != value) { d->settings->setValue("value", value); - Q_EMIT optionChanged(key, value); } d->settings->endGroup(); d->settings->sync(); -- 2.30.2