Merge lp:~vanvugt/compiz/fix-CompOption-templates into lp:compiz/0.9.9

Proposed by Daniel van Vugt
Status: Merged
Approved by: Sam Spilsbury
Approved revision: 3418
Merged at revision: 3420
Proposed branch: lp:~vanvugt/compiz/fix-CompOption-templates
Merge into: lp:compiz/0.9.9
Diff against target: 29 lines (+8/-0)
2 files modified
include/core/option.h (+4/-0)
src/option.cpp (+4/-0)
To merge this branch: bzr merge lp:~vanvugt/compiz/fix-CompOption-templates
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Sam Spilsbury Approve
Review via email: mp+129796@code.launchpad.net

Commit message

Instantiate CompOption-related templates in core only. There's no need for
duplicates to be instantiated in plugins too.
(LP: #1066793) (LP: #1066803)

This not only reduces build times and will improve stability during plugin
reloads, but it also reduces the footprint of lib/compiz/* from 89MB to 75MB!

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'include/core/option.h'
--- include/core/option.h 2012-10-02 05:51:26 +0000
+++ include/core/option.h 2012-10-16 03:41:20 +0000
@@ -302,6 +302,10 @@
302 PrivateOption *priv;302 PrivateOption *priv;
303};303};
304304
305extern template class std::vector<unsigned short>;
306extern template class std::vector<CompOption::Value>;
307extern template class std::vector<CompOption>;
308
305namespace compiz {309namespace compiz {
306namespace detail {310namespace detail {
307311
308312
=== modified file 'src/option.cpp'
--- src/option.cpp 2012-10-02 05:51:26 +0000
+++ src/option.cpp 2012-10-16 03:41:20 +0000
@@ -38,6 +38,10 @@
38#include <boost/variant/static_visitor.hpp>38#include <boost/variant/static_visitor.hpp>
39#include <boost/type_traits/remove_const.hpp>39#include <boost/type_traits/remove_const.hpp>
4040
41template class std::vector<unsigned short>;
42template class std::vector<CompOption::Value>;
43template class std::vector<CompOption>;
44
41namespace45namespace
42{46{
43 CompOption::Value::Vector & emptyList ()47 CompOption::Value::Vector & emptyList ()

Subscribers

People subscribed via source and target branches