Merge lp://staging/~charlesk/indicator-power/lp-1100546-not-present-mouse into lp://staging/indicator-power

Proposed by Charles Kerr
Status: Merged
Approved by: Charles Kerr
Approved revision: 302
Merged at revision: 302
Proposed branch: lp://staging/~charlesk/indicator-power/lp-1100546-not-present-mouse
Merge into: lp://staging/indicator-power
Diff against target: 546 lines (+144/-73)
8 files modified
debian/changelog (+6/-0)
src/device-provider-upower.c (+5/-1)
src/device.c (+37/-5)
src/device.h (+4/-1)
src/service.c (+22/-6)
src/testing.c (+2/-1)
tests/test-device.cc (+62/-56)
tests/test-notify.cc (+6/-3)
To merge this branch: bzr merge lp://staging/~charlesk/indicator-power/lp-1100546-not-present-mouse
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
Review via email: mp+314869@code.staging.launchpad.net

Commit message

When sorting devices by priority, give preference to devices that power the system.

Description of the change

When sorting devices by priority, give preference to devices that power the system.

To post a comment you must log in.
Revision history for this message
Charles Kerr (charlesk) wrote :

Self-approving as I'm just the code reviewer; Shih-Yuan Lee authored the patch and submitted as a debdiff.

review: Approve

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