From: David Surovell <davids@osafoundation.org>
Date: Fri, 20 Jan 2006 19:49:10 +0000 (+0000)
Subject: fixed compilation errors in prior checkin; some minor cleanup
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/be6068f6c32d028fa47d710e35e012a3f23eeec1?ds=inline

fixed compilation errors in prior checkin; some minor cleanup


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@37017 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/mac/carbon/frame.cpp b/src/mac/carbon/frame.cpp
index dbc406c768..8a07a0bbc6 100644
--- a/src/mac/carbon/frame.cpp
+++ b/src/mac/carbon/frame.cpp
@@ -107,7 +107,7 @@ wxPoint wxFrame::GetClientAreaOrigin() const
         }
     }
 #endif
-    
+
     return pt;
 }
 
@@ -232,24 +232,23 @@ void wxFrame::DetachMenuBar()
 
 void wxFrame::AttachMenuBar( wxMenuBar *menuBar )
 {
-    wxToplLevelWindowMac* tlw = wxFindWinFromMacWindow(FrontNonFloatingWindow()) ;
+    wxTopLevelWindowMac* tlw = wxFindWinFromMacWindow( FrontNonFloatingWindow() );
+    bool makeCurrent = false;
 
-    bool makeCurrent = false ;
-    
     // if this is already the current menubar or we are the frontmost window
-    if ( m_frameMenuBar == wxMenuBar::MacGetInstalledMenuBar() || tlw == this )
-        makeCurrent = true ;
-    // or we have a situation where this is a App Level Menubar like MDI
-    else if ( tlw != NULL && tlw->GetMenuBar() == NULL && ((wxFrame*)wxTheApp->GetTopWindow()) == this )
-        makeCurrent = true ;
-    
-    wxFrameBase::AttachMenuBar(menuBar);
+    if ( (tlw == this) || (m_frameMenuBar == wxMenuBar::MacGetInstalledMenuBar()) )
+        makeCurrent = true;
+    // or there is an app-level menubar like MDI
+    else if ( (GetMenuBar() == NULL) && (((wxFrame*)wxTheApp->GetTopWindow()) == this) )
+        makeCurrent = true;
+
+    wxFrameBase::AttachMenuBar( menuBar );
 
     if (m_frameMenuBar)
     {
         m_frameMenuBar->SetInvokingWindow( this );
         if (makeCurrent)
-            m_frameMenuBar->MacInstallMenuBar() ;
+            m_frameMenuBar->MacInstallMenuBar();
     }
 }
 
@@ -325,7 +324,7 @@ void wxFrame::SetToolBar(wxToolBar *toolbar)
 {
     if ( m_frameToolBar == toolbar )
         return ;
-    
+
 #if wxMAC_USE_NATIVE_TOOLBAR
     if ( m_frameToolBar )
         m_frameToolBar->MacInstallNativeToolbar( false ) ;