lp:~sjakthol/compiz/fix-grid-abi-check

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:compiz/0.9.10 at revision 3716

PS Jenkins bot (community): Approve (continuous-integration) on 2013-05-18

Sam Spilsbury: Approve on 2013-05-18

Diff: 16 lines (+3/-3)

1 file modified

plugins/grid/src/grid.cpp (+3/-3)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1181461: Grid plugin fails to load in trunk

High

Fix Released

Link a bug report

Related blueprints

3716. By Sami Jaktholm on 2013-05-18

Grid: Check that composite and opengl ABIs are COMPIZ_COMPOSITE_ABI and
      COMPIZ_OPENGL_ABI instead of CORE_ABIVERSION.

3715. By MC Return on 2013-05-17

Titleinfo, minor code cleanup:

Declaration and assignment of local variables in one line.
Removed redundant brackets.
Added and removed newlines.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.

3714. By MC Return on 2013-05-17

Annotate plugin, code cleanup:

Use prefix instead of postfix increment.
Declaration and assignment of local variables in one line.
Removed redundant brackets.
Added and removed newlines if appropriate.
Fixed indentation.

Annotate plugin, fixes:

Use GL_LINE_LOOP to draw the rectangle outline as glRecti is not supported on GLES.
Simplified the coordinate calculation as the offset is not really needed here.

Approved by PS Jenkins bot, Sam Spilsbury.

3713. By MC Return on 2013-05-17

Mousepoll code cleanup:

Removed the unused variables MP_OPTION_MOUSE_POLL_INTERVAL
and MP_OPTION_NUM.
Declaration and assignment of variables in one line.
Added and removed newlines.
Removed redundant brackets.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.

3712. By Micheal Hsu on 2013-05-16

gtk-window-decorator: destroy action menu when any of the (close, min, max) buttons on the title bar is pressed. Fixes: https://bugs.launchpad.net/bugs/1101648.

Approved by PS Jenkins bot, Sam Spilsbury.

3711. By MC Return on 2013-05-16

EZoom plugin, code improvements:

Bail out of functions ASAP, do not calculate stuff you might not need.

Replaced: if (zoom_level == 1.0f && zooms.at (out).newZoom == 1.0f)
with: if (zoom_level == zooms.at (out).newZoom)
because we always want to get out of here if the target level is the same
as current zoom level, not only in the case both are 1.0f.

EZoom plugin, code cleanup:

Assignment and declaration of variables in one line, if possible.
Added and removed some newlines.
Merged if condition checks.
Indentation fixes.

Approved by PS Jenkins bot, Sam Spilsbury.

3710. By MC Return on 2013-05-16

Wallpaper .xml.in cleanup:

Better plugin description.
Improved option names.
Better wording of tooltips.
Removed redundant newlines.

Wallpaper code cleanup:

Use prefix instead of postfix increments.
Declaration and assignment of variables in one line.
Added some newlines to improve readability.
Removed redundant brackets.
Fixed indentation.

Approved by PS Jenkins bot, Sam Spilsbury.

3709. By Sam Spilsbury on 2013-05-16

Add a small compiz_autopilot_acceptance_tests binary based on gtest.

This will find autopilot and run a predefined list of tests, wrapped
up in the familiar google test format.

The predefined list is basically a bunch of tests which effectively
interact with the window manager.

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

Approved by PS Jenkins bot, Łukasz Zemczak, MC Return.

3708. By Marco Trevisan (Treviño) on 2013-05-15

GLScreen: break the iteration as soon as we've found a valid egl config.

Approved by PS Jenkins bot, Sam Spilsbury.

3707. By MC Return on 2013-05-15

Wall:
Fixed broken wall edge flipping functions:
"Edge Flip Pointer"
"Edge Flip Move"

Removed void WallScreen::updateScreenEdgeRegions () completely.
This function not needed hereat all, no other plugin
needs to re-define the screen edge regions by itself -
just core should do that.
The screen edge region updating also confused the edge flipping functionality.
case ConfigureNotify: just breaks out now
Added missing default: case and a break (just a style issue).

Minor cleanup in bool WallScreen::initiateFlip (...).
One declaration per line.

Note: "Edge Flip DnD" still seems to be broken, but will be fixed in a follow-up
proposal.

(LP: #771448, LP: #858845). Fixes: https://bugs.launchpad.net/bugs/771448, https://bugs.launchpad.net/bugs/858845.

Approved by PS Jenkins bot, Sam Spilsbury.