[Differential] [Request, 6 lines] D3383: Use KWin's KSharedConfigPtr in ScriptedEffect
    graesslin (Martin Gräßlin) 
    noreply at phabricator.kde.org
       
    Wed Nov 16 12:46:37 UTC 2016
    
    
  
graesslin created this revision.
graesslin added reviewers: KWin, Plasma.
Restricted Application added a project: KWin.
Restricted Application added subscribers: kwin, plasma-devel.
REVISION SUMMARY
  So far ScriptedEffect used EffectsHandler::effectConfig to get the
  KConfigGroup for the ScriptedEffect. This has the disadvantage that the
  config file name is hardcoded to kwinrc in EffectsHandler::effectConfig.
  
  Inside KWin a KSharedConfigPtr is used which can point to somwhere else
  than kwinrc. If that was the case the ScriptedEffects were not able to
  pick up this customized config and instead continued to read values from
  kwinrc.
  
  With this change the ScriptedEffects use the KSharedConfigPtr provided
  by KWin. Thus in e.g. autotests we can use the general way to configure
  the effects and don't need to write to the config.
REPOSITORY
  rKWIN KWin
BRANCH
  scripted-effect-ksharedconfig
REVISION DETAIL
  https://phabricator.kde.org/D3383
AFFECTED FILES
  autotests/integration/effects/translucency_test.cpp
  scripting/scriptedeffect.cpp
EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/
To: graesslin, #kwin, #plasma
Cc: plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20161116/3e75e79f/attachment.html>
    
    
More information about the Plasma-devel
mailing list