Review Request 116959: Remove KDE 4 references from kcmutils

Alex Merry alex.merry at kde.org
Mon Mar 24 11:44:55 UTC 2014


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

(Updated March 24, 2014, 11:44 a.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Repository: kcmutils


Description
-------

KCModuleProxy: remove deprecated forwarding methods

These have been deprecated since at least kdelibs 4.0, and LXR says the
only user is KCMultiDialog (since I fixed the call in systemsettings).

KCModuleContainer: tidy up overridden methods

Use the "@reimp" macro in the docs and use Q_DECL_OVERRIDE.

KCModuleContainer: move modulelist variables to private object

According to LXR, these were not used by anything.

Remove TODO file

This is full of API changes for KDE 4.  Some were done, and others not
(but would probably be considered too disruptive for KF5).


Diffs
-----

  TODO 82f28e12903e51279ba12871c7b184a97c79a12a 
  src/kcmodulecontainer.h 68aaeb008195aae154b1fbaf44c6f620753884b0 
  src/kcmodulecontainer.cpp 5b698ab2e320654a9ff3396c22b8aa239d4395d7 
  src/kcmoduleproxy.h d80d519dbf6e4015864d61daf65f67b99c136f26 
  src/kcmoduleproxy.cpp c58b68ccb1716493d0537fbfe5a89733ff72e22b 
  src/kcmultidialog.cpp cc0c8d8228ad03e1b4de62152bf7bc7ecdc1f5ee 

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


Testing
-------

Compiles, installs.


Thanks,

Alex Merry

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


More information about the Kde-frameworks-devel mailing list