Review Request 126001: In KConfigTest::testEntryMap, convert QByteArray with nulls using a char *
Matthew Dawson
matthew at mjdsystems.ca
Mon Nov 9 00:23:42 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126001/
-----------------------------------------------------------
Review request for KDE Frameworks.
Repository: kconfig
Description
-------
Due to https://codereview.qt-project.org/#/c/106473/, Qt 5.6 keeps null
characters in QByteArray -> QString conversions, which breaks this test as
one QByteArray contains nulls. Instead, convert the QByteArray to const
char * first, so QString stops at the first null.
The actual behaviour of KConfig is unchanged, as internally the conversion
always went through a const char *, which avoids creating QStrings with
null characters.
Diffs
-----
autotests/kconfigtest.cpp 9a2998647b5e5f54d63059172b727505a8ae1c80
Diff: https://git.reviewboard.kde.org/r/126001/diff/
Testing
-------
Tested on both Qt 5.5.1 and Qt 5.6 (commit e996d68f6130847637ba287518cff1289cfa48e5), tests all pass now.
Thanks,
Matthew Dawson
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151109/20e85293/attachment.html>
More information about the Kde-frameworks-devel
mailing list