Review Request 109524: Remove some compile warnings
Kevin Ottens
ervin at kde.org
Wed Apr 10 21:46:52 UTC 2013
> On April 9, 2013, 6:10 a.m., Kevin Ottens wrote:
> > kdeui/tests/kreplacetest.cpp, line 57
> > <http://git.reviewboard.kde.org/r/109524/diff/1-2/?file=120092#file120092line57>
> >
> > In fact I'm not sure I get this one... It's fine for the slot to have only a QString (I see you introduce the three ints but then you don't use them).
>
> Miquel Canes Gonzalez wrote:
> I introduced the three ints because I removed it in the first revision of this review.
> You requested change it to Q_UNUSED instead of remove the three ints in the original review, to make it easier to re-enable in the futur.
Meh, you're right, I'm loosing my mind. :-)
Sorry about the noise on that one, feel free to ignore.
- Kevin
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109524/#review30735
-----------------------------------------------------------
On April 7, 2013, 10:26 p.m., Miquel Canes Gonzalez wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109524/
> -----------------------------------------------------------
>
> (Updated April 7, 2013, 10:26 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/kedittoolbar.cpp b03d2cb
> kdeui/dialogs/kinputdialog.cpp 28ce074
> kdeui/dialogs/kshortcutseditordelegate.cpp d458106
> kdeui/kernel/kstyle.cpp 681b940
> 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
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20130410/d20241a5/attachment.html>
More information about the Kde-frameworks-devel
mailing list