Review Request 123679: Fix build with Qt 5.5

Jan Kundrát jkt at kde.org
Fri May 8 11:48:35 UTC 2015


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

(Updated May 8, 2015, 11:48 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Stephen Kelly.


Changes
-------

Submitted with commit d598382536d8c9982bba3ee210869e50d1313dff by Jan Kundrát to branch master.


Repository: kitemmodels


Description
-------

Turns out that there's apparently no overload accepting a QPointer as
the first argument.

This is on Gentoo with qtbase e374ffc29c67493a51527117b55a53dfa5dd4267
and GCC 4.8.3.

Also make sure to tie the lifetime of the connection with `this` because
the lambda accesses our D-pointer, which means that it has to be
disconnected on destruction of `this`. Thanks to Milian Wolff for
catching this.

REVIEW: 123679


Diffs
-----

  src/kselectionproxymodel.cpp 0755fb1ca527dde8287ba2607d915a4e18c20134 

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


Testing
-------


Thanks,

Jan Kundrát

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150508/f1b411d0/attachment-0001.html>


More information about the Kde-frameworks-devel mailing list