diff options
author | Thiago Macieira <thiago.macieira@intel.com> | 2012-04-16 11:27:11 -0300 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-04-16 17:54:05 +0200 |
commit | 5eaf0b8ef0e0ad74e4c21cb6e688071a6cf8f66f (patch) | |
tree | 490dbe7b4b65cad9454ec0fb95a9e75e09fccbce /src/corelib/io/qsettings.cpp | |
parent | 1afff0e5fae90eaa6d7d95055d56969a29f068af (diff) |
Remove the Qt 5 todo about empty string lists in QSettings
There's no need to change behaviour now. The current marker (@Invalid())
returns an empty string list, as opposed to a list with one item.
Changing it would break compatibility unpredictably with Qt 4.
So I choose not to change QSettings. Let it be "Done".
Task-number: QTBUG-25110
Change-Id: Id1f353dfed800005d927183da237f3f8357c496d
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@nokia.com>
Reviewed-by: Jason McDonald <jason.mcdonald@nokia.com>
Diffstat (limited to 'src/corelib/io/qsettings.cpp')
-rw-r--r-- | src/corelib/io/qsettings.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/corelib/io/qsettings.cpp b/src/corelib/io/qsettings.cpp index 1ec0390235..8a6eef053e 100644 --- a/src/corelib/io/qsettings.cpp +++ b/src/corelib/io/qsettings.cpp @@ -775,8 +775,6 @@ void QSettingsPrivate::iniEscapedStringList(const QStringList &strs, QByteArray with Qt 4.0. @Invalid() stands for QVariant(), and QVariant().toStringList() returns an empty QStringList, so we're in good shape. - - ### Qt 5: Use a nicer syntax, e.g. @List, for variant lists */ result += "@Invalid()"; } else { |