diff options
author | Marc Mutz <marc.mutz@kdab.com> | 2021-06-14 14:47:35 +0200 |
---|---|---|
committer | Marc Mutz <marc.mutz@kdab.com> | 2021-06-30 07:30:13 +0200 |
commit | c587ebf54cecc78e47434670e9d392112670129f (patch) | |
tree | a6912bcadc752edd385b091c6dbd20a75660e633 /src/corelib/io/qsettings_mac.cpp | |
parent | 82063d9af1f9ad6cddab4d5526316816cbe1a77c (diff) |
Port QSettingsPrivate::get() to std::optional
... instead of a bool return and a QVariant out parameter.
Change-Id: I9d937668ede668075d1de5bb57f61e4c260aaddc
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/corelib/io/qsettings_mac.cpp')
-rw-r--r-- | src/corelib/io/qsettings_mac.cpp | 15 |
1 files changed, 6 insertions, 9 deletions
diff --git a/src/corelib/io/qsettings_mac.cpp b/src/corelib/io/qsettings_mac.cpp index 53715a124f..29ea995f2d 100644 --- a/src/corelib/io/qsettings_mac.cpp +++ b/src/corelib/io/qsettings_mac.cpp @@ -332,7 +332,7 @@ public: void remove(const QString &key) override; void set(const QString &key, const QVariant &value) override; - bool get(const QString &key, QVariant *value) const override; + std::optional<QVariant> get(const QString &key) const override; QStringList children(const QString &prefix, ChildSpec spec) const override; void clear() override; void sync() override; @@ -448,7 +448,7 @@ void QMacSettingsPrivate::set(const QString &key, const QVariant &value) domains[0].userName, hostName); } -bool QMacSettingsPrivate::get(const QString &key, QVariant *value) const +std::optional<QVariant> QMacSettingsPrivate::get(const QString &key) const { QCFString k = macKey(key); for (int i = 0; i < numDomains; ++i) { @@ -456,17 +456,14 @@ bool QMacSettingsPrivate::get(const QString &key, QVariant *value) const QCFType<CFPropertyListRef> ret = CFPreferencesCopyValue(k, domains[i].applicationOrSuiteId, domains[i].userName, hostNames[j]); - if (ret) { - if (value) - *value = qtValue(ret); - return true; - } + if (ret) + return qtValue(ret); } if (!fallbacks) break; } - return false; + return std::nullopt; } QStringList QMacSettingsPrivate::children(const QString &prefix, ChildSpec spec) const @@ -536,7 +533,7 @@ bool QMacSettingsPrivate::isWritable() const that->set(impossibleKey, QVariant()); that->sync(); - bool writable = (status == QSettings::NoError) && that->get(impossibleKey, 0); + bool writable = (status == QSettings::NoError) && that->get(impossibleKey).has_value(); that->remove(impossibleKey); that->sync(); |