Merge lp:~charlesk/indicator-datetime/lp-1295271 into lp:indicator-datetime/14.04

Proposed by Charles Kerr
Status: Merged
Approved by: Ted Gould
Approved revision: 325
Merged at revision: 326
Proposed branch: lp:~charlesk/indicator-datetime/lp-1295271
Merge into: lp:indicator-datetime/14.04
Diff against target: 20 lines (+8/-0)
1 file modified
src/snap.cpp (+8/-0)
To merge this branch: bzr merge lp:~charlesk/indicator-datetime/lp-1295271
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+212028@code.launchpad.net

Commit message

Add debug logging of what capabilities the notification server said it supports.

Description of the change

Add debug logging of what capabilities the notification server said it supports.

This debugging information would have simplified tracking down the cause of this bug.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Ted Gould (ted) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/snap.cpp'
--- src/snap.cpp 2014-02-27 16:14:52 +0000
+++ src/snap.cpp 2014-03-20 18:50:35 +0000
@@ -186,8 +186,16 @@
186{186{
187 std::set<std::string> caps_set;187 std::set<std::string> caps_set;
188 auto caps_gl = notify_get_server_caps();188 auto caps_gl = notify_get_server_caps();
189 std::string caps_str;
189 for(auto l=caps_gl; l!=nullptr; l=l->next)190 for(auto l=caps_gl; l!=nullptr; l=l->next)
191 {
190 caps_set.insert((const char*)l->data);192 caps_set.insert((const char*)l->data);
193
194 caps_str += (const char*) l->data;;
195 if (l->next != nullptr)
196 caps_str += ", ";
197 }
198 g_debug ("%s notify_get_server() returned [%s]", G_STRFUNC, caps_str.c_str());
191 g_list_free_full(caps_gl, g_free);199 g_list_free_full(caps_gl, g_free);
192 return caps_set;200 return caps_set;
193}201}

Subscribers

People subscribed via source and target branches