davidedmundson accepted this revision. davidedmundson added inline comments. This revision is now accepted and ready to land.
INLINE COMMENTS > geolocator.cpp:25 > +{ > + m_engine = dataEngine("geolocation"); > + meh, I'd rather we didn't have to link the rather heavy Plasma from KDED. I'm not going to make you change it, but in future hopefully we can just use the geo code directly. REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D5931 To: subdiff, #plasma, davidedmundson Cc: davidedmundson, HuShifang, leezu, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart