Review Request 123479: Adapt to Qt 5.6 changes and prevent nullbytes in QStrings.
Milian Wolff
mail at milianw.de
Thu Apr 23 16:50:06 UTC 2015
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123479/
-----------------------------------------------------------
(Updated April 23, 2015, 4:50 p.m.)
Review request for KDE Frameworks, David Faure, Marc Mutz, and Volker Krause.
Repository: kcoreaddons
Description
-------
The len in inotify_event includes nulls of the name. To prevent
them from being included in the QString/QByteArray we must filter
them manually with a recent Qt 5 dev build now. See also:
https://codereview.qt-project.org/#/c/106473/
REVIEW: 123479
Diffs
-----
src/lib/io/kdirwatch.cpp a75fae012c59021996b46845db2e97abb5526930
Diff: https://git.reviewboard.kde.org/r/123479/diff/
Testing (updated)
-------
I used the test and looked at the output and also ran it against a patched qtbase with this:
https://paste.kde.org/pmoue241d
Thanks,
Milian Wolff
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150423/e830091a/attachment.html>
More information about the Kde-frameworks-devel
mailing list