lp:~mc-return/compiz/compiz.merge-fix1182794-screenshot-not-compatible-with-unityshell

Branch merges

Propose for merging

No branches dependent on this one.

On hold for merging into lp:compiz/0.9.10

Sam Spilsbury: Needs Information on 2013-05-22

Diff: 128 lines (+17/-24)

2 files modified

plugins/screenshot/src/screenshot.cpp (+7/-20)
plugins/screenshot/src/screenshot.h (+10/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

3727. By MC Return on 2013-05-22

Removed redundant damage call

3726. By MC Return on 2013-05-22

Fixed comment

3725. By MC Return on 2013-05-22

Fixed indentation

3724. By MC Return on 2013-05-22

Added newline

3723. By MC Return on 2013-05-22

Removed line committed by accident

3722. By MC Return on 2013-05-22

Screenshot plugin:
We have to redraw and damage the painted region the whole time,
if unityshell is enabled, not only when it's size changed
Otherwise the constant whole-screen damaging of Unity makes the
screenshot selection box disappear

Indentation fixes

Note:
Screenshot is less efficient now, so this should probably be
reverted, once unityshell has learned to correctly damage just
repainted regions

(LP: #1182794)

3721. By Sami Jaktholm on 2013-05-21

Fade: Fix a typo that causes fade to constantly damage windows with a modified
      saturation. Fixes: https://bugs.launchpad.net/bugs/1159054.

Approved by PS Jenkins bot, Stephen M. Webb, MC Return.

3720. By MC Return on 2013-05-20

Cube code cleanup:

Replaced 0xffff with GLfloat MaxUShortFloat = static_cast <GLfloat>
(std::numeric_limits <unsigned short>::max ());.
Use prefix instead of postfix de- and increments.
Declaration and assignment of variables in one line.
Declare variables shortly before you need them.
Merged if condition checks.
Added missing breaks (just a style issue).
Added and removed brackets.
Added and removed newlines.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.

3719. By MC Return on 2013-05-20

Commands, minor code cleanup:

Declaration and assignment of variables in one line.
Return ASAP.
Added and removed newlines.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.

3718. By MC Return on 2013-05-18

ccp code cleanup:

Declare variables if you need them, do not declare them if you return.
Declare CCSSettingValue *val; outside the foreach loop.
Use prefix instead of postfix increments.
Merged if condition checks.
Declaration and assignment of variables in one line.
Removed redundant brackets.
Added and removed newlines.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.