Unity

Merge lp:~azzar1/unity/fix-crash-acc-controller into lp:unity

  1. fix-crash-acc-controller
  2. Merge into trunk
Proposed by Andrea Azzarone
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: no longer in the source branch.
Merged at revision: 4155
Proposed branch: lp:~azzar1/unity/fix-crash-acc-controller
Merge into: lp:unity
Diff against target: 15 lines (+2/-2)
1 file modified
lockscreen/LockScreenAcceleratorController.cpp (+2/-2)
To merge this branch: bzr merge lp:~azzar1/unity/fix-crash-acc-controller
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Review via email: mp+300738@code.launchpad.net

Commit message

Make sure initiate and terminate callbacks are not empty before calling them.

To post a comment you must log in.
Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lockscreen/LockScreenAcceleratorController.cpp'
--- lockscreen/LockScreenAcceleratorController.cpp 2016-07-05 10:22:08 +0000
+++ lockscreen/LockScreenAcceleratorController.cpp 2016-07-21 10:09:18 +0000
@@ -193,10 +193,10 @@
193193
194 CompOption::Vector options;194 CompOption::Vector options;
195195
196 if (action.state() & CompAction::StateInitKey)196 if (action.initiate() && action.state() & CompAction::StateInitKey)
197 action.initiate()(&action, 0, options);197 action.initiate()(&action, 0, options);
198198
199 if (action.state() & CompAction::StateTermKey)199 if (action.terminate() && action.state() & CompAction::StateTermKey)
200 action.terminate()(&action, CompAction::StateTermTapped, options);200 action.terminate()(&action, CompAction::StateTermTapped, options);
201}201}
202202