QSettings: support reading UTF-8 keys in INI files

[ChangeLog][QtCore][QSettings] The INI file reader now supports
keys encoded with UTF-8, as well as the %-encoded format. Writing
the keys back to the INI file is still done using %-encoded format.
This change does not touch the way the *values* are handled - they
are both read and written in UTF-8.

Drive-by: remove misleading comments from the reading algorithm.

Task-number: QTBUG-99401
Change-Id: I6a83cbf24d919a499540403688615f93cb195e93
Reviewed-by: Edward Welbourne <edward.welbourne@qt.io>

4 files changed, 62 insertions, 0 deletions

diff --git a/tests/auto/corelib/io/qsettings/CMakeLists.txt b/tests/auto/corelib/io/qsettings/CMakeLists.txt
index c9ba64f0ca..fa6af4eaf4 100644
--- a/tests/auto/corelib/io/qsettings/CMakeLists.txt
+++ b/tests/auto/corelib/io/qsettings/CMakeLists.txt
@@ -15,6 +15,8 @@ set(qsettings_resource_files
     "resourcefile6.plist"
     "withcomments.ini"
     "float.ini"
+    "qt5settings.ini"
+    "utf8settings.ini"
 )
 
 qt_internal_add_test(tst_qsettings
diff --git a/tests/auto/corelib/io/qsettings/qt5settings.ini b/tests/auto/corelib/io/qsettings/qt5settings.ini
new file mode 100644
index 0000000000..59239d29f3
--- /dev/null
+++ b/tests/auto/corelib/io/qsettings/qt5settings.ini
@@ -0,0 +1,11 @@
+[General]
+.%2C%27%25U%U0430%U0431%U0432%U0433%22%09=".,'%!@#$"
+%U265F=\x2658\x265a
+%UD83C%UDF0D=\xd83c\xdf10
+
+[Test]
+BAR=BAR
+B%C4R=B\xc4R
+OST=OST
+%D6SE=\xd6SE
+%U042D%U0442%U043E\%U0442%U0435%U0441%U0442=\x42d\x442\x43e \x442\x435\x441\x442
diff --git a/tests/auto/corelib/io/qsettings/tst_qsettings.cpp b/tests/auto/corelib/io/qsettings/tst_qsettings.cpp
index 48087b6a65..820a879d63 100644
--- a/tests/auto/corelib/io/qsettings/tst_qsettings.cpp
+++ b/tests/auto/corelib/io/qsettings/tst_qsettings.cpp
@@ -198,6 +198,10 @@ private slots:
     void floatAsQVariant();
 
     void testXdg();
+
+    void testReadKeys_data();
+    void testReadKeys();
+
 private:
     void cleanupTestFiles();
 
@@ -3663,5 +3667,39 @@ void tst_QSettings::testXdg()
 #endif
 }
 
+void tst_QSettings::testReadKeys_data()
+{
+    QTest::addColumn<QString>("filepath");
+
+    QTest::newRow("escaped") << ":/qt5settings.ini";
+    QTest::newRow("utf-8") << ":/utf8settings.ini";
+}
+
+void tst_QSettings::testReadKeys()
+{
+    QFETCH(QString, filepath);
+
+    QSettings settings(filepath, QSettings::IniFormat);
+    QCOMPARE(settings.status(), QSettings::NoError);
+
+    QVariantMap expectedValues;
+    expectedValues.insert("Test/BAR", "BAR");
+    expectedValues.insert("Test/OST", "OST");
+    expectedValues.insert("Test/B\xC3\x84R", "B\xC3\x84R"); // BÄR
+    expectedValues.insert("Test/\xC3\x96SE", "\xC3\x96SE"); // ÖSE
+    expectedValues.insert(
+            "Test/\xD0\xAD\xD1\x82\xD0\xBE/\xD1\x82\xD0\xB5\xD1\x81\xD1\x82", // Это/тест
+            "\xD0\xAD\xD1\x82\xD0\xBE \xD1\x82\xD0\xB5\xD1\x81\xD1\x82"); // Это тест
+    expectedValues.insert(".,'%U\xD0\xB0\xD0\xB1\xD0\xB2\xD0\xB3\"\t", ".,'%!@#$");
+    expectedValues.insert("\xE2\x99\x9F", "\xE2\x99\x98\xE2\x99\x9A"); // ♟︎ ♘♚
+    expectedValues.insert("\xF0\x9F\x8C\x8D", "\xF0\x9F\x8C\x90"); // 🌍 🌐
+
+    QVariantMap readValues;
+    for (const auto &key : settings.allKeys())
+        readValues.insert(key, settings.value(key));
+
+    QCOMPARE(readValues, expectedValues);
+}
+
 QTEST_MAIN(tst_QSettings)
 #include "tst_qsettings.moc"
diff --git a/tests/auto/corelib/io/qsettings/utf8settings.ini b/tests/auto/corelib/io/qsettings/utf8settings.ini
new file mode 100644
index 0000000000..bac010254f
--- /dev/null
+++ b/tests/auto/corelib/io/qsettings/utf8settings.ini
@@ -0,0 +1,11 @@
+[General]
+.%2C%27%25Uабвг%22%09=".,'%!@#$"
+♟=♘♚
+🌍=🌐
+
+[Test]
+BAR=BAR
+BÄR=BÄR
+OST=OST
+ÖSE=ÖSE
+Это\тест=Это тест