Review Request 109524: Remove some compile warnings

[Miquel Canes Gonzalez]

> On April 8, 2013, 6:30 p.m., David Faure wrote:
> > staging/ki18n/src/klocalizedstring.cpp, line 1349
> > <http://git.reviewboard.kde.org/r/109524/diff/2/?file=132245#file132245line1349>
> >
> >     This one looks like a bug to me. Just ignoging plural handling sounds like a bug...

It's ok with "uncomment" it again? Maybe adding a TODO or FIXME message?


- Miquel


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109524/#review30713
-----------------------------------------------------------


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/20130409/00cb8249/attachment.html>