Review Request 121036: Convert util/ to Qt5 signal/slot syntax

Alexander Richardson arichardson.kde at gmail.com
Sat Nov 8 14:22:38 UTC 2014



> On Nov. 8, 2014, 3:19 nachm., Milian Wolff wrote:
> > util/objectlist.h, lines 85-87
> > <https://git.reviewboard.kde.org/r/121036/diff/1/?file=326724#file326724line85>
> >
> >     use a lambda to delegate to the function of the private class

That means we have to store all the connections so that we can disconnect again.


- Alexander


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


On Nov. 7, 2014, 6:34 nachm., Alexander Richardson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/121036/
> -----------------------------------------------------------
> 
> (Updated Nov. 7, 2014, 6:34 nachm.)
> 
> 
> Review request for KDevelop.
> 
> 
> Repository: kdevplatform
> 
> 
> Description
> -------
> 
> Convert util/ to Qt5 signal/slot syntax
> 
> 
> Diffs
> -----
> 
>   util/duchainify/main.cpp fb0ae318e40a80ce9dc60386e4672663acb1961e 
>   util/environmentconfigurebutton.cpp 23703e95ef1ebdb1f5d0bfa087dd9ca297f9dae1 
>   util/focusedtreeview.cpp 9050c30469a4834ab654128811905f3d0005adf5 
>   util/multilevellistview.cpp cb5f08c85bdcddcaafdac716566a8f7c78da0322 
>   util/objectlist.h 43ec0f474b7423de9b86eeb54928f660513492c7 
>   util/objectlist.cpp c4b01af408866004ca6f40bf1b6dc44eb9a8405e 
>   util/processlinemaker.cpp f95d3781832e63f4c3a50156ea91c6c4bbb05eb6 
>   util/projecttestjob.cpp 677c88b6750d5f59d6d9f32a288e5ee9f9249043 
>   util/commandexecutor.cpp 097868aa6044d5ce603626d1663f825a347dbe78 
>   util/activetooltip.cpp 0357842268d989d9f6c2008d5df239bd4a0f8432 
> 
> Diff: https://git.reviewboard.kde.org/r/121036/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alexander Richardson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20141108/a9892eea/attachment.html>


More information about the KDevelop-devel mailing list