Application Menu Indicator

Merge lp:~charlesk/indicator-appmenu/lp-957412 into lp: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:~charlesk/indicator-appmenu/lp-957412
Merge into: lp: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:~charlesk/indicator-appmenu/lp-957412
Reviewer Review Type Date Requested Status
Conor Curran (community) Approve
Review via email: mp+98004@code.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
1=== modified file 'src/indicator-appmenu.c'
2--- src/indicator-appmenu.c 2012-03-16 15:48:16 +0000
3+++ src/indicator-appmenu.c 2012-03-16 22:00:32 +0000
4@@ -184,7 +184,7 @@
5 static gboolean retry_registration (gpointer user_data);
6 static void bus_method_call (GDBusConnection * connection,
7 const gchar * sender,
8- const gchar * path,
9+ const gchar * object_path,
10 const gchar * interface,
11 const gchar * method,
12 GVariant * params,
13@@ -1282,7 +1282,7 @@
14 is and dispatch it. */
15 static void
16 bus_method_call (GDBusConnection * connection, const gchar * sender,
17- const gchar * path, const gchar * interface,
18+ const gchar * object_path, const gchar * interface,
19 const gchar * method, GVariant * params,
20 GDBusMethodInvocation * invocation, gpointer user_data)
21 {

Subscribers

People subscribed via source and target branches