Merge lp:~robert-ancell/lightdm/test-runner-timer-logic into lp:lightdm

Proposed by Robert Ancell
Status: Merged
Approved by: Robert Ancell
Approved revision: 1906
Merged at revision: 1906
Proposed branch: lp:~robert-ancell/lightdm/test-runner-timer-logic
Merge into: lp:lightdm
Diff against target: 21 lines (+2/-2)
1 file modified
tests/src/test-runner.c (+2/-2)
To merge this branch: bzr merge lp:~robert-ancell/lightdm/test-runner-timer-logic
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Robert Ancell Approve
Review via email: mp+208501@code.launchpad.net

Commit message

Fix up test runner timeout logic

To post a comment you must log in.
Revision history for this message
Robert Ancell (robert-ancell) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/src/test-runner.c'
--- tests/src/test-runner.c 2014-02-26 03:52:39 +0000
+++ tests/src/test-runner.c 2014-02-26 23:46:22 +0000
@@ -831,7 +831,8 @@
831 line->done = TRUE;831 line->done = TRUE;
832832
833 /* Restart timeout */833 /* Restart timeout */
834 g_source_remove (status_timeout);834 if (status_timeout)
835 g_source_remove (status_timeout);
835 status_timeout = g_timeout_add (status_timeout_ms, status_timeout_cb, NULL);836 status_timeout = g_timeout_add (status_timeout_ms, status_timeout_cb, NULL);
836837
837 run_commands ();838 run_commands ();
@@ -1916,7 +1917,6 @@
1916ready (void)1917ready (void)
1917{1918{
1918 run_commands ();1919 run_commands ();
1919 status_timeout = g_timeout_add (status_timeout_ms, status_timeout_cb, NULL);
1920}1920}
19211921
1922static gboolean1922static gboolean

Subscribers

People subscribed via source and target branches