lp:~compiz-team/compiz/new-gmock
- Get this branch:
- bzr branch lp:~compiz-team/compiz/new-gmock
Branch merges
- PS Jenkins bot (community): Approve (continuous-integration)
- Brandon Schaefer (community): Approve
-
Diff: 1136 lines (+568/-164)20 files modifiedCMakeLists.txt (+1/-4)
cmake/GoogleTest.cmake (+0/-20)
cmake/src/compiz/compiz_discover_gtest_tests.cpp (+5/-3)
compizconfig/libcompizconfig/tests/compizconfig_test_ccs_mock_backend_conformance.cpp (+11/-10)
compizconfig/libcompizconfig/tests/compizconfig_test_ccs_object.cpp (+21/-10)
compizconfig/libcompizconfig/tests/compizconfig_test_ccs_setting.cpp (+0/-2)
compizconfig/mocks/libcompizconfig/CMakeLists.txt (+10/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_setting_stub.cpp (+425/-0)
compizconfig/mocks/libcompizconfig/compizconfig_ccs_setting_stub.h (+44/-0)
compizconfig/tests/CMakeLists.txt (+2/-1)
compizconfig/tests/compizconfig_backend_concept_test.cpp (+31/-16)
compizconfig/tests/compizconfig_ccs_list_wrapper.cpp (+12/-17)
compizconfig/tests/compizconfig_ccs_list_wrapper.h (+4/-5)
plugins/composite/src/pixmapbinding/tests/CMakeLists.txt (+0/-12)
plugins/grid/src/grabhandler/tests/CMakeLists.txt (+0/-12)
plugins/opengl/src/doublebuffer/tests/CMakeLists.txt (+0/-12)
plugins/opengl/src/glxtfpbind/tests/CMakeLists.txt (+0/-12)
plugins/resize/src/logic/tests/CMakeLists.txt (+0/-12)
plugins/wall/src/offset_movement/tests/CMakeLists.txt (+0/-12)
src/region/tests/test-region.cpp (+2/-4)
Related bugs
Related blueprints
Branch information
Recent revisions
- 3763. By Andrea Azzarone
-
Merge Sam's branch.
- 3759. By MC Return
-
Thumbnail, cleanup:
Merged if condition checks.
Declaration of variables when they are needed (C++ style).
Removed redundant brackets.
Removed useless "/* Could someone please explain how this works */"
comment.
Removed extern const unsigned short TEXT_DISTANCE, because this
variable does not exist anymore (value is configurable already).
Changed if (something > 0.0) to if (something), 10 times.
Changed pointedWin = 0; to pointedWin = NULL;.
Used centerX () and centerY () abstractions from
compiz::window: :Geometry to determine the icon center coordinates.
Added and removed newlines, if appropriate.
Fixed indentation.Thumbnail, code speedup:
Return ASAP, do not calculate stuff you might not need.
No need for WIN_W (w) and WIN_H (w) macros, removed those and
replaced them with the local variables int winWidth, int winHeight,
int dockWidth and int dockHeight.
No need for WIN_X (w) and WIN_Y (w) macros, replaced those with
int dockX and int dockY.
Introduced int tHeight = thumb.height;, int tWidth = thumb.width;,
int halfTWidth = tWidth / 2; and int halfTHeight =
tHeight / 2; and used those variables in the following calculations.
Use igMidPoint[0] and igMidPoint[1] as arguments in the
screen->outputDeviceFo rPoint (arg1, arg2) function, instead of re-
calculating the arguments again.Thumbnail, GL speedup:
Introduced
GLfloat wxPlusWidth = wx + width;,
GLfloat wyPlusHeight = wy + height;,
GLfloat wxPlusWPlusOff = wxPlusWidth + off;,
GLfloat wyPlusHPlusOff = wyPlusHeight + off;,
GLfloat wxMinusOff = wx - off; and
GLfloat wyMinusOff = wy - off;
and used those coordinates in the vertexData arrays. No need to re-
calculate those values multiple times.
Just query optionGetShowDelay () once and use int showDelay later.
We just enable blending if it is currently disabled and disable
blending only, if it was disabled before.Approved by PS Jenkins bot, MC Return, Sami Jaktholm, Sam Spilsbury.
- 3758. By Sami Jaktholm
-
Port dbus introspection to compiz 0.9.
- move xml creation to a separate class (IntrospectionResponse) for easier
memory management (allocate buffer and writer in ctor, free in dtor).
- move duplicated response sending code to a separate method that takes
IntrospectionResponse and sends the resulting xml.
- Refactor handle*IntrospectMessa ge to work with compiz 0.9 interfaces. This also fixes the broken list method which was a result of logic error. The
code to invoke list handler was never reached.This fixes most of the issues noted in bug 749084. Fixes: https:/
/bugs.launchpad .net/bugs/ 749084. Approved by PS Jenkins bot, MC Return.
- 3757. By Sami Jaktholm
-
Expo: Cast denominators of integer divisions to floats before performing a
matrix translation to viewports with the results. Otherwise rounding
errors in cases the results are not an integers cause the viewports to
be painted offscreen. Fixes: https://bugs.launchpad .net/bugs/ 438580. Approved by PS Jenkins bot, MC Return.
- 3756. By Michail Bitzes
-
Plugin wizard ported to OpenGL|ES.
Use GLVertexBuffer.
Enable building for GLES. Enable architectures armel and armhf.(LP: #1196003). Fixes: https:/
/bugs.launchpad .net/bugs/ 1196003. Approved by PS Jenkins bot, MC Return.
- 3755. By Sami Jaktholm
-
Expo plugin:
1) Always fetch the dndButton action in dndFini.The action given to dndFini might be a key or edge binding action if expo
was terminated during window DnD. We shouldn't set their state to
CompAction::StateInitButto n or the bindings break. 2) Pass NULL action to termExpo if invoked from handleEvent
Previously we passed the action of expoKey to termExpo which in turn passed
it to dndFini. DndFini changed the action state to
CompAction::StateInitButto n that broke the keybinding. Now that dndFini fetches the dndButton action itself, there's no need to
fetch it in handleEvent too as dndFini is the only one using the action. Fixes: https://bugs.launchpad .net/bugs/ 1131106. Approved by PS Jenkins bot, MC Return.
- 3754. By Sam Spilsbury
-
Add a simple script for making releases.
Approved by MC Return, PS Jenkins bot.
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:compiz/0.9.10