X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/b412f9be504e07559a98ae234f30bb1bd0b2aa1f..e69d5138dc8f95e294ae6d03beb4f321406f4e34:/src/motif/menuitem.cpp

diff --git a/src/motif/menuitem.cpp b/src/motif/menuitem.cpp
index e04e0e938a..2b57e57d6a 100644
--- a/src/motif/menuitem.cpp
+++ b/src/motif/menuitem.cpp
@@ -1,5 +1,5 @@
 ///////////////////////////////////////////////////////////////////////////////
-// Name:        menuitem.cpp
+// Name:        src/motif/menuitem.cpp
 // Purpose:     wxMenuItem implementation
 // Author:      Julian Smart
 // Modified by:
@@ -10,14 +10,28 @@
 ///////////////////////////////////////////////////////////////////////////////
 
 // ============================================================================
-// headers & declarations
+// declarations
 // ============================================================================
 
-#include "wx/menu.h"
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+
+// For compilers that support precompilation, includes "wx.h".
+#include "wx/wxprec.h"
+
 #include "wx/menuitem.h"
-#include "wx/utils.h"
-#include "wx/frame.h"
+#include "wx/stockitem.h"
+
+#ifndef WX_PRECOMP
+    #include "wx/utils.h"
+    #include "wx/frame.h"
+    #include "wx/menu.h"
+#endif
 
+#ifdef __VMS__
+#pragma message disable nosimpint
+#endif
 #include <Xm/Label.h>
 #include <Xm/LabelG.h>
 #include <Xm/CascadeBG.h>
@@ -27,15 +41,19 @@
 #include <Xm/ToggleB.h>
 #include <Xm/ToggleBG.h>
 #include <Xm/RowColumn.h>
+#ifdef __VMS__
+#pragma message enable nosimpint
+#endif
 
 #include "wx/motif/private.h"
 
-void wxMenuItemCallback (Widget w, XtPointer clientData,
-			 XtPointer ptr);
-void wxMenuItemArmCallback (Widget w, XtPointer clientData,
-			    XtPointer ptr);
-void wxMenuItemDisarmCallback (Widget w, XtPointer clientData,
-			       XtPointer ptr);
+// ----------------------------------------------------------------------------
+// functions prototypes
+// ----------------------------------------------------------------------------
+
+static void wxMenuItemCallback(Widget w, XtPointer clientData, XtPointer ptr);
+static void wxMenuItemArmCallback(Widget w, XtPointer clientData, XtPointer ptr);
+static void wxMenuItemDisarmCallback(Widget w, XtPointer clientData, XtPointer ptr);
 
 // ============================================================================
 // implementation
@@ -45,9 +63,7 @@ void wxMenuItemDisarmCallback (Widget w, XtPointer clientData,
 // dynamic classes implementation
 // ----------------------------------------------------------------------------
 
-#if !USE_SHARED_LIBRARY
-  IMPLEMENT_DYNAMIC_CLASS(wxMenuItem, wxObject)
-#endif  //USE_SHARED_LIBRARY
+IMPLEMENT_DYNAMIC_CLASS(wxMenuItem, wxObject)
 
 // ----------------------------------------------------------------------------
 // wxMenuItem
@@ -56,309 +72,353 @@ void wxMenuItemDisarmCallback (Widget w, XtPointer clientData,
 // ctor & dtor
 // -----------
 
-wxMenuItem::wxMenuItem(wxMenu *pParentMenu, int id,
-                       const wxString& strName, const wxString& strHelp,
-                       bool bCheckable,
-                       wxMenu *pSubMenu) :
-                        m_strHelp(strHelp),
-                        m_bCheckable(bCheckable),
-                        m_strName(strName)
+wxMenuItem::wxMenuItem(wxMenu *pParentMenu,
+                       int id,
+                       const wxString& strName,
+                       const wxString& strHelp,
+                       wxItemKind kind,
+                       wxMenu *pSubMenu)
+          : wxMenuItemBase(pParentMenu, id, strName, strHelp, kind, pSubMenu)
 {
-  wxASSERT( pParentMenu != NULL );
-
-  m_pParentMenu = pParentMenu;
-  m_pSubMenu    = pSubMenu;
-  m_idItem      = id;
-  m_bEnabled    = TRUE;
-  m_bChecked    = FALSE;
-
-  //// Motif-specific
-  m_menuBar = NULL;
-  m_buttonWidget = (WXWidget) NULL;
-  m_topMenu = NULL;
+    // Motif-specific
+    m_menuBar      = NULL;
+    m_buttonWidget = (WXWidget) NULL;
+    m_topMenu      = NULL;
 }
 
-wxMenuItem::~wxMenuItem() 
+wxMenuItem::~wxMenuItem()
 {
 }
 
-// misc
-// ----
-
-// delete the sub menu
-void wxMenuItem::DeleteSubMenu()
-{
-  wxASSERT( m_pSubMenu != NULL );
-
-  delete m_pSubMenu;
-  m_pSubMenu = NULL;
-}
-
 // change item state
 // -----------------
 
 void wxMenuItem::Enable(bool bDoEnable)
 {
-  if ( m_bEnabled != bDoEnable )
-  {
-    if ( m_pSubMenu == NULL )
-    {     // normal menu item
-        if (m_buttonWidget)
-            XtSetSensitive( (Widget) m_buttonWidget, (Boolean) bDoEnable);
-    }
-    else                            // submenu
+    if ( m_isEnabled != bDoEnable )
     {
-        // Maybe we should apply this to all items in the submenu?
-        // Or perhaps it works anyway.
-        if (m_buttonWidget)
-            XtSetSensitive( (Widget) m_buttonWidget, (Boolean) bDoEnable);
-    }
+        if ( !IsSubMenu() )
+        {
+            // normal menu item
+            if (m_buttonWidget)
+                XtSetSensitive( (Widget) m_buttonWidget, (Boolean) bDoEnable);
+        }
+        else                            // submenu
+        {
+            // Maybe we should apply this to all items in the submenu?
+            // Or perhaps it works anyway.
+            if (m_buttonWidget)
+                XtSetSensitive( (Widget) m_buttonWidget, (Boolean) bDoEnable);
+        }
 
-    m_bEnabled = bDoEnable;
-  }
+        wxMenuItemBase::Enable(bDoEnable);
+    }
 }
 
 void wxMenuItem::Check(bool bDoCheck)
 {
-  wxCHECK_RET( IsCheckable(), "only checkable items may be checked" );
+    wxCHECK_RET( IsCheckable(), "only checkable items may be checked" );
 
-  if ( m_bChecked != bDoCheck )
-  {
-    if (m_buttonWidget && XtIsSubclass ((Widget) m_buttonWidget, xmToggleButtonGadgetClass))
+    if ( m_isChecked != bDoCheck )
     {
-      XtVaSetValues ( (Widget) m_buttonWidget, XmNset, (Boolean) bDoCheck, NULL);
+        if ( m_buttonWidget )
+        {
+            wxASSERT_MSG( XtIsSubclass((Widget)m_buttonWidget,
+                                       xmToggleButtonGadgetClass),
+                          wxT("checkable menu item must be a toggle button") );
+
+            XtVaSetValues((Widget)m_buttonWidget,
+                          XmNset, (Boolean)bDoCheck,
+                          NULL);
+        }
+
+        wxMenuItemBase::Check(bDoCheck);
     }
-    m_bChecked = bDoCheck;
-  }
 }
 
-//// Motif-specific
+/* static */
+wxString wxMenuItemBase::GetLabelFromText(const wxString& text)
+{
+    return wxStripMenuCodes(text);
+}
+
+// ----------------------------------------------------------------------------
+// wxMenuItemBase
+// ----------------------------------------------------------------------------
+
+wxMenuItem *wxMenuItemBase::New(wxMenu *parentMenu,
+                                int id,
+                                const wxString& name,
+                                const wxString& help,
+                                wxItemKind kind,
+                                wxMenu *subMenu)
+{
+    return new wxMenuItem(parentMenu, id, name, help, kind, subMenu);
+}
+
+// ----------------------------------------------------------------------------
+// Motif-specific
+// ----------------------------------------------------------------------------
 
-void wxMenuItem::CreateItem (WXWidget menu, wxMenuBar * menuBar, wxMenu * topMenu)
+void wxMenuItem::CreateItem (WXWidget menu, wxMenuBar * menuBar,
+                             wxMenu * topMenu, size_t index)
 {
-  m_menuBar = menuBar;
-  m_topMenu = topMenu;
+    m_menuBar = menuBar;
+    m_topMenu = topMenu;
 
-  if (GetId() == -2)
+    if (GetId() == -3)
     {
-      // Id=-2 identifies a Title item.
-      wxStripMenuCodes ((char*) (const char*) m_strName, wxBuffer);
-      m_buttonWidget = (WXWidget) XtVaCreateManagedWidget (wxBuffer,
-					    xmLabelGadgetClass, (Widget) menu, NULL);
+        // Id=-3 identifies a Title item.
+        m_buttonWidget = (WXWidget) XtVaCreateManagedWidget
+            (wxStripMenuCodes(m_text),
+            xmLabelGadgetClass, (Widget) menu, NULL);
     }
-  else if ((!m_strName.IsNull() && m_strName != "") && (!m_pSubMenu))
+    else if (!IsSeparator() && !m_subMenu)
     {
-      wxStripMenuCodes ((char*) (const char*) m_strName, wxBuffer);
-      if (IsCheckable())
-	{
-	  m_buttonWidget = (WXWidget) XtVaCreateManagedWidget (wxBuffer,
-					    xmToggleButtonGadgetClass, (Widget) menu,
-						  NULL);
-	  XtVaSetValues ((Widget) m_buttonWidget, XmNset, (Boolean) IsChecked(), NULL);
-	}
-      else
-	m_buttonWidget = (WXWidget) XtVaCreateManagedWidget (wxBuffer,
-					      xmPushButtonGadgetClass, (Widget) menu,
-						NULL);
-      char mnem = wxFindMnemonic (m_strName);
-      if (mnem != 0)
-	XtVaSetValues ((Widget) m_buttonWidget, XmNmnemonic, mnem, NULL);
-
-      //// TODO: proper accelerator treatment. What does wxFindAccelerator
-      //// look for?
-      strcpy(wxBuffer, (char*) (const char*) m_strName);
-      char *accel = wxFindAccelerator (wxBuffer);
-      if (accel)
-	XtVaSetValues ((Widget) m_buttonWidget, XmNaccelerator, accel, NULL);
-
-      // TODO: What does this do?
-      strcpy(wxBuffer, (char*) (const char*) m_strName);
-      XmString accel_str = wxFindAcceleratorText (wxBuffer);
-      if (accel_str)
-	{
-	  XtVaSetValues ((Widget) m_buttonWidget, XmNacceleratorText, accel_str, NULL);
-	  XmStringFree (accel_str);
-	}
-
-      if (IsCheckable())
-	XtAddCallback ((Widget) m_buttonWidget,
-		       XmNvalueChangedCallback,
-		       (XtCallbackProc) wxMenuItemCallback,
-		       (XtPointer) this);
-      else
-	XtAddCallback ((Widget) m_buttonWidget,
-		       XmNactivateCallback,
-		       (XtCallbackProc) wxMenuItemCallback,
-		       (XtPointer) this);
-      XtAddCallback ((Widget) m_buttonWidget,
-		     XmNarmCallback,
-		     (XtCallbackProc) wxMenuItemArmCallback,
-		     (XtPointer) this);
-      XtAddCallback ((Widget) m_buttonWidget,
-		     XmNdisarmCallback,
-		     (XtCallbackProc) wxMenuItemDisarmCallback,
-		     (XtPointer) this);
+        wxString txt = m_text;
+
+        if (m_text.IsEmpty())
+        {
+            wxASSERT_MSG(wxIsStockID(GetId()), wxT("A non-stock menu item with an empty label?"));
+            txt = wxGetStockLabel(GetId(), wxSTOCK_WITH_ACCELERATOR|wxSTOCK_WITH_MNEMONIC);
+        }
+
+        wxString strName = wxStripMenuCodes(txt);
+        if (IsCheckable())
+        {
+            m_buttonWidget = (WXWidget) XtVaCreateManagedWidget (strName,
+                xmToggleButtonGadgetClass, (Widget) menu,
+#ifdef XmNpositionIndex
+                XmNpositionIndex, index,
+#endif
+                NULL);
+            XtVaSetValues ((Widget) m_buttonWidget, XmNset, (Boolean) IsChecked(), NULL);
+        }
+        else
+            m_buttonWidget = (WXWidget) XtVaCreateManagedWidget (strName,
+            xmPushButtonGadgetClass, (Widget) menu,
+#ifdef XmNpositionIndex
+            XmNpositionIndex, index,
+#endif
+            NULL);
+        char mnem = wxFindMnemonic (m_text);
+        if (mnem != 0)
+            XtVaSetValues ((Widget) m_buttonWidget, XmNmnemonic, mnem, NULL);
+
+        //// TODO: proper accelerator treatment. What does wxFindAccelerator
+        //// look for?
+        strName = m_text;
+        char *accel = wxFindAccelerator (strName);
+        if (accel)
+            XtVaSetValues ((Widget) m_buttonWidget, XmNaccelerator, accel, NULL);
+
+        // TODO: What does this do?
+        XmString accel_str = wxFindAcceleratorText (strName);
+        if (accel_str)
+        {
+            XtVaSetValues ((Widget) m_buttonWidget, XmNacceleratorText, accel_str, NULL);
+            XmStringFree (accel_str);
+        }
+
+        if (IsCheckable())
+            XtAddCallback ((Widget) m_buttonWidget,
+            XmNvalueChangedCallback,
+            (XtCallbackProc) wxMenuItemCallback,
+            (XtPointer) this);
+        else
+            XtAddCallback ((Widget) m_buttonWidget,
+            XmNactivateCallback,
+            (XtCallbackProc) wxMenuItemCallback,
+            (XtPointer) this);
+        XtAddCallback ((Widget) m_buttonWidget,
+            XmNarmCallback,
+            (XtCallbackProc) wxMenuItemArmCallback,
+            (XtPointer) this);
+        XtAddCallback ((Widget) m_buttonWidget,
+            XmNdisarmCallback,
+            (XtCallbackProc) wxMenuItemDisarmCallback,
+            (XtPointer) this);
     }
-  else if (GetId() == -1)
+    else if (IsSeparator())
     {
-      m_buttonWidget = (WXWidget) XtVaCreateManagedWidget ("separator",
-					xmSeparatorGadgetClass, (Widget) menu, NULL);
+        m_buttonWidget = (WXWidget) XtVaCreateManagedWidget ("separator",
+            xmSeparatorGadgetClass, (Widget) menu,
+#ifndef XmNpositionIndex
+            XmNpositionIndex, index,
+#endif
+            NULL);
     }
-  else if (m_pSubMenu)
+    else if (m_subMenu)
     {
-      m_buttonWidget = m_pSubMenu->CreateMenu (menuBar, menu, topMenu, m_strName, TRUE);
-      m_pSubMenu->SetButtonWidget(m_buttonWidget);
-      XtAddCallback ((Widget) m_buttonWidget,
-		     XmNcascadingCallback,
-		     (XtCallbackProc) wxMenuItemArmCallback,
-		     (XtPointer) this);
+        m_buttonWidget = m_subMenu->CreateMenu (menuBar, menu, topMenu, index, m_text, true);
+        m_subMenu->SetButtonWidget(m_buttonWidget);
+        XtAddCallback ((Widget) m_buttonWidget,
+            XmNcascadingCallback,
+            (XtCallbackProc) wxMenuItemArmCallback,
+            (XtPointer) this);
     }
-  if (m_buttonWidget)
-    XtSetSensitive ((Widget) m_buttonWidget, (Boolean) IsEnabled());
+    if (m_buttonWidget)
+        XtSetSensitive ((Widget) m_buttonWidget, (Boolean) IsEnabled());
 }
 
 void wxMenuItem::DestroyItem(bool full)
 {
-  if (GetId() == -2)
+    if (GetId() == -3)
     {
-      ;			// Nothing
-      
+        ;      // Nothing
+
     }
-  else if ((!m_strName.IsNull() && (m_strName != "")) && !m_pSubMenu)
+    else if (!m_text.empty() && !m_subMenu)
     {
-      if (m_buttonWidget)
-	{
-	  if (IsCheckable())
-	    XtRemoveCallback ((Widget) m_buttonWidget, XmNvalueChangedCallback,
-			      wxMenuItemCallback, (XtPointer) this);
-	  else
-	    XtRemoveCallback ((Widget) m_buttonWidget, XmNactivateCallback,
-			      wxMenuItemCallback, (XtPointer) this);
-	  XtRemoveCallback ((Widget) m_buttonWidget, XmNarmCallback,
-			    wxMenuItemArmCallback, (XtPointer) this);
-	  XtRemoveCallback ((Widget) m_buttonWidget, XmNdisarmCallback,
-			    wxMenuItemDisarmCallback, (XtPointer) this);
-	}
+        if (m_buttonWidget)
+        {
+            if (IsCheckable())
+                XtRemoveCallback ((Widget) m_buttonWidget, XmNvalueChangedCallback,
+                wxMenuItemCallback, (XtPointer) this);
+            else
+                XtRemoveCallback ((Widget) m_buttonWidget, XmNactivateCallback,
+                wxMenuItemCallback, (XtPointer) this);
+            XtRemoveCallback ((Widget) m_buttonWidget, XmNarmCallback,
+                wxMenuItemArmCallback, (XtPointer) this);
+            XtRemoveCallback ((Widget) m_buttonWidget, XmNdisarmCallback,
+                wxMenuItemDisarmCallback, (XtPointer) this);
+        }
     }
-  else if (GetId() == -1)
+    else if (GetId() == wxID_SEPARATOR)
     {
-      ;			// Nothing
-      
+        ;      // Nothing
+
     }
-  else if (GetSubMenu())
+    else if (GetSubMenu())
     {
-      if (m_buttonWidget)
-	{
-	  XtRemoveCallback ((Widget) m_buttonWidget, XmNcascadingCallback,
-			    wxMenuItemArmCallback, (XtPointer) this);
-	}
-      m_pSubMenu->DestroyMenu(full);
-      if (full)
-        m_buttonWidget = NULL;
+        if (m_buttonWidget)
+        {
+            XtRemoveCallback ((Widget) m_buttonWidget, XmNcascadingCallback,
+                wxMenuItemArmCallback, (XtPointer) this);
+        }
+        m_subMenu->DestroyMenu(full);
+        if (full)
+            m_buttonWidget = NULL;
     }
 
-  if (m_buttonWidget && full)
+    if (m_buttonWidget && full)
     {
-      XtDestroyWidget ((Widget) m_buttonWidget);
-      m_buttonWidget = (WXWidget) 0;
+        XtDestroyWidget ((Widget) m_buttonWidget);
+        m_buttonWidget = (WXWidget) 0;
     }
 }
 
-void wxMenuItem::SetLabel(const wxString& label)
+void wxMenuItem::SetText(const wxString& label)
 {
-  char mnem = wxFindMnemonic (label);
-  wxStripMenuCodes ((char*) (const char*) label, wxBuffer);
+    char mnem = wxFindMnemonic (label);
+    wxString label2 = wxStripMenuCodes(label);
 
-  m_strName = label;
+    m_text = label;
 
-  if (m_buttonWidget)
+    if (m_buttonWidget)
     {
-      XmString label_str = XmStringCreateSimple (wxBuffer);
-      XtVaSetValues ((Widget) m_buttonWidget,
-                     XmNlabelString, label_str,
-                     NULL);
-      XmStringFree (label_str);
-      if (mnem != 0)
-        XtVaSetValues ((Widget) m_buttonWidget, XmNmnemonic, mnem, NULL);
-      strcpy(wxBuffer, (char*) (const char*) label);
-      char *accel = wxFindAccelerator (wxBuffer);
-      if (accel)
-        XtVaSetValues ((Widget) m_buttonWidget, XmNaccelerator, accel, NULL);
-
-      strcpy(wxBuffer, (char*) (const char*) label);
-      XmString accel_str = wxFindAcceleratorText (wxBuffer);
-      if (accel_str)
+        wxXmString label_str(label2);
+        XtVaSetValues ((Widget) m_buttonWidget,
+            XmNlabelString, label_str(),
+            NULL);
+        if (mnem != 0)
+            XtVaSetValues ((Widget) m_buttonWidget, XmNmnemonic, mnem, NULL);
+        char *accel = wxFindAccelerator (label2);
+        if (accel)
+            XtVaSetValues ((Widget) m_buttonWidget, XmNaccelerator, accel, NULL);
+
+        XmString accel_str = wxFindAcceleratorText (label2);
+        if (accel_str)
         {
-          XtVaSetValues ((Widget) m_buttonWidget, XmNacceleratorText, accel_str, NULL);
-          XmStringFree (accel_str);
+            XtVaSetValues ((Widget) m_buttonWidget, XmNacceleratorText, accel_str, NULL);
+            XmStringFree (accel_str);
         }
     }
 }
 
-void wxMenuItemCallback (Widget w, XtPointer clientData,
-		    XtPointer ptr)
+// ----------------------------------------------------------------------------
+// Motif callbacks
+// ----------------------------------------------------------------------------
+
+void wxMenuItemCallback (Widget WXUNUSED(w), XtPointer clientData,
+                         XtPointer WXUNUSED(ptr))
 {
-  wxMenuItem *item = (wxMenuItem *) clientData;
-  if (item)
+    wxMenuItem *item = (wxMenuItem *) clientData;
+    if (item)
     {
-      if (item->IsCheckable())
-	{
-	  Boolean isChecked = FALSE;
-	  XtVaGetValues ((Widget) item->GetButtonWidget(), XmNset, & isChecked, NULL);
-	  item->SetChecked(isChecked);
-	}
-      if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
-	{
-          wxCommandEvent commandEvent(wxEVT_COMMAND_MENU_SELECTED, item->GetId());
-          commandEvent.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
-          commandEvent.SetInt( item->GetId() );
-
-	  item->GetMenuBar()->GetMenuBarFrame()->GetEventHandler()->ProcessEvent(commandEvent);
-	}
-      else if (item->GetTopMenu())
-	{
-	  wxCommandEvent event (wxEVT_COMMAND_MENU_SELECTED, item->GetId());
-	  event.SetEventObject(item->GetTopMenu());
-          event.SetInt( item->GetId() );
-
-	  item->GetTopMenu()->ProcessCommand (event);
-	}
+        wxCommandEvent event(wxEVT_COMMAND_MENU_SELECTED, item->GetId());
+        event.SetInt( item->GetId() );
+
+        if (item->IsCheckable())
+        {
+            Boolean isChecked = false;
+            XtVaGetValues ((Widget) item->GetButtonWidget(),
+                           XmNset, & isChecked,
+                           NULL);
+
+            // only set the flag, don't actually check anything
+            item->wxMenuItemBase::Check(isChecked);
+            event.SetInt(isChecked);
+        }
+
+        if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
+        {
+            event.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
+
+            item->GetMenuBar()->GetMenuBarFrame()
+                ->GetEventHandler()->ProcessEvent(event);
+        }
+        // this is the child of a popup menu
+        else if (item->GetTopMenu())
+        {
+            event.SetEventObject(item->GetTopMenu());
+
+            item->GetTopMenu()->ProcessCommand (event);
+
+            // Since PopupMenu under Motif still grab right mouse
+            // button events after it was closed, we need to delete
+            // the associated widgets to allow next PopUpMenu to
+            // appear; this needs to be done there because doing it in
+            // a WorkProc as before may cause crashes if a menu item causes
+            // the parent window of the menu to be destroyed
+            item->GetTopMenu()->DestroyWidgetAndDetach();
+        }
     }
 }
 
-void 
-wxMenuItemArmCallback (Widget w, XtPointer clientData,
-		       XtPointer ptr)
+void wxMenuItemArmCallback (Widget WXUNUSED(w), XtPointer clientData,
+                       XtPointer WXUNUSED(ptr))
 {
-  wxMenuItem *item = (wxMenuItem *) clientData;
-  if (item)
+    wxMenuItem *item = (wxMenuItem *) clientData;
+    if (item)
     {
-      if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
-	{
-          wxMenuEvent menuEvent(wxEVT_MENU_HIGHLIGHT, item->GetId());
-          menuEvent.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
+        if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
+        {
+            wxMenuEvent menuEvent(wxEVT_MENU_HIGHLIGHT, item->GetId());
+            menuEvent.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
 
-          item->GetMenuBar()->GetMenuBarFrame()->GetEventHandler()->ProcessEvent(menuEvent);
-	}
+            item->GetMenuBar()->GetMenuBarFrame()
+                ->GetEventHandler()->ProcessEvent(menuEvent);
+        }
     }
 }
 
-void 
-wxMenuItemDisarmCallback (Widget w, XtPointer clientData,
-			  XtPointer ptr)
+void
+wxMenuItemDisarmCallback (Widget WXUNUSED(w), XtPointer clientData,
+                          XtPointer WXUNUSED(ptr))
 {
-  wxMenuItem *item = (wxMenuItem *) clientData;
-  if (item)
+    wxMenuItem *item = (wxMenuItem *) clientData;
+    if (item)
     {
-      if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
-	{
-          // TODO: not sure this is correct, since -1 means something
-          // special to event system
-          wxMenuEvent menuEvent(wxEVT_MENU_HIGHLIGHT, -1);
-          menuEvent.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
-
-          item->GetMenuBar()->GetMenuBarFrame()->GetEventHandler()->ProcessEvent(menuEvent);
-	}
+        if (item->GetMenuBar() && item->GetMenuBar()->GetMenuBarFrame())
+        {
+            // TODO: not sure this is correct, since -1 means something
+            // special to event system
+            wxMenuEvent menuEvent(wxEVT_MENU_HIGHLIGHT, -1);
+            menuEvent.SetEventObject(item->GetMenuBar()->GetMenuBarFrame());
+
+            item->GetMenuBar()->GetMenuBarFrame()
+                ->GetEventHandler()->ProcessEvent(menuEvent);
+        }
     }
 }
-