Merge lp://staging/~ted/libdbusmenu/lp907635 into lp://staging/libdbusmenu/14.04

Proposed by Ted Gould
Status: Merged
Approved by: Ted Gould
Approved revision: 464
Merged at revision: 464
Proposed branch: lp://staging/~ted/libdbusmenu/lp907635
Merge into: lp://staging/libdbusmenu/14.04
Diff against target: 18 lines (+6/-2)
1 file modified
libdbusmenu-gtk/parser.c (+6/-2)
To merge this branch: bzr merge lp://staging/~ted/libdbusmenu/lp907635
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Ted Gould (community) Approve
Review via email: mp+198961@code.staging.launchpad.net

Commit message

Check to ensure the property isn't set to NULL

Description of the change

Patch from bug

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

the build issue is due to invalid g_source_remove() use, the new glib is reporting those. I've opened https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/1260779 about the issue

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
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
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: