Fix some leaks in the tests in the process.. Fixes: . Approved by Daniel van Vugt.

callback. (LP: #960652). Fixes: https://bugs.launchpad.net/bugs/960652. Approved by Daniel van Vugt.

Reduced the scope of the variable 'icccm_version', removed a break that can never be executed, added the variables 'frameType', 'frameState' and 'frameActions' to decor_quads_to_property in the #else branch in KWD::Switcher::updateWindowProperties () as well and simplified calc_button_size (decor_t *d) in cairo.c.. Fixes: https://bugs.launchpad.net/bugs/1030473. Approved by Daniel van Vugt.

 Commands addon desripion typo Buning -> binding. Fixes: https://bugs.launchpad.net/bugs/1033085. Approved by MC Return, Omer Akram.

Redeclared and reassigned frameType, frameState, frameActions unsigned int variables andalso added those to decor_quads_to_property in the #else branch in KWD::Switcher::updateWindowProperties () (switcher.cpp)

Simplified updateWindowProperties () by removing redundant declaration of the variables 'frameType', 'frameState' and 'frameActions'

Optimized performance and style following suggestions reported by cppcheck:

1. Used !xxxx.empty() instead of xxxx.size(), because it can be faster. xxxx.size() can take linear time while xxxx.empty() is guaranteed to take constant time.

2. Defensive programming: Reordered array-out-of-bound condition checks such as '(a[i] && i < 10)' to '(i < 10 && a[i])'. That way the array will not be accessed, because the index is checked first.. Fixes: . Approved by Daniel van Vugt, Sam Spilsbury.

Optimized performance and style following suggestions reported by cppcheck:

1. Reduced the scope of various variables.

2. Used prefix ++ operators for non-primitive types, because those can be more efficient than post-increment. Post-increment usually keeps a copy of the previous value, adds extra code and is slower.. Fixes: https://bugs.launchpad.net/bugs/1030473. Approved by Daniel van Vugt, Sam Spilsbury.

ParseArguments isn't installed, having it around is misleading, move its functionality to CompizCommon which is installed so that we don't cause cascading build failures everywhere else .... Fixes: . Approved by Daniel van Vugt.

installed by default

Adds a COVERAGE argument to compiz_discover_tests so clients can specify which libraries the tests actually cover (so that coverage reports are correctly generated for our various utility libs). Fixes: . Approved by Daniel van Vugt.

Fixed possible performance problem by changing 'cfs->filtersFunctions.size ()' to 'cfs->!filtersFunctions.empty ()' inside if statement

Fixed possible performance problem by changing 'if (points.size ())' to 'if (!points.empty ())' (x2)

Fixed possible performance problem by changing 'if (animations.size ())' to 'if (!animations.empty ())'

Fixed possible performance problem by changing '(mGroups.size ())' to '(!mGroups.empty ())' in if statements (x2)

Fixed possible performance problem by changing '(passive.size ())' to 'if (!passive.empty ())' in if statement

Fixed possible performance problem by changing 'if (slotAreas.size ())' to 'if (!slotAreas.empty ())'

Fixed possible performance problem by changing 'if (damageThumbs.size ())' to 'if (!damageThumbs.empty ())'

Fixed possible performance problem by replacing 'backgroundsPrimary.size()' with '!backgroundsPrimary.empty' in if statements (x2)

Fixed possible performance problem by changing 'if (screenInfo.size ())' to 'if (!screenInfo.empty ())'

Fixed style problem by reordering conditions check and checking first that the array index is within limits, before accessing it

Replaced the magic number assignment of the variable 'lastX' by 'float lastX = std::numeric_limits <float>::min ()'