----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/ -----------------------------------------------------------
(Updated Nov. 4, 2013, 7:58 p.m.) Status ------ This change has been marked as submitted. Review request for KDE Frameworks. Repository: kdelibs Description ------- Another bunch of fairly obvious warning fixes; this should be pretty much all the noise that can easily be dealt with. Everything else requires either more consideration or the enabling of KDE_NO_DEPRECATED. Speaking of which, several of the fixes get us closer to kf5 actually compiling with that defined. Remove call to KUrlRequester::fileDialog() from test This is (a) unnecessary and (b) deprecated. KIO::RenameDialog: use KIO::suggestName instead of local method Also, wrap the local implementation of suggestName in KDE_NO_DEPRECATED, like its declaration. Remove use of deprecated form of KRecentDocument::add KCategoryDrawerV3 -> KCategoryDrawer in KPluginSelector Mark KPluginInfo::fromServices as deprecated It is simply the multiple-at-once version of the deprecated KPluginInfo constructor that takes a KService. Mark unused (except for debugging) args with Q_UNUSED Wrap test of deprecated method in KDE_NO_DEPRECATED The test will not compile with KDE_NO_DEPRECATED set... Do not compile KComboBox::setUrlDropsEnabled if KDE_NO_DEPRECATED is set It calls a deprecated method, and its declaration is wrapped in KDE_NO_DEPRECATED as well. KAuth: handle QtTraceMsg messages We just treat them like debug messages. Fix warnings in Solid UDisks2 backend Make it explicit that UDisks2 does not support Keyboard or PointingDevice. Also make the switch statement easier to read. Wrap assertion function in #ifndef QT_NO_DEBUG This function is only used in a Q_ASSERT, so prevent it from being compiled if assertions are disabled. Diffs ----- staging/kio/src/filewidgets/kencodingfiledialog.cpp c6c159fb016b8b8b1a7caa20d814f62e1989bb57 staging/kio/src/filewidgets/kfilewidget.cpp 48215315ea1a4c335992185735b8a717b8d5add6 staging/kio/src/widgets/renamedialog.cpp 0830c90a199a780e79966d3220e595866c8f7b96 staging/kio/tests/kurlrequestertest_gui.cpp 948085f090e3c388246ae7b11ae348a089a5893c tier1/itemmodels/src/kselectionproxymodel.cpp df83ae6c51edcdf01d1d16b351a63507d6a5a982 tier1/solid/src/solid/backends/udisks2/udisksdeviceinterface.h 43a1b6f896834e9fe9e94e9580ff5845036aa754 tier2/kauth/src/backends/dbus/DBusHelperProxy.cpp ee44d930eddd647e8ad5d1cf20097a1b8a719efa tier2/kauth/src/kauthhelpersupport.cpp 8eaa97dead64e745ab9e4e674ffa02f3d4b72bdb tier2/kcompletion/src/kcombobox.cpp 4043ea5448a67c01751cc849d56c3fdd4d61035c tier2/kcompletion/tests/klineedittest.cpp 233bdd3afd8b489e9f32f81ac89b57114948c034 tier3/kservice/src/kbuildsycoca/kmimeassociations.cpp dfd2a6d83c637215219c7fd928dcd4fb37097ddf tier3/kservice/src/services/kplugininfo.h 2555ef27d1ccc2fa58a905175bdcefab4e6491ea tier3/kservice/src/services/kplugininfo.cpp e19ae7e3fcadfbcadfcecea3795a0d9afad09753 tier3/xmlgui/src/kxmlguifactory.cpp d2980f45fe8e32f1b4aa535661eb002c69969698 tier4/kcmutils/src/kpluginselector.cpp 768dbf33b136b79f3cab4c6f09e84d2a4378a022 tier4/kcmutils/src/kpluginselector_p.h de42f03a99ba35e4c43927a7e2c07fc3a541de5a Diff: http://git.reviewboard.kde.org/r/113586/diff/ Testing ------- Builds; tests pass. Thanks, Alex Merry
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel