Diffusion KWin fa5242b3eefd

[effects] Use more effectData() in BuiltInEffects

Authored by zzag on Jun 18 2018, 12:25 PM.

Description

[effects] Use more effectData() in BuiltInEffects

Summary:
While BuiltInEffects has effectData() function, many functions repeat
s_effectData.at(index(effect)), which is what effectData() is doing.

By using effectData(), we'll get rid of those repetitions and maybe make
easier transition to other underlying data structure that stores metadata
for builtin effects.

Test Plan: Compiles, all enabled builtin effects are loaded and working.

Reviewers: KWin, davidedmundson

Reviewed By: KWin, davidedmundson

Subscribers: kwin

Tags: KWin

Differential Revision: https://phabricator.kde.org/D13587

(cherry picked from commit 7bfaa6e913dff867e6b6c3d4b451eb274683ca87)

Details

Committed
broulikAug 30 2018, 11:32 AM
Reviewer
KWin
Differential Revision
D13587: [effects] Use more effectData() in BuiltInEffects
Parents
R108:a5b820ee7e1c: SVN_SILENT made messages (.desktop file) - always resolve ours
Branches
Unknown
Tags
Unknown