~ivenvd/compiz/compiz.fix_1189374

Viewing all changes in revision 3727.

Opacify code cleanup:

Declaration of local variables outside of loops.
Calculate activeOpacity and targetOpacity just once, not for each window.
Use prefix instead of postfix increments.
Merged if condition checks.
Declaration and assignment of local variables in one line.
Added and removed newlines.
Removed redundant brackets.
Fixed indentation.

Opacify, functionality fix:

Improved the toggling logic by setting isToggle in the constructor according to
the startup setting, then calling setFunctions (isToggle);,which then calls
screen->handleEventSetEnabled (os, os->isToggle); to fix Opacify ignoring the
"Toggle Opacify on by default" setting.

(LP: #787814). Fixes: https://bugs.launchpad.net/bugs/787814.

Approved by PS Jenkins bot, Sam Spilsbury.

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: