banshee r3582 - in trunk/banshee: . src/Core/Banshee.ThickClient/Banshee.Gui

[gburt svn gnome org]

Author: gburt
Date: Sat Mar 29 05:16:58 2008
New Revision: 3582
URL: http://svn.gnome.org/viewvc/banshee?rev=3582&view=rev

Log:
2008-03-29  Gabriel Burt  <gabriel burt gmail com>

	* src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs: Be more
	careful with types when iterating over action proxies.  Should fix BGO
	#524706.  Thanks to Matt Enright for the pointer.


Modified:
   trunk/banshee/ChangeLog
   trunk/banshee/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs

Modified: trunk/banshee/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs
==============================================================================
--- trunk/banshee/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs	(original)
+++ trunk/banshee/src/Core/Banshee.ThickClient/Banshee.Gui/TrackActions.cs	Sat Mar 29 05:16:58 2008
@@ -287,7 +287,11 @@
             Source active_source = ServiceManager.SourceManager.ActiveSource;
 
             // TODO find just the menu that was activated instead of modifying all proxies
-            foreach (MenuItem menu in (o as Gtk.Action).Proxies) {
+            foreach (Widget proxy_widget in (o as Gtk.Action).Proxies) {
+                MenuItem menu = proxy_widget as MenuItem;
+                if (menu == null)
+                    continue;
+
                 Menu submenu = new Menu ();
                 menu.Submenu = submenu;