Review Request 117271: KIconDialog: do not connect to a signal that does not exist

Alex Merry alex.merry at kde.org
Tue Apr 1 09:44:23 UTC 2014


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

(Updated April 1, 2014, 9:44 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks and Christoph Feck.


Repository: kiconthemes


Description
-------

Add a new test that just checks kicondialog

The old kicondialogtest actually tested KIconButton.  That is now
renamed to kiconbuttontest, and there is a new kicondialogtest that
tests KIconDialog.

KIconDialog: do not connect to a signal that does not exist

QDialog::finished(int) does what we want, so use that instead.


Diffs
-----

  tests/CMakeLists.txt b3b4c90a9ffbc9df81dd008f9cfbc668866fafe3 
  tests/kiconbuttontest.cpp PRE-CREATION 
  src/kicondialog.cpp 4b241c1591b321486c925ef5e61f09bc7b6ddd1c 
  tests/kicondialogtest.cpp 66432ba878a4133ea3210089d3fdfbaec145bf1b 

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


Testing
-------

Tested with the new kicondialogtest, and everything seems to be in order.  The old kiconbuttontest is not working properly, apparently because QIcon::fromTheme() is not working.


Thanks,

Alex Merry

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


More information about the Kde-frameworks-devel mailing list