velurimithun updated this revision to Diff 25677. velurimithun added a comment.
- use QLating1String - compile commenting remove_definition(DQT_NO_CAST_FROM_ASCII - Make necessary changes according to comments in https://phabricator.kde.org/D9420 - Use QString::fromUtf8(..) to decode an escape sequence like \x21 - Merge with origin/master - Rewrite strings using QLatin1String() and QString::fromLatin1() REPOSITORY R244 KCoreAddons CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D9420?vs=25076&id=25677 BRANCH complieWithoutRemoveDef REVISION DETAIL https://phabricator.kde.org/D9420 AFFECTED FILES autotests/CMakeLists.txt autotests/desktoptojsontest.cpp autotests/kaboutdataapplicationdatatest.cpp autotests/kaboutdatatest.cpp autotests/kautosavefiletest.cpp autotests/kdelibs4migrationtest.cpp autotests/kdirwatch_unittest.cpp autotests/kformattest.cpp autotests/kpluginfactorytest.cpp autotests/kpluginloadertest.cpp autotests/kpluginmetadatatest.cpp autotests/kprocesstest.cpp autotests/krandomtest.cpp autotests/kshelltest.cpp autotests/kstringhandlertest.cpp autotests/ktexttohtmltest.cpp autotests/kusertest.cpp autotests/multiplugin.cpp To: velurimithun, mlaurent Cc: dfaure, #frameworks