Merge lp://qastaging/~vanvugt/compiz-core/fix-924691 into lp://qastaging/compiz-core/0.9.5
Proposed by
Daniel van Vugt
| Status: | Merged |
|---|---|
| Approved by: | Sam Spilsbury |
| Approved revision: | 2977 |
| Merged at revision: | 2977 |
| Proposed branch: | lp://qastaging/~vanvugt/compiz-core/fix-924691 |
| Merge into: | lp://qastaging/compiz-core/0.9.5 |
| Diff against target: |
112 lines (+22/-26) 4 files modified
include/core/global.h (+2/-0) src/global.cpp (+1/-0) src/main.cpp (+16/-19) src/screen.cpp (+3/-7) |
| To merge this branch: | bzr merge lp://qastaging/~vanvugt/compiz-core/fix-924691 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Sam Spilsbury | Approve | ||
| Alan Griffiths | Approve | ||
|
Review via email:
|
|||
Description of the change
Disable XSynchronize by default, and add an option to enable it (LP: #924691)
In testing, this almost halves the CPU usage of compiz and Xorg. But don't get too excited because XSynchronize was correctly disabled in previous compiz versions. We're only fixing a recent regression in lp:compiz-core here.
To post a comment you must log in.
Revision history for this message
| Alan Griffiths (alan-griffiths) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
I think this somehow snuck in here from some debugging work I was doing with getting compiz up and running on xig. Been meaning to remove it for a while. Thanks for making it an option though!