Application Menu Indicator

Merge lp://staging/~charlesk/indicator-appmenu/lp-957412 into lp://staging/indicator-appmenu/0.4

  1. lp-957412
  2. Merge into trunk.0.4
Proposed by Charles Kerr
Status: Merged
Approved by: Charles Kerr
Approved revision: 181
Merged at revision: 183
Proposed branch: lp://staging/~charlesk/indicator-appmenu/lp-957412
Merge into: lp://staging/indicator-appmenu/0.4
Diff against target: 21 lines (+2/-2)
1 file modified
src/indicator-appmenu.c (+2/-2)
To merge this branch: bzr merge lp://staging/~charlesk/indicator-appmenu/lp-957412
Reviewer Review Type Date Requested Status
Conor Curran (community) Approve
Review via email: mp+98004@code.staging.launchpad.net

Description of the change

The function's unused argument "path" was shadowed by a local variable. Fixed by renaming the argument to "object_path"

To post a comment you must log in.
Revision history for this message
Conor Curran (cjcurran) :
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