Messaging Menu

Merge lp://staging/~larsu/indicator-messages/lp1058386 into lp://staging/indicator-messages/12.10

  1. lp1058386
  2. Merge into trunk.12.10
Proposed by Lars Karlitski
Status: Merged
Approved by: Lars Karlitski
Approved revision: 318
Merge reported by: Lars Karlitski
Merged at revision: not available
Proposed branch: lp://staging/~larsu/indicator-messages/lp1058386
Merge into: lp://staging/indicator-messages/12.10
Diff against target: 62 lines (+14/-3)
1 file modified
libmessaging-menu/messaging-menu.c (+14/-3)
To merge this branch: bzr merge lp://staging/~larsu/indicator-messages/lp1058386
Reviewer Review Type Date Requested Status
jenkins (community) continuous-integration Needs Fixing
Charles Kerr (community) Approve
Review via email: mp+127308@code.staging.launchpad.net

Description of the change

libmessaging-menu: don't crash when getting an invalid desktop id

Instead, silently don't export menus and actions. The single warning about the desktop id being invalid should be enough.

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

PASSED: Continuous integration, rev:318
http://jenkins.qa.ubuntu.com/job/indicator-messages-ci/21/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-messages-ci/./label=quantal/21/console

review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) :
review: Approve
Revision history for this message
jenkins (martin-mrazik+qa) wrote :

FAILED: Autolanding.
No commit message was specified.
http://jenkins.qa.ubuntu.com/job/indicator-messages-autolanding/9/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/indicator-messages-autolanding/./label=quantal/9/console

review: Needs Fixing (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