Merge lp:~ted/url-dispatcher/lp1608009-no-message into lp:url-dispatcher/16.10

Proposed by Ted Gould
Status: Merged
Approved by: Marcus Tomlinson
Approved revision: 92
Merged at revision: 94
Proposed branch: lp:~ted/url-dispatcher/lp1608009-no-message
Merge into: lp:url-dispatcher/16.10
Diff against target: 12 lines (+1/-1)
1 file modified
service/dispatcher.c (+1/-1)
To merge this branch: bzr merge lp:~ted/url-dispatcher/lp1608009-no-message
Reviewer Review Type Date Requested Status
Marcus Tomlinson (community) Approve
unity-api-1-bot continuous-integration Needs Fixing
Review via email: mp+302931@code.launchpad.net

Commit message

Not using a message that isn't set properly

Description of the change

Hate it when GIO sets these to bad values sometimes :-/

To post a comment you must log in.
Revision history for this message
unity-api-1-bot (unity-api-1-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

Ack

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'service/dispatcher.c'
--- service/dispatcher.c 2015-09-03 21:50:06 +0000
+++ service/dispatcher.c 2016-08-15 14:02:27 +0000
@@ -517,7 +517,7 @@
517517
518 if (error != NULL) {518 if (error != NULL) {
519 if (!g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {519 if (!g_error_matches(error, G_IO_ERROR, G_IO_ERROR_CANCELLED)) {
520 g_error("Unable to connect to D-Bus: %s", error->message);520 g_error("Unable to connect to D-Bus");
521 g_main_loop_quit(mainloop);521 g_main_loop_quit(mainloop);
522 }522 }
523 g_error_free(error);523 g_error_free(error);

Subscribers

People subscribed via source and target branches