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
1=== modified file 'src/snap.cpp'
2--- src/snap.cpp 2014-02-27 16:14:52 +0000
3+++ src/snap.cpp 2014-03-20 18:50:35 +0000
4@@ -186,8 +186,16 @@
5 {
6 std::set<std::string> caps_set;
7 auto caps_gl = notify_get_server_caps();
8+ std::string caps_str;
9 for(auto l=caps_gl; l!=nullptr; l=l->next)
10+ {
11 caps_set.insert((const char*)l->data);
12+
13+ caps_str += (const char*) l->data;;
14+ if (l->next != nullptr)
15+ caps_str += ", ";
16+ }
17+ g_debug ("%s notify_get_server() returned [%s]", G_STRFUNC, caps_str.c_str());
18 g_list_free_full(caps_gl, g_free);
19 return caps_set;
20 }

Subscribers

People subscribed via source and target branches