D11957: [potd dataengine] De-pimplify potd providers for more simple code
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Thu Apr 5 13:16:22 UTC 2018
kossebau created this revision.
kossebau added a reviewer: xuetianweng.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
kossebau requested review of this revision.
REVISION SUMMARY
There is no need to use pimpl for non-public-API classes.
While touching the code, also change to use consistently
modern signal-slot connections.
TEST PLAN
All plugins still build, load & work as before.
REPOSITORY
R114 Plasma Addons
BRANCH
depimplpotdproviderplugins
REVISION DETAIL
https://phabricator.kde.org/D11957
AFFECTED FILES
dataengines/potd/apodprovider.cpp
dataengines/potd/apodprovider.h
dataengines/potd/bingprovider.cpp
dataengines/potd/bingprovider.h
dataengines/potd/epodprovider.cpp
dataengines/potd/epodprovider.h
dataengines/potd/flickrprovider.cpp
dataengines/potd/flickrprovider.h
dataengines/potd/natgeoprovider.cpp
dataengines/potd/natgeoprovider.h
dataengines/potd/noaaprovider.cpp
dataengines/potd/noaaprovider.h
dataengines/potd/wcpotdprovider.cpp
dataengines/potd/wcpotdprovider.h
To: kossebau, xuetianweng
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180405/08cfd5ac/attachment.html>
More information about the Plasma-devel
mailing list