D10554: Change include namespace from KRunner/ to Plasma/, to match C++ ns
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Thu Feb 15 19:58:50 UTC 2018
kossebau created this revision.
kossebau added a reviewer: broulik.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
kossebau requested review of this revision.
REVISION SUMMARY
It has been a little bit confusing that the prefix dir of the include does
not match the C++ namespace, as elsewhere.
TEST PLAN
Existing runner code (e.g. plasma-workspace/runners) still builds,
with warnings in the log.
Runner code ported to new include path builds, without warnings in the log.
REPOSITORY
R308 KRunner
BRANCH
fixincludenamespacetoplasma
REVISION DETAIL
https://phabricator.kde.org/D10554
AFFECTED FILES
src/CMakeLists.txt
src/abstractrunner.h
src/dbusrunner_p.h
src/dbusutils_p.h
src/declarative/runnermodel.cpp
src/declarative/runnermodel.h
src/declarative/runnermodelplugin.cpp
src/querymatch.h
src/runnercontext.h
src/runnermanager.h
src/runnersyntax.h
templates/runner/CMakeLists.txt
templates/runner/src/%{APPNAMELC}.h
To: kossebau, broulik
Cc: #frameworks, michaelh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180215/aa331851/attachment.html>
More information about the Kde-frameworks-devel
mailing list