Review Request 123479: Adapt to Qt 5.6 changes and prevent nullbytes in QStrings.

Milian Wolff mail at milianw.de
Thu Apr 23 17:19:22 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123479/
-----------------------------------------------------------

(Updated April 23, 2015, 5:19 p.m.)


Review request for KDE Frameworks, David Faure, Marc Mutz, and Volker Krause.


Changes
-------

extended an existing test to verify this bug is fixed


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 (updated)
-----

  autotests/kdirwatch_unittest.cpp 83b5b410e987fb45a08f8251ec65496c8074b077 
  src/lib/io/kdirwatch.cpp a75fae012c59021996b46845db2e97abb5526930 

Diff: https://git.reviewboard.kde.org/r/123479/diff/


Testing
-------

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/88d47a5d/attachment.html>


More information about the Kde-frameworks-devel mailing list