Merge lp://staging/~compiz-team/compiz/compiz.fix_1058577 into lp://staging/compiz/0.9.9
Proposed by
Sam Spilsbury
Status: | Merged |
---|---|
Approved by: | Daniel van Vugt |
Approved revision: | 3408 |
Merged at revision: | 3413 |
Proposed branch: | lp://staging/~compiz-team/compiz/compiz.fix_1058577 |
Merge into: | lp://staging/compiz/0.9.9 |
Diff against target: |
500 lines (+371/-26) 5 files modified
compizconfig/libcompizconfig/src/bindings.c (+52/-26) compizconfig/libcompizconfig/src/ccs-modifier-list-inl.h (+50/-0) compizconfig/libcompizconfig/src/ccs-private.h (+19/-0) compizconfig/libcompizconfig/tests/CMakeLists.txt (+12/-0) compizconfig/libcompizconfig/tests/compizconfig_test_ccs_util.cpp (+238/-0) |
To merge this branch: | bzr merge lp://staging/~compiz-team/compiz/compiz.fix_1058577 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Daniel van Vugt | Approve | ||
jenkins (community) | continuous-integration | Needs Fixing | |
Sam Spilsbury | Approve | ||
Review via email: mp+128924@code.staging.launchpad.net |
This proposal supersedes a proposal from 2012-10-10.
Commit message
Added tests for the StringToModifiers and ModifiersToString code, move the
code which detects <Primary> as ControlMask upstream and use two functions to
ensure that duplicates aren't added.
Fixes Jenkins failures. (LP: #1058577)
Description of the change
Added tests for the StringToModifiers and ModifiersToString code, move the code which detects <Primary> as ControlMask upstream and use two functions to ensure that duplicates aren't added.
Two tests were failing in bug 1058577 because both <Control> and <Primary> were added to the modifier string instead of just <Control>.
To post a comment you must log in.
Note, jenkins will probably mark this needs fixing because of conflicts with a distro patch. That distro patch should be dropped as its effectively made redundant by this branch.