ayatana-commits team mailing list archive

[Ted Gould <ted@xxxxxxxx>]

Ted Gould has proposed merging lp:~ted/indicator-appmenu/alt-d-problem into lp:indicator-appmenu.

Requested reviews:
  Indicator Applet Developers (indicator-applet-developers)


Switching the call order to get_position so we check the right child against the right parent.
-- 
https://code.launchpad.net/~ted/indicator-appmenu/alt-d-problem/+merge/42699
Your team ayatana-commits is subscribed to branch lp:indicator-appmenu.

=== modified file 'src/window-menus.c'
--- src/window-menus.c	2010-09-21 21:38:18 +0000
+++ src/window-menus.c	2010-12-03 22:56:58 +0000
@@ -317,7 +317,7 @@
 		return;
 	}
 
-	guint position = dbusmenu_menuitem_get_position(priv->root, item);
+	guint position = dbusmenu_menuitem_get_position(item, priv->root);
 	if (position == 0) {
 		/* Ugly, ugly hack.  I shouldn't have used guint in the function
 		   above, but now I have to do this.  Ew! */