Merge lp:~smspillaz/compiz-libcompizconfig/compiz-libcompizconfig.fix_965220 into lp:compiz-libcompizconfig

Proposed by Sam Spilsbury
Status: Merged
Merged at revision: 430
Proposed branch: lp:~smspillaz/compiz-libcompizconfig/compiz-libcompizconfig.fix_965220
Merge into: lp:compiz-libcompizconfig
Diff against target: 14 lines (+1/-4)
1 file modified
plugin/ccp/src/ccp.cpp (+1/-4)
To merge this branch: bzr merge lp:~smspillaz/compiz-libcompizconfig/compiz-libcompizconfig.fix_965220
Reviewer Review Type Date Requested Status
Daniel van Vugt Approve
Alan Griffiths Approve
Review via email: mp+99307@code.launchpad.net

Description of the change

== Problem ==

We don't ship the GLib main loop plugin anymore but we were still processing glib events inside of compizconfig

== Solution ==

Never process GLib events in compizconfig

To post a comment you must log in.
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

LGTM

review: Approve
Revision history for this message
Daniel van Vugt (vanvugt) :
review: Approve
Revision history for this message
Daniel van Vugt (vanvugt) wrote :

Should also fix bug 932125?

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'plugin/ccp/src/ccp.cpp'
--- plugin/ccp/src/ccp.cpp 2011-08-20 19:03:37 +0000
+++ plugin/ccp/src/ccp.cpp 2012-03-26 13:04:17 +0000
@@ -422,10 +422,7 @@
422bool422bool
423CcpScreen::timeout ()423CcpScreen::timeout ()
424{424{
425 unsigned int flags = 0;425 unsigned int flags = ProcessEventsNoGlibMainLoopMask;
426
427 if (CompPlugin::find ("glib"))
428 flags |= ProcessEventsNoGlibMainLoopMask;
429426
430 ccsProcessEvents (mContext, flags);427 ccsProcessEvents (mContext, flags);
431428

Subscribers

People subscribed via source and target branches