----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/109524/#review29336 -----------------------------------------------------------
kdecore/io/kdebug.cpp <http://git.reviewboard.kde.org/r/109524/#comment21920> This keeps an unused argument around for nothing. The code about coloring (both inside that method and in the caller) should be removed. I can do that if you want, but in any case, don't commit "silencing the issue". kdeui/dialogs/kaboutapplicationpersonmodel_p.cpp <http://git.reviewboard.kde.org/r/109524/#comment21921> See this line? The variable is used! But only if HAVE_ATTICA, which is clearly off for you. The fix is not to remove the variable, but to put it under #if HAVE_ATTICA. - David Faure On March 16, 2013, 6:16 p.m., Miquel Canes Gonzalez wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/109524/ > ----------------------------------------------------------- > > (Updated March 16, 2013, 6:16 p.m.) > > > Review request for KDE Frameworks and Aleix Pol Gonzalez. > > > Description > ------- > > Remove some compile warnings like unused variables, reordering initializers, > uint -> int... > > > Diffs > ----- > > interfaces/kimproxy/library/kimproxy.cpp 66e263f > kdecore/io/kdebug.cpp 6fd7584 > kdeui/dialogs/kaboutapplicationpersonmodel_p.cpp 717a6cd > kdeui/dialogs/kedittoolbar.cpp 401a292 > kdeui/dialogs/kinputdialog.cpp 28ce074 > kdeui/dialogs/kshortcutseditordelegate.cpp d458106 > kdeui/kernel/kstyle.cpp 681b940 > kdeui/tests/kreplacetest.h 72674eb > kdeui/tests/kreplacetest.cpp c59bd76 > kdeui/util/kmodifierkeyinfoprovider_dummy.cpp d456e82 > kdeui/widgets/kdatetable.cpp 4d24290 > kdeui/widgets/kkeysequencewidget.cpp 28eb47e > kio/bookmarks/kbookmarkimporter_opera.cc d39f7a4 > kio/kio/ksambashare.cpp 0239af5 > kio/tests/kbookmarktest.cpp 3a0c742 > kioslave/file/file.cpp d57517b > knewstuff/knewstuff3/ui/itemsgridviewdelegate.cpp eb72240 > staging/ki18n/src/klocalizedstring.cpp 452dd59 > staging/kwidgets/src/icons/kiconengine.cpp 3e5fcc0 > tier1/itemmodels/src/kdescendantsproxymodel.cpp 8ee0702 > tier1/itemmodels/src/kselectionproxymodel.cpp f002f44 > tier1/kcodecs/autotests/kcharsetstest.cpp 0c7aac1 > tier1/kjs/src/kjs/array_object.cpp b726c09 > tier1/kjs/src/kjs/bytecode/machine.cpp.in a434c36 > tier1/kwindowsystem/src/kxutils.cpp f3e276a > tier1/solid/src/solid/backends/upnp/upnpinternetgateway.cpp 2c4ca85 > > Diff: http://git.reviewboard.kde.org/r/109524/diff/ > > > Testing > ------- > > compile again with less warnings. > > > Thanks, > > Miquel Canes Gonzalez > >
_______________________________________________ Kde-frameworks-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
