[Differential] [Request, 89 lines] D1331: Provide the platform() -> AbstractBackend* in KWin::Application

graesslin (Martin Gräßlin) noreply at phabricator.kde.org
Wed Apr 6 15:36:26 UTC 2016


graesslin created this revision.
graesslin added reviewers: Plasma, sebas.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  This is the first change in a refactoring series. The aim is to:
  
  - rename AbstractBackend to Platform
  - move backends/ to plugins/platforms/
  - don't bind platforms to Wayland only
  - provide a platform plugin for "normal" X11
  - share more code between X11 and Wayland
  
  This change moves the platform/backend from waylandServer to Application.
  The init of the plugin happens directly in the Application from the
  KPluginMetaData. There is no need to externally init it and set the
  parent.
  
  WaylandServer::backend() currently just delegates to
  kwinApp()->platform(), the idea is to drop this method completely.
  
  The test infrastructure is also adjusted to this change.

TEST PLAN
  kwin_wayland still works, all tests pass

REPOSITORY
  rKWIN KWin

BRANCH
  platform-application

REVISION DETAIL
  https://phabricator.kde.org/D1331

AFFECTED FILES
  abstract_backend.cpp
  autotests/wayland/kwin_wayland_test.cpp
  main.cpp
  main.h
  main_wayland.cpp
  wayland_server.cpp
  wayland_server.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: graesslin, Plasma, sebas
Cc: plasma-devel, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160406/91eadba6/attachment.html>


More information about the Plasma-devel mailing list