Merge lp:~vanvugt/compiz/fix-1067218 into lp:compiz/0.9.9

Proposed by Daniel van Vugt
Status: Merged
Approved by: Sam Spilsbury
Approved revision: 3421
Merged at revision: 3427
Proposed branch: lp:~vanvugt/compiz/fix-1067218
Merge into: lp:compiz/0.9.9
Diff against target: 44 lines (+7/-2)
3 files modified
plugins/opengl/CMakeLists.txt (+2/-2)
src/region/include/core/region.h (+3/-0)
src/region/src/region.cpp (+2/-0)
To merge this branch: bzr merge lp:~vanvugt/compiz/fix-1067218
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Sam Spilsbury Approve
Review via email: mp+129807@code.launchpad.net

Commit message

Templates relating to CompRegion should only be instantiated in core and
never in any plugins.
(LP: #1067218)

Note: This required removing -std=c++0x from the opengl plugin. That flag
was causing opengl to have slightly different symbol requirements to the
template instance in core. Not that it matters because -std=c++0x appears to
have been unused by opengl.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Sam Spilsbury (smspillaz) :
review: Approve
Revision history for this message
Sam Spilsbury (smspillaz) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugins/opengl/CMakeLists.txt'
--- plugins/opengl/CMakeLists.txt 2012-09-22 04:00:52 +0000
+++ plugins/opengl/CMakeLists.txt 2012-10-16 07:05:21 +0000
@@ -15,11 +15,11 @@
15include_directories (src/glxtfpbind/include)15include_directories (src/glxtfpbind/include)
1616
17if (USE_GLES)17if (USE_GLES)
18 compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD "-DUSE_GLES -std=c++0x" LIBRARIES ${OPENGLES2_LIBRARIES} ${INTERNAL_LIBRARIES} dl INCDIRS ${OPENGLES2_INCLUDE_DIR})18 compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD "-DUSE_GLES" LIBRARIES ${OPENGLES2_LIBRARIES} ${INTERNAL_LIBRARIES} dl INCDIRS ${OPENGLES2_INCLUDE_DIR})
19else (USE_GLES)19else (USE_GLES)
20 find_package (OpenGL)20 find_package (OpenGL)
21 if (OPENGL_FOUND)21 if (OPENGL_FOUND)
22 compiz_plugin(opengl PLUGINDEPS composite CFLAGSADD -std=c++0x LIBRARIES ${OPENGL_gl_LIBRARY} ${INTERNAL_LIBRARIES} dl INCDIRS ${OPENGL_INCLUDE_DIR})22 compiz_plugin(opengl PLUGINDEPS composite LIBRARIES ${OPENGL_gl_LIBRARY} ${INTERNAL_LIBRARIES} dl INCDIRS ${OPENGL_INCLUDE_DIR})
23 endif (OPENGL_FOUND)23 endif (OPENGL_FOUND)
24endif (USE_GLES)24endif (USE_GLES)
2525
2626
=== modified file 'src/region/include/core/region.h'
--- src/region/include/core/region.h 2012-05-21 06:43:20 +0000
+++ src/region/include/core/region.h 2012-10-16 07:05:21 +0000
@@ -32,6 +32,9 @@
32#include <core/rect.h>32#include <core/rect.h>
33#include <core/point.h>33#include <core/point.h>
3434
35class CompRegion;
36extern template class std::vector<CompRegion>;
37
35/**38/**
36 * A 2D region with an (x,y) position and arbitrary dimensions similar to39 * A 2D region with an (x,y) position and arbitrary dimensions similar to
37 * an XRegion. It's data membmers are private and must be manipulated with40 * an XRegion. It's data membmers are private and must be manipulated with
3841
=== modified file 'src/region/src/region.cpp'
--- src/region/src/region.cpp 2012-05-21 06:43:20 +0000
+++ src/region/src/region.cpp 2012-10-16 07:05:21 +0000
@@ -36,6 +36,8 @@
36#include <boost/foreach.hpp>36#include <boost/foreach.hpp>
37#define foreach BOOST_FOREACH37#define foreach BOOST_FOREACH
3838
39template class std::vector<CompRegion>;
40
39const CompRegion infiniteRegion (CompRect (MINSHORT, MINSHORT,41const CompRegion infiniteRegion (CompRect (MINSHORT, MINSHORT,
40 MAXSHORT * 2, MAXSHORT * 2));42 MAXSHORT * 2, MAXSHORT * 2));
41const CompRegion emptyRegion;43const CompRegion emptyRegion;

Subscribers

People subscribed via source and target branches