Re: Review Request 113586: More general warning cleanups
--- 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43003 --- This review has been submitted with commit 7d21b11b3cbb464e4ef9477157e84a411df59fd4 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43008 --- This review has been submitted with commit 66ada97096ef1b01af247ebcd45600bd748a42ed by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43006 --- This review has been submitted with commit dc2ee505b6dbc3e11b949566ed24eac307a6a5eb by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43010 --- This review has been submitted with commit 8404a524f2f9902144015f7247500f9ed8e7ec2e by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43002 --- This review has been submitted with commit 0c6dad468d06d45a68ce08e47563d4b6825a2685 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43006 --- This review has been submitted with commit 4fa0949c5a60ed9d282735552ea5addf5aff6da3 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43009 --- This review has been submitted with commit 0c5d856cade6fd0c06d239744c97a4223cb9fe16 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43011 --- This review has been submitted with commit 7fcde34e2cb197250e46e935d6948a1582955567 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43005 --- This review has been submitted with commit 9b07bdb0fcf04ea7fa31a27f8cc1cdcceb3cf293 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43007 --- This review has been submitted with commit cf4ca76056d95f4a214b993e041c503393f65c31 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review43004 --- This review has been submitted with commit 586362f10e1a34f30df4d519b9ae62041222b7b6 by Alex Merry to branch frameworks. - Commit Hook On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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
Re: Review Request 113586: More general warning cleanups
--- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/113586/#review42959 --- Ship it! Ship It! - Kevin Ottens On Nov. 3, 2013, 5:24 p.m., Alex Merry wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/113586/ > --- > > (Updated Nov. 3, 2013, 5:24 p.m.) > > > 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