Indicator Date and Time

Merge lp:~charlesk/indicator-datetime/lp-1064546 into lp:indicator-datetime/12.10

  1. lp-1064546
  2. Merge into trunk.12.10
Proposed by Charles Kerr
Status: Merged
Approved by: Lars Karlitski
Approved revision: 190
Merged at revision: 190
Proposed branch: lp:~charlesk/indicator-datetime/lp-1064546
Merge into: lp:indicator-datetime/12.10
Diff against target: 12 lines (+1/-1)
1 file modified
src/datetime-prefs.c (+1/-1)
To merge this branch: bzr merge lp:~charlesk/indicator-datetime/lp-1064546
Reviewer Review Type Date Requested Status
Lars Karlitski (community) Approve
jenkins (community) continuous-integration Approve
Review via email: mp+128771@code.launchpad.net

Commit message

remove stray semicolon found by Coverity

To post a comment you must log in.
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

PASSED: Continuous integration, rev:190
http://jenkins.qa.ubuntu.com/job/indicator-datetime-ci/14/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-ci/./build=pbuilder,distribution=quantal,flavor=amd64/14/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-datetime-ci/./build=pbuilder,distribution=quantal,flavor=i386/14/console

review: Approve (continuous-integration)
Revision history for this message
Lars Karlitski (larsu) wrote :

Yeah.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/datetime-prefs.c'
2--- src/datetime-prefs.c 2012-05-17 20:46:42 +0000
3+++ src/datetime-prefs.c 2012-10-09 17:22:24 +0000
4@@ -308,7 +308,7 @@
5 g_dbus_proxy_call (priv->proxy, "GetUsingNtp", NULL, G_DBUS_CALL_FLAGS_NONE, -1,
6 priv->ntp_query_cancel, (GAsyncReadyCallback)ntp_query_answered, self);
7 }
8- if (priv->tz_query_cancel == NULL); {
9+ if (priv->tz_query_cancel == NULL) {
10 priv->tz_query_cancel = g_cancellable_new();
11 g_dbus_proxy_call (priv->proxy, "GetTimezone", NULL, G_DBUS_CALL_FLAGS_NONE, -1,
12 priv->tz_query_cancel, (GAsyncReadyCallback)tz_query_answered, self);

Subscribers

People subscribed via source and target branches