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