Review Request 125686: Drop the Dashboard Effect

Martin Gräßlin mgraesslin at kde.org
Sun Oct 18 11:23:31 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/125686/
-----------------------------------------------------------

Review request for kwin and Plasma.


Repository: kwin


Description
-------

Basically dead code given that Plasma doesn't have a dedicated dashboard
mode anymore and also doesn't set the required window role for it to
work.

By deleting we save one string comparison for each newly opened window.


Diffs
-----

  effects/CMakeLists.txt 3e88bfa754bf1581d5e951b602838746a3826acd 
  effects/dashboard/CMakeLists.txt 81c5ee1ff878904ab06153e2e316775218d799ae 
  effects/dashboard/dashboard.h 9c46f12ebf5c222ba91a9f153b1f84671c25de82 
  effects/dashboard/dashboard.cpp 42651ecce1b8a3189e5b47382b4915b76adb1aec 
  effects/dashboard/dashboard.kcfg 9fbdf09c5a45181f3099ef1e7a8777416f14d6b9 
  effects/dashboard/dashboard_config.h aaa09d6fb9f3b82faa7a073826ca725e0739886e 
  effects/dashboard/dashboard_config.cpp 75696bd07b45ff0247fc39a8ca058bd392f2e681 
  effects/dashboard/dashboard_config.desktop de25b623bcb79012f9bbb3222c1ac1916dbfdcea 
  effects/dashboard/dashboard_config.ui fd9a19e0c5b979049fe7e3443974eef3286fb2f2 
  effects/dashboard/dashboardconfig.kcfgc 772ad03cbe6f5f213759d394a54eeede08411bf0 
  effects/effect_builtins.h b471cf077a2bb284ac448229c9967b00023d8842 
  effects/effect_builtins.cpp be508e6fea57be49a1bd320194f18aaeb1465f42 

Diff: https://git.reviewboard.kde.org/r/125686/diff/


Testing
-------

Compiles without the effect. Verified that "plasma-dashboard" is nowhere set in plasma-framework, plasma-workspace and plasma-desktop.


Thanks,

Martin Gräßlin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20151018/398d4eac/attachment.html>


More information about the Plasma-devel mailing list