]> git.saurik.com Git - wxWidgets.git/blobdiff - src/mac/carbon/uma.cpp
projects / wxWidgets.git / blobdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
simplify OnActivate menubar install code
[wxWidgets.git] / src / mac / carbon / uma.cpp
diff --git a/src/mac/carbon/uma.cpp b/src/mac/carbon/uma.cpp
index aedb25e2749b9959ca758aae8cd59d4fef1f690c..d5f59cef31745af5053a90363969c9130cd9f647 100644 (file)
--- a/src/mac/carbon/uma.cpp
+++ b/src/mac/carbon/uma.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////
-// Name:        uma.cpp
+// Name:        src/mac/carbon/uma.cpp
 // Purpose:     UMA support
 // Author:      Stefan Csomor
 // Modified by:
 // Purpose:     UMA support
 // Author:      Stefan Csomor
 // Modified by:
@@ -9,14 +9,14 @@
 // Licence:     The wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:     The wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#include "wx/defs.h"
+#include "wx/wxprec.h"
 
 #if wxUSE_GUI
 
 #include "wx/dc.h"
 
 #if wxUSE_GUI
 
 #include "wx/dc.h"
-#include <MacTextEditor.h>
 
 #ifndef __DARWIN__
 
 #ifndef __DARWIN__
+#  include <MacTextEditor.h>
 #  include <Navigation.h>
 #  if defined(TARGET_CARBON)
 #    if PM_USE_SESSION_APIS
 #  include <Navigation.h>
 #  if defined(TARGET_CARBON)
 #    if PM_USE_SESSION_APIS
@@ -33,14 +33,17 @@
 #endif
 #include "wx/mac/uma.h"
 
 #endif
 #include "wx/mac/uma.h"
 
+#if TARGET_API_MAC_OSX
+#include "wx/toplevel.h"
+#endif
 
 // since we have decided that we only support 8.6 upwards we are
 // checking for these minimum requirements in the startup code of
 
 // since we have decided that we only support 8.6 upwards we are
 // checking for these minimum requirements in the startup code of
-// the application so all wxWindows code can safely assume that appearance 1.1
+// the application so all wxWidgets code can safely assume that appearance 1.1
 // windows manager, control manager, navigation services etc. are
 // present
 
 // windows manager, control manager, navigation services etc. are
 // present
 
-static bool    sUMAHasAppearance = false ;
+static bool sUMAHasAppearance = false ;
 static long sUMAAppearanceVersion = 0 ;
 static long sUMASystemVersion = 0 ;
 static bool sUMAHasAquaLayout = false ;
 static long sUMAAppearanceVersion = 0 ;
 static long sUMASystemVersion = 0 ;
 static bool sUMAHasAquaLayout = false ;
@@ -48,11 +51,12 @@ static bool sUMAHasAquaLayout = false ;
 static bool sUMAHasInittedAppearance = false;
 
 extern int gAGABackgroundColor ;
 static bool sUMAHasInittedAppearance = false;
 
 extern int gAGABackgroundColor ;
+
 bool UMAHasAppearance() { return sUMAHasAppearance ; }
 long UMAGetAppearanceVersion() { return sUMAAppearanceVersion ; }
 long UMAGetSystemVersion() { return sUMASystemVersion ; }
 
 bool UMAHasAppearance() { return sUMAHasAppearance ; }
 long UMAGetAppearanceVersion() { return sUMAAppearanceVersion ; }
 long UMAGetSystemVersion() { return sUMASystemVersion ; }
 
-static bool    sUMAHasWindowManager = false ;
+static bool sUMAHasWindowManager = false ;
 static long sUMAWindowManagerAttr = 0 ;
 
 bool UMAHasWindowManager() { return sUMAHasWindowManager ; }
 static long sUMAWindowManagerAttr = 0 ;
 
 bool UMAHasWindowManager() { return sUMAHasWindowManager ; }
@@ -63,16 +67,15 @@ bool UMAHasAquaLayout() { return sUMAHasAquaLayout ; }
 void UMACleanupToolbox()
 {
     if (sUMAHasInittedAppearance)
 void UMACleanupToolbox()
 {
     if (sUMAHasInittedAppearance)
-    {
         UnregisterAppearanceClient() ;
         UnregisterAppearanceClient() ;
-    }
+
     if ( NavServicesAvailable() )
     if ( NavServicesAvailable() )
-    {
         NavUnload() ;
         NavUnload() ;
-    }
+
   if ( TXNTerminateTextension != (void*) kUnresolvedCFragSymbolAddress )
       TXNTerminateTextension( ) ;
 }
   if ( TXNTerminateTextension != (void*) kUnresolvedCFragSymbolAddress )
       TXNTerminateTextension( ) ;
 }
+
 void UMAInitToolbox( UInt16 inMoreMastersCalls, bool isEmbedded )
 {
 #if !TARGET_CARBON
 void UMAInitToolbox( UInt16 inMoreMastersCalls, bool isEmbedded )
 {
 #if !TARGET_CARBON
@@ -102,12 +105,13 @@ void UMAInitToolbox( UInt16 inMoreMastersCalls, bool isEmbedded )
     long theAppearance ;
     if ( Gestalt( gestaltAppearanceAttr, &theAppearance ) == noErr )
     {
     long theAppearance ;
     if ( Gestalt( gestaltAppearanceAttr, &theAppearance ) == noErr )
     {
-        sUMAHasAppearance = true ;
-        OSStatus status = RegisterAppearanceClient();
         // If status equals appearanceProcessRegisteredErr it means the
         // appearance client already was registered (For example if we run
         // embedded, the host might have registered it). In such a case
         // we don't unregister it later on.
         // If status equals appearanceProcessRegisteredErr it means the
         // appearance client already was registered (For example if we run
         // embedded, the host might have registered it). In such a case
         // we don't unregister it later on.
+
+        sUMAHasAppearance = true ;
+        OSStatus status = RegisterAppearanceClient();
         if (status != appearanceProcessRegisteredErr)
         {
             // Appearance client wasn't registered yet.
         if (status != appearanceProcessRegisteredErr)
         {
             // Appearance client wasn't registered yet.
@@ -115,18 +119,13 @@ void UMAInitToolbox( UInt16 inMoreMastersCalls, bool isEmbedded )
         }
 
         if ( Gestalt( gestaltAppearanceVersion, &theAppearance ) == noErr )
         }
 
         if ( Gestalt( gestaltAppearanceVersion, &theAppearance ) == noErr )
-        {
             sUMAAppearanceVersion = theAppearance ;
             sUMAAppearanceVersion = theAppearance ;
-        }
         else
         else
-        {
             sUMAAppearanceVersion = 0x0100 ;
             sUMAAppearanceVersion = 0x0100 ;
-        }
     }
     }
+
     if ( Gestalt( gestaltWindowMgrAttr, &sUMAWindowManagerAttr ) == noErr )
     if ( Gestalt( gestaltWindowMgrAttr, &sUMAWindowManagerAttr ) == noErr )
-    {
         sUMAHasWindowManager = sUMAWindowManagerAttr & gestaltWindowMgrPresent ;
         sUMAHasWindowManager = sUMAWindowManagerAttr & gestaltWindowMgrPresent ;
-    }
 
 #if TARGET_CARBON
 // Call currently implicitely done :        InitFloatingWindows() ;
 
 #if TARGET_CARBON
 // Call currently implicitely done :        InitFloatingWindows() ;
@@ -141,52 +140,49 @@ void UMAInitToolbox( UInt16 inMoreMastersCalls, bool isEmbedded )
 #endif
 
     if ( NavServicesAvailable() )
 #endif
 
     if ( NavServicesAvailable() )
-    {
         NavLoad() ;
         NavLoad() ;
-    }
 
 
-  long menuMgrAttr ;
-  Gestalt( gestaltMenuMgrAttr , &menuMgrAttr ) ;
-  if ( menuMgrAttr & gestaltMenuMgrAquaLayoutMask )
-    sUMAHasAquaLayout = true ;
-
-  if ( TXNInitTextension != (void*) kUnresolvedCFragSymbolAddress )
-  {
-    FontFamilyID fontId ;
-    Str255 fontName ;
-    SInt16 fontSize ;
-    Style fontStyle ;
-    GetThemeFont(kThemeSmallSystemFont , GetApplicationScript() , fontName , &fontSize , &fontStyle ) ;
-    GetFNum( fontName, &fontId );
-
-    TXNMacOSPreferredFontDescription fontDescriptions[] =
+    long menuMgrAttr ;
+    Gestalt( gestaltMenuMgrAttr , &menuMgrAttr ) ;
+    if ( menuMgrAttr & gestaltMenuMgrAquaLayoutMask )
+        sUMAHasAquaLayout = true ;
+
+    if ( TXNInitTextension != (void*) kUnresolvedCFragSymbolAddress )
     {
     {
-        { fontId , (fontSize << 16) ,kTXNDefaultFontStyle, kTXNSystemDefaultEncoding }
-    } ;
-    int noOfFontDescriptions = sizeof( fontDescriptions ) / sizeof(TXNMacOSPreferredFontDescription) ;
+        FontFamilyID fontId ;
+        Str255 fontName ;
+        SInt16 fontSize ;
+        Style fontStyle ;
 
 
-    OptionBits options = 0 ;
+        GetThemeFont(kThemeSmallSystemFont , GetApplicationScript() , fontName , &fontSize , &fontStyle ) ;
+        GetFNum( fontName, &fontId );
 
 
-    if ( UMAGetSystemVersion() < 0x1000 )
-    {
-        options |= kTXNAlwaysUseQuickDrawTextMask ;
-    }
-    TXNInitTextension(fontDescriptions,  noOfFontDescriptions, options );
-  }
+        TXNMacOSPreferredFontDescription fontDescriptions[] =
+        {
+            { fontId , (fontSize << 16) , kTXNDefaultFontStyle, kTXNSystemDefaultEncoding }
+        } ;
+        int noOfFontDescriptions = sizeof( fontDescriptions ) / sizeof(TXNMacOSPreferredFontDescription) ;
 
 
+        OptionBits options = 0 ;
 
 
-  UMASetSystemIsInitialized(true);
+        if ( UMAGetSystemVersion() < 0x1000 )
+            options |= kTXNAlwaysUseQuickDrawTextMask ;
 
 
+        TXNInitTextension( fontDescriptions,  noOfFontDescriptions, options );
+    }
+
+    UMASetSystemIsInitialized( true );
 }
 
 }
 
-/*
+#if 0
 Boolean CanUseATSUI()
 Boolean CanUseATSUI()
-    {
+{
     long result;
     OSErr err = Gestalt(gestaltATSUVersion, &result);
     return (err == noErr);
     long result;
     OSErr err = Gestalt(gestaltATSUVersion, &result);
     return (err == noErr);
-    }
-*/
+}
+#endif
+
 // process manager
 long UMAGetProcessMode()
 {
 // process manager
 long UMAGetProcessMode()
 {
@@ -194,7 +190,7 @@ long UMAGetProcessMode()
     ProcessInfoRec processinfo;
     ProcessSerialNumber procno ;
 
     ProcessInfoRec processinfo;
     ProcessSerialNumber procno ;
 
-    procno.highLongOfPSN = NULL ;
+    procno.highLongOfPSN = 0 ;
     procno.lowLongOfPSN = kCurrentProcess ;
     processinfo.processInfoLength = sizeof(ProcessInfoRec);
     processinfo.processName = NULL;
     procno.lowLongOfPSN = kCurrentProcess ;
     processinfo.processInfoLength = sizeof(ProcessInfoRec);
     processinfo.processName = NULL;
@@ -202,6 +198,7 @@ long UMAGetProcessMode()
 
     err = ::GetProcessInformation( &procno , &processinfo ) ;
     wxASSERT( err == noErr ) ;
 
     err = ::GetProcessInformation( &procno , &processinfo ) ;
     wxASSERT( err == noErr ) ;
+
     return processinfo.processMode ;
 }
 
     return processinfo.processMode ;
 }
 
@@ -216,6 +213,7 @@ MenuRef UMANewMenu( SInt16 id , const wxString& title , wxFontEncoding encoding
 {
     wxString str = wxStripMenuCodes( title ) ;
     MenuRef menu ;
 {
     wxString str = wxStripMenuCodes( title ) ;
     MenuRef menu ;
+
 #if TARGET_CARBON
     CreateNewMenu( id , 0 , &menu ) ;
     SetMenuTitleWithCFString( menu , wxMacCFStringHolder(str , encoding ) ) ;
 #if TARGET_CARBON
     CreateNewMenu( id , 0 , &menu ) ;
     SetMenuTitleWithCFString( menu , wxMacCFStringHolder(str , encoding ) ) ;
@@ -224,14 +222,17 @@ MenuRef UMANewMenu( SInt16 id , const wxString& title , wxFontEncoding encoding
     wxMacStringToPascal( str , ptitle ) ;
     menu = ::NewMenu( id , ptitle ) ;
 #endif
     wxMacStringToPascal( str , ptitle ) ;
     menu = ::NewMenu( id , ptitle ) ;
 #endif
+
     return menu ;
 }
 
     return menu ;
 }
 
-void UMASetMenuTitle( MenuRef menu , const wxString& title , wxFontEncoding encoding)
+void UMASetMenuTitle( MenuRef menu , const wxString& title , wxFontEncoding encoding )
 {
     wxString str = wxStripMenuCodes( title ) ;
 {
     wxString str = wxStripMenuCodes( title ) ;
+
 #if TARGET_CARBON
     SetMenuTitleWithCFString( menu , wxMacCFStringHolder(str , encoding) ) ;
 #if TARGET_CARBON
     SetMenuTitleWithCFString( menu , wxMacCFStringHolder(str , encoding) ) ;
+
 #else
     Str255 ptitle ;
     wxMacStringToPascal( str , ptitle ) ;
 #else
     Str255 ptitle ;
     wxMacStringToPascal( str , ptitle ) ;
@@ -239,11 +240,14 @@ void UMASetMenuTitle( MenuRef menu , const wxString& title , wxFontEncoding enco
 #endif
 }
 
 #endif
 }
 
-void UMASetMenuItemText(  MenuRef menu,  MenuItemIndex item, const wxString& title , wxFontEncoding encoding)
+void UMASetMenuItemText( MenuRef menu,  MenuItemIndex item, const wxString& title, wxFontEncoding encoding )
 {
 {
-    wxString str = wxStripMenuCodes( title ) ;
+    // we don't strip the accels here anymore, must be done before
+    wxString str = title ;
+
 #if TARGET_CARBON
     SetMenuItemTextWithCFString( menu , item , wxMacCFStringHolder(str , encoding) ) ;
 #if TARGET_CARBON
     SetMenuItemTextWithCFString( menu , item , wxMacCFStringHolder(str , encoding) ) ;
+
 #else
     Str255 ptitle ;
     wxMacStringToPascal( str , ptitle ) ;
 #else
     Str255 ptitle ;
     wxMacStringToPascal( str , ptitle ) ;
@@ -251,10 +255,9 @@ void UMASetMenuItemText(  MenuRef menu,  MenuItemIndex item, const wxString& tit
 #endif
 }
 
 #endif
 }
 
-
 UInt32 UMAMenuEvent( EventRecord *inEvent )
 {
 UInt32 UMAMenuEvent( EventRecord *inEvent )
 {
-        return MenuEvent( inEvent ) ;
+    return MenuEvent( inEvent ) ;
 }
 
 void UMAEnableMenuItem( MenuRef inMenu , MenuItemIndex inItem , bool enable)
 }
 
 void UMAEnableMenuItem( MenuRef inMenu , MenuItemIndex inItem , bool enable)
@@ -267,16 +270,16 @@ void UMAEnableMenuItem( MenuRef inMenu , MenuItemIndex inItem , bool enable)
 
 void UMAAppendSubMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , SInt16 id )
 {
 
 void UMAAppendSubMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , SInt16 id )
 {
-    MacAppendMenu(menu, "\pA");
-    UMASetMenuItemText(menu, (SInt16) ::CountMenuItems(menu), title , encoding );
+    MacAppendMenu( menu, "\pA" );
+    UMASetMenuItemText( menu, (SInt16) ::CountMenuItems(menu), title , encoding );
     SetMenuItemHierarchicalID( menu , CountMenuItems( menu ) , id ) ;
 }
 
 void UMAInsertSubMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , MenuItemIndex item , SInt16 id  )
 {
     SetMenuItemHierarchicalID( menu , CountMenuItems( menu ) , id ) ;
 }
 
 void UMAInsertSubMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , MenuItemIndex item , SInt16 id  )
 {
-    MacInsertMenuItem(menu, "\pA" , item);
-    UMASetMenuItemText(menu, item , title , encoding);
-    SetMenuItemHierarchicalID( menu , item , id ) ;
+    MacInsertMenuItem( menu, "\pA" , item );
+    UMASetMenuItemText( menu, item+1, title , encoding );
+    SetMenuItemHierarchicalID( menu , item+1 , id ) ;
 }
 
 void UMASetMenuItemShortcut( MenuRef menu , MenuItemIndex item , wxAcceleratorEntry *entry )
 }
 
 void UMASetMenuItemShortcut( MenuRef menu , MenuItemIndex item , wxAcceleratorEntry *entry )
@@ -288,27 +291,21 @@ void UMASetMenuItemShortcut( MenuRef menu , MenuItemIndex item , wxAcceleratorEn
     SInt16 key = entry->GetKeyCode() ;
     if ( key )
     {
     SInt16 key = entry->GetKeyCode() ;
     if ( key )
     {
-        bool explicitCommandKey = false ;
-
-        if ( entry->GetFlags() & wxACCEL_CTRL )
-        {
-            explicitCommandKey = true ;
-        }
+        bool explicitCommandKey = (entry->GetFlags() & wxACCEL_CTRL);
 
 
-        if (entry->GetFlags() & wxACCEL_ALT )
-        {
+        if (entry->GetFlags() & wxACCEL_ALT)
             modifiers |= kMenuOptionModifier ;
             modifiers |= kMenuOptionModifier ;
-        }
 
         if (entry->GetFlags() & wxACCEL_SHIFT)
 
         if (entry->GetFlags() & wxACCEL_SHIFT)
-        {
             modifiers |= kMenuShiftModifier ;
             modifiers |= kMenuShiftModifier ;
-        }
 
         SInt16 glyph = 0 ;
         SInt16 macKey = key ;
         if ( key >= WXK_F1 && key <= WXK_F15 )
         {
 
         SInt16 glyph = 0 ;
         SInt16 macKey = key ;
         if ( key >= WXK_F1 && key <= WXK_F15 )
         {
+            if ( !explicitCommandKey )
+                modifiers |= kMenuNoCommandModifier ;
+
             // for some reasons this must be 0 right now
             // everything else leads to just the first function key item
             // to be selected. Thanks to Ryan Wilcox for finding out.
             // for some reasons this must be 0 right now
             // everything else leads to just the first function key item
             // to be selected. Thanks to Ryan Wilcox for finding out.
@@ -316,90 +313,129 @@ void UMASetMenuItemShortcut( MenuRef menu , MenuItemIndex item , wxAcceleratorEn
             glyph = kMenuF1Glyph + ( key - WXK_F1 ) ;
             if ( key >= WXK_F13 )
                 glyph += 13 ;
             glyph = kMenuF1Glyph + ( key - WXK_F1 ) ;
             if ( key >= WXK_F13 )
                 glyph += 13 ;
-            if ( !explicitCommandKey )
-                modifiers |= kMenuNoCommandModifier ;
-          }
+        }
         else
         {
         else
         {
-            switch( key )
+            switch ( key )
             {
                 case WXK_BACK :
                     macKey = kBackspaceCharCode ;
                     glyph = kMenuDeleteLeftGlyph ;
                     break ;
             {
                 case WXK_BACK :
                     macKey = kBackspaceCharCode ;
                     glyph = kMenuDeleteLeftGlyph ;
                     break ;
+
                 case WXK_TAB :
                     macKey = kTabCharCode ;
                     glyph = kMenuTabRightGlyph ;
                     break ;
                 case WXK_TAB :
                     macKey = kTabCharCode ;
                     glyph = kMenuTabRightGlyph ;
                     break ;
+
                 case kEnterCharCode :
                     macKey = kEnterCharCode ;
                     glyph = kMenuEnterGlyph ;
                     break ;
                 case kEnterCharCode :
                     macKey = kEnterCharCode ;
                     glyph = kMenuEnterGlyph ;
                     break ;
+
                 case WXK_RETURN :
                     macKey = kReturnCharCode ;
                     glyph = kMenuReturnGlyph ;
                     break ;
                 case WXK_RETURN :
                     macKey = kReturnCharCode ;
                     glyph = kMenuReturnGlyph ;
                     break ;
+
                 case WXK_ESCAPE :
                     macKey = kEscapeCharCode ;
                     glyph = kMenuEscapeGlyph ;
                     break ;
                 case WXK_ESCAPE :
                     macKey = kEscapeCharCode ;
                     glyph = kMenuEscapeGlyph ;
                     break ;
+
                 case WXK_SPACE :
                     macKey = ' ' ;
                     glyph = kMenuSpaceGlyph ;
                     break ;
                 case WXK_SPACE :
                     macKey = ' ' ;
                     glyph = kMenuSpaceGlyph ;
                     break ;
+
                 case WXK_DELETE :
                     macKey = kDeleteCharCode ;
                     glyph = kMenuDeleteRightGlyph ;
                     break ;
                 case WXK_DELETE :
                     macKey = kDeleteCharCode ;
                     glyph = kMenuDeleteRightGlyph ;
                     break ;
+
                 case WXK_CLEAR :
                     macKey = kClearCharCode ;
                     glyph = kMenuClearGlyph ;
                     break ;
                 case WXK_CLEAR :
                     macKey = kClearCharCode ;
                     glyph = kMenuClearGlyph ;
                     break ;
-                case WXK_PRIOR : // PAGE UP
+
+                case WXK_PAGEUP :
                     macKey = kPageUpCharCode ;
                     glyph = kMenuPageUpGlyph ;
                     break ;
                     macKey = kPageUpCharCode ;
                     glyph = kMenuPageUpGlyph ;
                     break ;
-                case WXK_NEXT :
+
+                case WXK_PAGEDOWN :
                     macKey = kPageDownCharCode ;
                     glyph = kMenuPageDownGlyph ;
                     break ;
                     macKey = kPageDownCharCode ;
                     glyph = kMenuPageDownGlyph ;
                     break ;
+
                 case WXK_LEFT :
                     macKey = kLeftArrowCharCode ;
                     glyph = kMenuLeftArrowGlyph ;
                     break ;
                 case WXK_LEFT :
                     macKey = kLeftArrowCharCode ;
                     glyph = kMenuLeftArrowGlyph ;
                     break ;
+
                 case WXK_UP :
                     macKey = kUpArrowCharCode ;
                     glyph = kMenuUpArrowGlyph ;
                     break ;
                 case WXK_UP :
                     macKey = kUpArrowCharCode ;
                     glyph = kMenuUpArrowGlyph ;
                     break ;
+
                 case WXK_RIGHT :
                     macKey = kRightArrowCharCode ;
                     glyph = kMenuRightArrowGlyph ;
                     break ;
                 case WXK_RIGHT :
                     macKey = kRightArrowCharCode ;
                     glyph = kMenuRightArrowGlyph ;
                     break ;
+
                 case WXK_DOWN :
                     macKey = kDownArrowCharCode ;
                     glyph = kMenuDownArrowGlyph ;
                     break ;
                 case WXK_DOWN :
                     macKey = kDownArrowCharCode ;
                     glyph = kMenuDownArrowGlyph ;
                     break ;
+
+                case WXK_HOME :
+                    macKey = kHomeCharCode ;
+                    glyph = kMenuNorthwestArrowGlyph ;
+                    break ;
+
+                case WXK_END :
+                    macKey = kEndCharCode ;
+                    glyph = kMenuSoutheastArrowGlyph ;
+                    break ;
+                default :
+                    macKey = toupper( key ) ;
+                    break ;
             }
             }
+
+            // we now allow non command key shortcuts
+            // remove in case this gives problems
+            if ( !explicitCommandKey )
+                modifiers |= kMenuNoCommandModifier ;
         }
 
         }
 
-        SetItemCmd( menu, item , macKey );
-        SetMenuItemModifiers(menu, item , modifiers ) ;
+        // 1d and 1e have special meaning to SetItemCmd, so
+        // do not use for these character codes.
+        if (key != WXK_UP && key != WXK_RIGHT && key != WXK_DOWN && key != WXK_LEFT)
+            SetItemCmd( menu, item , macKey );
+
+        SetMenuItemModifiers( menu, item , modifiers ) ;
 
         if ( glyph )
 
         if ( glyph )
-            SetMenuItemKeyGlyph(menu, item , glyph ) ;
+            SetMenuItemKeyGlyph( menu, item , glyph ) ;
     }
 }
 
 void UMAAppendMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , wxAcceleratorEntry *entry )
 {
     MacAppendMenu(menu, "\pA");
     }
 }
 
 void UMAAppendMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , wxAcceleratorEntry *entry )
 {
     MacAppendMenu(menu, "\pA");
+
+    // don't attempt to interpret metacharacters like a '-' at the beginning (would become a separator otherwise)
+    ChangeMenuItemAttributes( menu , ::CountMenuItems(menu), kMenuItemAttrIgnoreMeta , 0 ) ;
     UMASetMenuItemText(menu, (SInt16) ::CountMenuItems(menu), title , encoding );
     UMASetMenuItemText(menu, (SInt16) ::CountMenuItems(menu), title , encoding );
-    UMASetMenuItemShortcut( menu ,  (SInt16) ::CountMenuItems(menu), entry ) ;
+    UMASetMenuItemShortcut( menu , (SInt16) ::CountMenuItems(menu), entry ) ;
 }
 
 void UMAInsertMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , MenuItemIndex item , wxAcceleratorEntry *entry )
 {
     MacInsertMenuItem( menu , "\pA" , item) ;
 }
 
 void UMAInsertMenuItem( MenuRef menu , const wxString& title, wxFontEncoding encoding , MenuItemIndex item , wxAcceleratorEntry *entry )
 {
     MacInsertMenuItem( menu , "\pA" , item) ;
+
+    // don't attempt to interpret metacharacters like a '-' at the beginning (would become a separator otherwise)
+    ChangeMenuItemAttributes( menu , item+1, kMenuItemAttrIgnoreMeta , 0 ) ;
     UMASetMenuItemText(menu, item+1 , title , encoding );
     UMASetMenuItemShortcut( menu , item+1 , entry ) ;
 }
     UMASetMenuItemText(menu, item+1 , title , encoding );
     UMASetMenuItemShortcut( menu , item+1 , entry ) ;
 }
@@ -413,32 +449,39 @@ int gPrOpenCounter = 0 ;
 OSStatus UMAPrOpen()
 {
     OSErr err = noErr ;
 OSStatus UMAPrOpen()
 {
     OSErr err = noErr ;
+
     ++gPrOpenCounter ;
     ++gPrOpenCounter ;
+
     if ( gPrOpenCounter == 1 )
     {
         PrOpen() ;
         err = PrError() ;
         wxASSERT( err == noErr ) ;
     }
     if ( gPrOpenCounter == 1 )
     {
         PrOpen() ;
         err = PrError() ;
         wxASSERT( err == noErr ) ;
     }
+
     return err ;
 }
 
 OSStatus UMAPrClose()
 {
     OSErr err = noErr ;
     return err ;
 }
 
 OSStatus UMAPrClose()
 {
     OSErr err = noErr ;
+
     wxASSERT( gPrOpenCounter >= 1 ) ;
     wxASSERT( gPrOpenCounter >= 1 ) ;
+
     if ( gPrOpenCounter == 1 )
     {
         PrClose() ;
         err = PrError() ;
         wxASSERT( err == noErr ) ;
     }
     if ( gPrOpenCounter == 1 )
     {
         PrClose() ;
         err = PrError() ;
         wxASSERT( err == noErr ) ;
     }
+
     --gPrOpenCounter ;
     --gPrOpenCounter ;
+
     return err ;
 }
 
     return err ;
 }
 
-pascal QDGlobalsPtr GetQDGlobalsPtr (void) ;
-pascal QDGlobalsPtr GetQDGlobalsPtr (void)
+pascal QDGlobalsPtr GetQDGlobalsPtr() ;
+pascal QDGlobalsPtr GetQDGlobalsPtr()
 {
     return QDGlobalsPtr (* (Ptr*) LMGetCurrentA5 ( ) - 0xCA);
 }
 {
     return QDGlobalsPtr (* (Ptr*) LMGetCurrentA5 ( ) - 0xCA);
 }
@@ -449,30 +492,32 @@ void UMAShowWatchCursor()
 {
     OSErr err = noErr;
 
 {
     OSErr err = noErr;
 
-    CursHandle watchFob = GetCursor (watchCursor);
+    CursHandle watchFob = GetCursor(watchCursor);
 
 
-    if (!watchFob)
+    if (watchFob == NULL)
+    {
         err = nilHandleErr;
         err = nilHandleErr;
+    }
     else
     {
     else
     {
-    #if TARGET_CARBON
+#if TARGET_CARBON
 //        Cursor preservedArrow;
 //        Cursor preservedArrow;
-//        GetQDGlobalsArrow (&preservedArrow);
-//        SetQDGlobalsArrow (*watchFob);
-//        InitCursor ( );
-//        SetQDGlobalsArrow (&preservedArrow);
-        SetCursor (*watchFob);
-    #else
-        SetCursor (*watchFob);
-    #endif
+//        GetQDGlobalsArrow(&preservedArrow);
+//        SetQDGlobalsArrow(*watchFob);
+//        InitCursor();
+//        SetQDGlobalsArrow(&preservedArrow);
+        SetCursor(*watchFob);
+#else
+        SetCursor(*watchFob);
+#endif
     }
 }
 
     }
 }
 
-void            UMAShowArrowCursor()
+void UMAShowArrowCursor()
 {
 #if TARGET_CARBON
     Cursor arrow;
 {
 #if TARGET_CARBON
     Cursor arrow;
-    SetCursor (GetQDGlobalsArrow (&arrow));
+    SetCursor( GetQDGlobalsArrow(&arrow) );
 #else
     SetCursor (&(qd.arrow));
 #endif
 #else
     SetCursor (&(qd.arrow));
 #endif
@@ -480,9 +525,10 @@ void            UMAShowArrowCursor()
 
 // window manager
 
 
 // window manager
 
-GrafPtr        UMAGetWindowPort( WindowRef inWindowRef )
+GrafPtr UMAGetWindowPort( WindowRef inWindowRef )
 {
     wxASSERT( inWindowRef != NULL ) ;
 {
     wxASSERT( inWindowRef != NULL ) ;
+
 #if TARGET_CARBON
     return (GrafPtr) GetWindowPort( inWindowRef ) ;
 #else
 #if TARGET_CARBON
     return (GrafPtr) GetWindowPort( inWindowRef ) ;
 #else
@@ -490,16 +536,18 @@ GrafPtr        UMAGetWindowPort( WindowRef inWindowRef )
 #endif
 }
 
 #endif
 }
 
-void             UMADisposeWindow( WindowRef inWindowRef )
+void UMADisposeWindow( WindowRef inWindowRef )
 {
     wxASSERT( inWindowRef != NULL ) ;
 {
     wxASSERT( inWindowRef != NULL ) ;
+
     DisposeWindow( inWindowRef ) ;
 }
 
     DisposeWindow( inWindowRef ) ;
 }
 
-void UMASetWTitle( WindowRef inWindowRef , const wxString& title , wxFontEncoding encoding)
+void UMASetWTitle( WindowRef inWindowRef , const wxString& title , wxFontEncoding encoding )
 {
 #if TARGET_CARBON
     SetWindowTitleWithCFString( inWindowRef , wxMacCFStringHolder(title , encoding) ) ;
 {
 #if TARGET_CARBON
     SetWindowTitleWithCFString( inWindowRef , wxMacCFStringHolder(title , encoding) ) ;
+
 #else
     Str255 ptitle ;
     wxMacStringToPascal( title , ptitle ) ;
 #else
     Str255 ptitle ;
     wxMacStringToPascal( title , ptitle ) ;
@@ -509,10 +557,11 @@ void UMASetWTitle( WindowRef inWindowRef , const wxString& title , wxFontEncodin
 
 // appearance additions
 
 
 // appearance additions
 
-void UMASetControlTitle( ControlRef inControl , const wxString& title , wxFontEncoding encoding)
+void UMASetControlTitle( ControlRef inControl , const wxString& title , wxFontEncoding encoding )
 {
 #if TARGET_CARBON
     SetControlTitleWithCFString( inControl , wxMacCFStringHolder(title , encoding) ) ;
 {
 #if TARGET_CARBON
     SetControlTitleWithCFString( inControl , wxMacCFStringHolder(title , encoding) ) ;
+
 #else
     Str255 ptitle ;
     wxMacStringToPascal( title , ptitle ) ;
 #else
     Str255 ptitle ;
     wxMacStringToPascal( title , ptitle ) ;
@@ -522,7 +571,10 @@ void UMASetControlTitle( ControlRef inControl , const wxString& title , wxFontEn
 
 void UMAActivateControl( ControlRef inControl )
 {
 
 void UMAActivateControl( ControlRef inControl )
 {
-#if !TARGET_API_MAC_OSX
+#if TARGET_API_MAC_OSX
+    ::ActivateControl( inControl ) ;
+
+#else
     // we have to add the control after again to the update rgn
     // otherwise updates get lost
     if ( !IsControlActive( inControl ) )
     // we have to add the control after again to the update rgn
     // otherwise updates get lost
     if ( !IsControlActive( inControl ) )
@@ -530,140 +582,124 @@ void UMAActivateControl( ControlRef inControl )
         bool visible = IsControlVisible( inControl ) ;
         if ( visible )
             SetControlVisibility( inControl , false , false ) ;
         bool visible = IsControlVisible( inControl ) ;
         if ( visible )
             SetControlVisibility( inControl , false , false ) ;
-#endif    
+
         ::ActivateControl( inControl ) ;
         ::ActivateControl( inControl ) ;
-#if !TARGET_API_MAC_OSX
-        if ( visible ) {
+
+        if ( visible )
+        {
             SetControlVisibility( inControl , true , false ) ;
             SetControlVisibility( inControl , true , false ) ;
-        
+
             Rect ctrlBounds ;
             Rect ctrlBounds ;
-            InvalWindowRect(GetControlOwner(inControl),UMAGetControlBoundsInWindowCoords(inControl,&ctrlBounds) ) ;
+            InvalWindowRect( GetControlOwner(inControl), UMAGetControlBoundsInWindowCoords(inControl, &ctrlBounds) ) ;
         }
     }
         }
     }
-#endif    
-}
-
-void UMADrawControl( ControlRef inControl )
-{
-#if TARGET_API_MAC_CARBON
-    ::Draw1Control( inControl );
-#else
-    WindowRef theWindow = GetControlOwner(inControl) ;
-    wxMacPortStateHelper help( (GrafPtr) GetWindowPort(theWindow) ) ;
-    RgnHandle updateRgn = NewRgn() ;
-    GetWindowUpdateRgn( theWindow , updateRgn ) ;
-    Point zero = { 0 , 0 } ;
-    LocalToGlobal( &zero ) ;
-    OffsetRgn( updateRgn , -zero.h , -zero.v ) ;
-    ::DrawControlInCurrentPort( inControl ) ;
-    InvalWindowRgn( theWindow, updateRgn) ;
-    DisposeRgn( updateRgn ) ;
 #endif
 }
 
 void UMAMoveControl( ControlRef inControl , short x , short y )
 {
 #endif
 }
 
 void UMAMoveControl( ControlRef inControl , short x , short y )
 {
-#if !TARGET_API_MAC_OSX
+#if TARGET_API_MAC_OSX
+    ::MoveControl( inControl , x , y ) ;
+
+#else
     bool visible = IsControlVisible( inControl ) ;
     bool visible = IsControlVisible( inControl ) ;
-    if ( visible ) {
+    if ( visible )
+    {
         SetControlVisibility( inControl , false , false ) ;
         Rect ctrlBounds ;
         SetControlVisibility( inControl , false , false ) ;
         Rect ctrlBounds ;
-        InvalWindowRect(GetControlOwner(inControl),GetControlBounds(inControl,&ctrlBounds) ) ;
+        InvalWindowRect( GetControlOwner(inControl), GetControlBounds(inControl, &ctrlBounds) ) ;
     }
     }
-#endif    
+
     ::MoveControl( inControl , x , y ) ;
     ::MoveControl( inControl , x , y ) ;
-#if !TARGET_API_MAC_OSX
-    if ( visible ) {
+
+    if ( visible )
+    {
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
-        InvalWindowRect(GetControlOwner(inControl),GetControlBounds(inControl,&ctrlBounds) ) ;
+        InvalWindowRect( GetControlOwner(inControl), GetControlBounds(inControl, &ctrlBounds) ) ;
     }
 #endif
 }
 
 void UMASizeControl( ControlRef inControl , short x , short y )
 {
     }
 #endif
 }
 
 void UMASizeControl( ControlRef inControl , short x , short y )
 {
-#if !TARGET_API_MAC_OSX
+#if TARGET_API_MAC_OSX
+    ::SizeControl( inControl , x , y ) ;
+
+#else
     bool visible = IsControlVisible( inControl ) ;
     bool visible = IsControlVisible( inControl ) ;
-    if ( visible ) {
+    if ( visible )
+    {
         SetControlVisibility( inControl , false , false ) ;
         Rect ctrlBounds ;
         SetControlVisibility( inControl , false , false ) ;
         Rect ctrlBounds ;
-        InvalWindowRect(GetControlOwner(inControl),GetControlBounds(inControl,&ctrlBounds) ) ;
+        InvalWindowRect( GetControlOwner(inControl), GetControlBounds(inControl, &ctrlBounds) ) ;
     }
     }
-#endif
+
     ::SizeControl( inControl , x , y ) ;
     ::SizeControl( inControl , x , y ) ;
-#if !TARGET_API_MAC_OSX
-    if ( visible ) {
+
+    if ( visible )
+    {
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
-        InvalWindowRect(GetControlOwner(inControl),GetControlBounds(inControl,&ctrlBounds) ) ;
+        InvalWindowRect( GetControlOwner(inControl), GetControlBounds(inControl, &ctrlBounds) ) ;
     }
 #endif
 }
 
 void UMADeactivateControl( ControlRef inControl )
 {
     }
 #endif
 }
 
 void UMADeactivateControl( ControlRef inControl )
 {
-#if !TARGET_API_MAC_OSX
+#if TARGET_API_MAC_OSX
+    ::DeactivateControl( inControl ) ;
+
+#else
     // we have to add the control after again to the update rgn
     // otherwise updates get lost
     bool visible = IsControlVisible( inControl ) ;
     if ( visible )
         SetControlVisibility( inControl , false , false ) ;
     // we have to add the control after again to the update rgn
     // otherwise updates get lost
     bool visible = IsControlVisible( inControl ) ;
     if ( visible )
         SetControlVisibility( inControl , false , false ) ;
-#endif
+
     ::DeactivateControl( inControl ) ;
     ::DeactivateControl( inControl ) ;
-#if !TARGET_API_MAC_OSX
-    if ( visible ) {
+
+    if ( visible )
+    {
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
         SetControlVisibility( inControl , true , false ) ;
         Rect ctrlBounds ;
-        InvalWindowRect(GetControlOwner(inControl),UMAGetControlBoundsInWindowCoords(inControl,&ctrlBounds) ) ;
+        InvalWindowRect( GetControlOwner(inControl), UMAGetControlBoundsInWindowCoords(inControl, &ctrlBounds) ) ;
     }
 #endif
 }
     }
 #endif
 }
+
 // shows the control and adds the region to the update region
 // shows the control and adds the region to the update region
-void UMAShowControl                        (ControlRef             inControl)
+void UMAShowControl( ControlRef inControl )
 {
     SetControlVisibility( inControl , true , false ) ;
     Rect ctrlBounds ;
 {
     SetControlVisibility( inControl , true , false ) ;
     Rect ctrlBounds ;
-    InvalWindowRect(GetControlOwner(inControl),UMAGetControlBoundsInWindowCoords(inControl,&ctrlBounds) ) ;
+    InvalWindowRect( GetControlOwner(inControl), UMAGetControlBoundsInWindowCoords(inControl, &ctrlBounds) ) ;
 }
 
 // hides the control and adds the region to the update region
 }
 
 // hides the control and adds the region to the update region
-void UMAHideControl                        (ControlRef             inControl)
+void UMAHideControl( ControlRef inControl )
 {
     SetControlVisibility( inControl , false , false ) ;
     Rect ctrlBounds ;
 {
     SetControlVisibility( inControl , false , false ) ;
     Rect ctrlBounds ;
-    InvalWindowRect(GetControlOwner(inControl),UMAGetControlBoundsInWindowCoords(inControl,&ctrlBounds) ) ;
+    InvalWindowRect( GetControlOwner(inControl), UMAGetControlBoundsInWindowCoords(inControl, &ctrlBounds) ) ;
 }
 }
+
 // keyboard focus
 // keyboard focus
-OSErr UMASetKeyboardFocus                (WindowPtr                 inWindow,
-                                 ControlRef             inControl,
-                                 ControlFocusPart         inPart)
+OSErr UMASetKeyboardFocus( WindowPtr inWindow,
+                                 ControlRef inControl,
+                                 ControlFocusPart inPart )
 {
     OSErr err = noErr;
     GrafPtr port ;
 {
     OSErr err = noErr;
     GrafPtr port ;
-    GetPort( &port ) ;
 
 
+    GetPort( &port ) ;
     SetPortWindowPort( inWindow ) ;
 
     err = SetKeyboardFocus( inWindow , inControl , inPart ) ;
     SetPort( port ) ;
     SetPortWindowPort( inWindow ) ;
 
     err = SetKeyboardFocus( inWindow , inControl , inPart ) ;
     SetPort( port ) ;
-    return err ;
-}
 
 
-// events
-void UMAUpdateControls( WindowPtr inWindow , RgnHandle inRgn )
-{
-    wxMacPortStateHelper help( (GrafPtr) GetWindowPort( (WindowRef) inWindow) ) ;
-    RgnHandle updateRgn = NewRgn() ;
-    GetWindowUpdateRgn( inWindow , updateRgn ) ;
-
-    Point zero = { 0 , 0 } ;
-    LocalToGlobal( &zero ) ;
-    OffsetRgn( updateRgn , -zero.h , -zero.v ) ;
-
-    UpdateControls( inWindow , inRgn ) ;
-    InvalWindowRgn( inWindow, updateRgn) ;
-    DisposeRgn( updateRgn ) ;
+    return err ;
 }
 
 bool UMAIsWindowFloating( WindowRef inWindow )
 }
 
 bool UMAIsWindowFloating( WindowRef inWindow )
@@ -689,13 +725,14 @@ void UMAHighlightAndActivateWindow( WindowRef inWindowRef , bool inActivate )
     if ( inWindowRef )
     {
 //        bool isHighlighted = IsWindowHighlited( inWindowRef ) ;
     if ( inWindowRef )
     {
 //        bool isHighlighted = IsWindowHighlited( inWindowRef ) ;
-//        if ( inActivate != isHightlited )
+//        if ( inActivate != isHighlighted )
+
         GrafPtr port ;
         GetPort( &port ) ;
         SetPortWindowPort( inWindowRef ) ;
         HiliteWindow( inWindowRef , inActivate ) ;
         ControlRef control = NULL ;
         GrafPtr port ;
         GetPort( &port ) ;
         SetPortWindowPort( inWindowRef ) ;
         HiliteWindow( inWindowRef , inActivate ) ;
         ControlRef control = NULL ;
-        ::GetRootControl( inWindowRef , & control ) ;
+        ::GetRootControl( inWindowRef , &control ) ;
         if ( control )
         {
             if ( inActivate )
         if ( control )
         {
             if ( inActivate )
@@ -703,6 +740,7 @@ void UMAHighlightAndActivateWindow( WindowRef inWindowRef , bool inActivate )
             else
                 UMADeactivateControl( control ) ;
         }
             else
                 UMADeactivateControl( control ) ;
         }
+
         SetPort( port ) ;
     }
 }
         SetPort( port ) ;
     }
 }
@@ -718,28 +756,29 @@ static MenuItemIndex firstCustomItemIndex = 0 ;
 #endif
 
 OSStatus UMAGetHelpMenu(
 #endif
 
 OSStatus UMAGetHelpMenu(
-  MenuRef *        outHelpMenu,
-  MenuItemIndex *  outFirstCustomItemIndex)
+    MenuRef *        outHelpMenu,
+    MenuItemIndex *  outFirstCustomItemIndex)
 {
 #if TARGET_CARBON
     return HMGetHelpMenu( outHelpMenu , outFirstCustomItemIndex ) ;
 {
 #if TARGET_CARBON
     return HMGetHelpMenu( outHelpMenu , outFirstCustomItemIndex ) ;
+
 #else
     MenuRef helpMenuHandle ;
 #else
     MenuRef helpMenuHandle ;
+
     helpMenuStatus = HMGetHelpMenuHandle( &helpMenuHandle ) ;
     if ( firstCustomItemIndex == 0 && helpMenuStatus == noErr )
     helpMenuStatus = HMGetHelpMenuHandle( &helpMenuHandle ) ;
     if ( firstCustomItemIndex == 0 && helpMenuStatus == noErr )
-    {
         firstCustomItemIndex = CountMenuItems( helpMenuHandle ) + 1 ;
         firstCustomItemIndex = CountMenuItems( helpMenuHandle ) + 1 ;
-    }
+
     if ( outFirstCustomItemIndex )
     if ( outFirstCustomItemIndex )
-    {
         *outFirstCustomItemIndex = firstCustomItemIndex ;
         *outFirstCustomItemIndex = firstCustomItemIndex ;
-    }
+
     *outHelpMenu = helpMenuHandle ;
     *outHelpMenu = helpMenuHandle ;
+
     return helpMenuStatus ;
 #endif
 }
 
     return helpMenuStatus ;
 #endif
 }
 
-wxMacPortStateHelper::wxMacPortStateHelper( GrafPtr newport)
+wxMacPortStateHelper::wxMacPortStateHelper( GrafPtr newport )
 {
     m_clip = NULL ;
     Setup( newport ) ;
 {
     m_clip = NULL ;
     Setup( newport ) ;
@@ -754,17 +793,19 @@ void wxMacPortStateHelper::Setup( GrafPtr newport )
 {
     GetPort( &m_oldPort ) ;
     SetPort( newport ) ;
 {
     GetPort( &m_oldPort ) ;
     SetPort( newport ) ;
-    SetOrigin(0,0);
+    SetOrigin(0, 0);
+
     wxASSERT_MSG( m_clip == NULL , wxT("Cannot call setup twice") ) ;
     m_clip = NewRgn() ;
     GetClip( m_clip );
     wxASSERT_MSG( m_clip == NULL , wxT("Cannot call setup twice") ) ;
     m_clip = NewRgn() ;
     GetClip( m_clip );
-    m_textFont = GetPortTextFont( (CGrafPtr) newport);
-    m_textSize = GetPortTextSize( (CGrafPtr) newport);
-    m_textStyle = GetPortTextFace( (CGrafPtr) newport);
-    m_textMode = GetPortTextMode( (CGrafPtr) newport);
+    m_textFont = GetPortTextFont( (CGrafPtr) newport );
+    m_textSize = GetPortTextSize( (CGrafPtr) newport );
+    m_textStyle = GetPortTextFace( (CGrafPtr) newport );
+    m_textMode = GetPortTextMode( (CGrafPtr) newport );
     GetThemeDrawingState( &m_drawingState ) ;
     m_currentPort = newport ;
 }
     GetThemeDrawingState( &m_drawingState ) ;
     m_currentPort = newport ;
 }
+
 void wxMacPortStateHelper::Clear()
 {
     if ( m_clip )
 void wxMacPortStateHelper::Clear()
 {
     if ( m_clip )
@@ -794,42 +835,39 @@ wxMacPortStateHelper::~wxMacPortStateHelper()
 OSStatus UMAPutScrap( Size size , OSType type , void *data )
 {
     OSStatus err = noErr ;
 OSStatus UMAPutScrap( Size size , OSType type , void *data )
 {
     OSStatus err = noErr ;
+
 #if !TARGET_CARBON
     err = PutScrap( size , type , data ) ;
 #else
     ScrapRef    scrap;
 #if !TARGET_CARBON
     err = PutScrap( size , type , data ) ;
 #else
     ScrapRef    scrap;
-    err = GetCurrentScrap (&scrap);
-    if ( !err )
-    {
-        err = PutScrapFlavor (scrap, type , 0, size, data);
-    }
+    err = GetCurrentScrap( &scrap );
+    if ( err == noErr )
+        err = PutScrapFlavor( scrap, type , 0, size, data );
 #endif
 #endif
+
     return err ;
 }
 
     return err ;
 }
 
-Rect* UMAGetControlBoundsInWindowCoords(ControlRef theControl, Rect *bounds)
+Rect * UMAGetControlBoundsInWindowCoords( ControlRef theControl, Rect *bounds )
 {
 {
-    wxWindow* win = wxFindControlFromMacControl( theControl ) ;
-    
     GetControlBounds( theControl , bounds ) ;
     GetControlBounds( theControl , bounds ) ;
+
 #if TARGET_API_MAC_OSX
 #if TARGET_API_MAC_OSX
-    if ( win != NULL && win->MacGetTopLevelWindow() != NULL )   
+    WindowRef tlwref = GetControlOwner( theControl ) ;
+
+    wxTopLevelWindowMac* tlwwx = wxFindWinFromMacWindow( tlwref ) ;
+    if ( tlwwx != NULL && tlwwx->MacUsesCompositing() )
     {
     {
-        int x , y ;
-        x = 0 ;
-        y = 0 ;
-        
-        win->GetParent()->MacWindowToRootWindow( &x , & y ) ;
-        bounds->left += x ;
-        bounds->right += x ;
-        bounds->top += y ;
-        bounds->bottom += y ;
+        ControlRef rootControl = tlwwx->GetPeer()->GetControlRef() ;
+        HIPoint hiPoint = CGPointMake( 0 , 0 ) ;
+        HIViewConvertPoint( &hiPoint , HIViewGetSuperview(theControl) , rootControl ) ;
+        OffsetRect( bounds , (short) hiPoint.x , (short) hiPoint.y ) ;
     }
 #endif
     }
 #endif
+
     return bounds ;
 }
 
     return bounds ;
 }
 
-
 #endif  // wxUSE_GUI
 
 #if wxUSE_BASE
 #endif  // wxUSE_GUI
 
 #if wxUSE_BASE
@@ -846,5 +884,4 @@ void UMASetSystemIsInitialized(bool val)
     sUMASystemInitialized = val;
 }
 
     sUMASystemInitialized = val;
 }
 
-
 #endif // wxUSE_BASE
 #endif // wxUSE_BASE
wxWidgets (Impactor)