Review Request 127214: Remove some warnings

David Edmundson david at davidedmundson.co.uk
Sun Feb 28 23:44:57 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127214/#review92873
-----------------------------------------------------------


Ship it!




Ship It!

- David Edmundson


On Feb. 28, 2016, 11:36 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127214/
> -----------------------------------------------------------
> 
> (Updated Feb. 28, 2016, 11:36 p.m.)
> 
> 
> Review request for KDE Frameworks and Plasma.
> 
> 
> Repository: plasma-framework
> 
> 
> Description
> -------
> 
> * Unused variables
> * Unused code
> * Unused arguments
> * Unnecessarily complex code
> * Initialization order
> * Duplicated properties
> 
> It would be _very_ good if we could get rid of Plasma::Package usage within Plasma Framework. I'm quite confident most of these warnings would have been fixed already if it wasn't because we're flooded with deprecated API usage warnings.
> 
> 
> Diffs
> -----
> 
>   autotests/pluginloadertest.cpp b75dd03 
>   src/declarativeimports/calendar/daysmodel.cpp 1faf43b 
>   src/declarativeimports/calendar/eventpluginsmanager.cpp 343d586 
>   src/declarativeimports/core/quicktheme.h b8017d2 
>   src/declarativeimports/platformcomponents/application.cpp bdcc9b4 
>   src/plasma/private/applet_p.cpp 1c0515a 
>   src/plasma/private/theme_p.cpp b18e1fa 
>   src/plasma/service.cpp a512569 
>   src/plasmapkg/plasmapkg.cpp 138b2fe 
> 
> Diff: https://git.reviewboard.kde.org/r/127214/diff/
> 
> 
> Testing
> -------
> 
> Builds, tests pass.
> 
> 
> Thanks,
> 
> Aleix Pol Gonzalez
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160228/592e6266/attachment.html>


More information about the Plasma-devel mailing list