Merge lp:~aacid/unity8/initialize_nextTimeoutTime into lp:unity8

[Albert Astals Cid]

Initialize m_nextTimeoutTime

It's not really needed since it's only used in update() if start()
has been called that then initializes m_nextTimeoutTime

But it's always nice to initialize all members

 * Are there any related MPs required for this MP to build/function as expected?
No

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

[Lukáš Tinkl (lukas-kde)]

LGTM, trivial change, it indeed never hurts to init member vars

review: Approve

[Albert Astals Cid (aacid)]

The code moved to the uitk

[Unity8 CI Bot (unity8-ci-bot)]

FAILED: Continuous integration, rev:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1353/
Executed test runs:
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build/1791/console
    FAILURE: https://unity8-jenkins.ubuntu.com/job/build-0-fetch/1821/console

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-ci/1353/rebuild

review: Needs Fixing (continuous-integration)

Unmerged revisions

2410. By Albert Astals Cid on 2016-05-26

Initialize m_nextTimeoutTime

It's not really needed since it's only used in update() if start()
has been called that then initializes m_nextTimeoutTime

But it's always nice to initialize all members

=== modified file 'libs/UbuntuGestures/Timer.cpp'
--- libs/UbuntuGestures/Timer.cpp	2015-04-24 13:19:24 +0000
+++ libs/UbuntuGestures/Timer.cpp	2016-05-26 08:37:20 +0000
@@ -63,6 +63,7 @@
     , m_interval(0)
     , m_singleShot(false)
     , m_timeSource(timeSource)
+    , m_nextTimeoutTime(0)
 {
 }