hpereiradacosta added inline comments.

INLINE COMMENTS

> breezeshadowhelper.cpp:145
>          }
> +    };
>  

ok. I know this whole chunc of code is to make it look like css, but this is 
ugly sorry.
The fact that you need to repeat all the member names for each new set of 
parameters is not maintainable in the long run. Please add a proper constructor 
to the structure (with three params), and use this to initialize the various 
sets of parameters. 
Idem in the kdecoration.

REPOSITORY
  R31 Breeze

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

To: zzag, #breeze, #vdg, hpereiradacosta
Cc: ngraham, plasma-devel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to