lp:~vanvugt/compiz/fix-1009320

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:compiz/0.9.8 at revision 3245

Sam Spilsbury: Approve on 2012-06-08

Diff: 12 lines (+1/-1)

1 file modified

plugins/bench/src/bench.cpp (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1009320: Benchmark key is not consumed by compiz. It gets passed to the underlying window.

Low

Fix Released

Link a bug report

Related blueprints

3245. By Daniel van Vugt on 2012-06-08

Benchmark plugin should consume its key binding, and not pass the key to
the underlying window. (LP: #1009320)

3244. By Sam Spilsbury on 2012-06-06

Adds some tests for the pixmap rebinding code ... which also means some
refactoring was done.

3243. By Sam Spilsbury on 2012-06-06

Avoid needless STL operations leading to expensive heap operations.
(LP: #1006335)

3242. By Sam Spilsbury on 2012-06-06

Fix a typo that was causing LP: #1002606 (widthIncBorders/heightIncBorders)

3241. By Sam Spilsbury on 2012-06-06

Check if the window is decorated before trying to change its event window
states (which won't exist if not decorated) (LP: #1007754)

3240. By Daniel van Vugt on 2012-06-05

Revert the previous commit. I can see a potential race that might lead to
damage artifacts.

3239. By Daniel van Vugt on 2012-06-05

Use the XDamage extension more efficiently (the way it was designed to be
used). This dramatically reduces CPU usage, reduces wakeups, and increases
frame rates. It also solves at least one observed performance bug
(LP: #1007299) and probably several more.

3238. By Daniel van Vugt on 2012-06-05

Fix up a few issues thumper pointed out in the fixes for LP: #1005569.

  * Use else instead of unconditional reset (makes the first call faster,
    but all others slower, negligibly)
  * Don't mask locals by reusing the same name
  * Remember to reset arg[7] for XkbStateNotify
  * Check for NULL strings. Don't dereference them, just treat as empty.

3237. By Daniel van Vugt on 2012-05-31

Improve the fix for LP: #1005569:
Calling CompOption::setName should not implicitly construct a new string
object every time, when the name is not changing.

This is also an alternative fix to the previous commit, but both together
don't hurt either.

3236. By Daniel van Vugt on 2012-05-30

Avoid constructing and destructing lots of strings on every single event,
which was wasting lots of CPU (LP: #1005569)