~compiz-team/compiz-core/0.9.7

Viewing all changes in revision 3010.

  • Committer: smspillaz
  • Date: 2012-02-20 12:16:31 UTC
  • mfrom: (3003.3.8 compiz-core.fix_931927)
  • Revision ID: sam.spilsbury@canonical.com-20120220121631-9mxmfwoeo80orp2h
Only call removeAction on the CompOption destructor and not the CompOption::Value
destructor, since plugins that make copies of CompOption::Value can cause actions to
be added through setOptionForPlugin and then those actions will be removed when the
value temporary goes away.

The action adding and removing only happens within the bounds of CompOption anyways, so
its its more appropriate to have it in its destructor.

Of course, this brings up another issue, which is that CompOption should be noncopyable
but this opens up a whole another can of worms.

How this even worked before is beyond me...

Merge lp:~smspillaz/compiz-core/compiz-core.fix_931927 into lp:compiz-core

expand all expand all

Show diffs side-by-side

added added

removed removed

Lines of Context: