Review Request 128349: Silence warning about widget already having a layout

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sat Jul 2 21:43:18 UTC 2016


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

Review request for KDE Frameworks and Martin Klapetek.


Repository: knotifications


Description
-------

Setting the parent of a QLayout manually is usually wrong.


Diffs
-----

  src/kpassivepopup.cpp 4561eb9 

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


Testing
-------

«QLayout: Attempting to add QLayout "" to QWidget "", which already has a layout» does not appear anymore, but the passive popup does.


Thanks,

Martin Tobias Holmedahl Sandsmark

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


More information about the Kde-frameworks-devel mailing list