Merge lp:~azzar1/unity/ignore-autorepeat-actions into lp:unity

Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 4110
Proposed branch: lp:~azzar1/unity/ignore-autorepeat-actions
Merge into: lp:unity
Diff against target: 17 lines (+5/-2)
1 file modified
unity-shared/GnomeKeyGrabber.cpp (+5/-2)
To merge this branch: bzr merge lp:~azzar1/unity/ignore-autorepeat-actions
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+292894@code.launchpad.net

Commit message

GnomeKeyGrabber: do not activate an action if it's a repeated one.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'unity-shared/GnomeKeyGrabber.cpp'
2--- unity-shared/GnomeKeyGrabber.cpp 2015-12-13 10:35:58 +0000
3+++ unity-shared/GnomeKeyGrabber.cpp 2016-04-26 09:53:34 +0000
4@@ -232,8 +232,11 @@
5 {
6 action.setState(CompAction::StateInitKey);
7 action.setInitiate([this, action_id](CompAction* action, CompAction::State state, CompOption::Vector& options) {
8- LOG_DEBUG(logger) << "pressed \"" << action->keyToString() << "\"";
9- ActivateDBusAction(*action, action_id, 0, CompOption::getIntOptionNamed(options, "time"));
10+ if (!CompOption::getBoolOptionNamed(options, "is_repeated"))
11+ {
12+ LOG_DEBUG(logger) << "pressed \"" << action->keyToString() << "\"";
13+ ActivateDBusAction(*action, action_id, 0, CompOption::getIntOptionNamed(options, "time"));
14+ }
15 return true;
16 });
17 }