Review Request 126329: kio: Do not use QStringLiteral with multi strings
Patrick Spendrin
ps_ml at gmx.de
Sat Dec 12 23:08:09 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126329/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kio
Description
-------
Strings that are separated into multiple parts don't work on Windows
together with QStringLiteral as the first string is interpreted as a
wide (16bit) string, and the second one as a narrow (8bit) string.
Replacing with QString::fromLatin1 is the easiest solution keeping
the code layout the same, joining the strings does work too though.
Diffs
-----
autotests/dataprotocoltest.cpp 9fe238fdbb0e9682141772d423a64edd5621921b
src/core/ksambashare.cpp a3f84ac3971141e687d9ab17e0131a66db34ed5a
src/filewidgets/kfileplacesmodel.cpp b409c1b1617f97f3cdbc79a2c76110a5f9449398
src/ioslaves/help/kio_help.cpp cb27a77b22fe378a126d985621985265edb93767
src/widgets/kpropertiesdialog.cpp 0ff506273a10dba238fefc5c552c71434681285e
Diff: https://git.reviewboard.kde.org/r/126329/diff/
Testing
-------
Windows.
Thanks,
Patrick Spendrin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151212/9c1265d4/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list