X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/c809f3be86c79dc44854433fec4ccfc030d4f7b9..12e049f6f09a7589e5f761ac50bfc1594785ef88:/src/mac/control.cpp

diff --git a/src/mac/control.cpp b/src/mac/control.cpp
index 15bc46b48e..9e6e919791 100644
--- a/src/mac/control.cpp
+++ b/src/mac/control.cpp
@@ -6,16 +6,20 @@
 // Created:     ??/??/98
 // RCS-ID:      $Id$
 // Copyright:   (c) AUTHOR
-// Licence:   	wxWindows licence
+// Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 #pragma implementation "control.h"
 #endif
 
+#include "wx/defs.h"
+
 #include "wx/control.h"
 #include "wx/panel.h"
 #include "wx/app.h"
+#include "wx/dc.h"
+#include "wx/dcclient.h"
 #include "wx/notebook.h"
 #include "wx/tabctrl.h"
 #include "wx/radiobox.h"
@@ -31,50 +35,121 @@
 IMPLEMENT_ABSTRACT_CLASS(wxControl, wxWindow)
 
 BEGIN_EVENT_TABLE(wxControl, wxWindow)
-	EVT_MOUSE_EVENTS( wxControl::OnMouseEvent ) 
-	EVT_CHAR( wxControl::OnKeyDown ) 
-	EVT_PAINT( wxControl::OnPaint ) 
+    EVT_MOUSE_EVENTS( wxControl::OnMouseEvent ) 
+//    EVT_CHAR( wxControl::OnKeyDown ) 
+    EVT_PAINT( wxControl::OnPaint ) 
 END_EVENT_TABLE()
 #endif
 
-#include <wx/mac/uma.h>
+#include "wx/mac/uma.h"
+#include "wx/mac/private.h"
 
 // Item members
 
+
+#if PRAGMA_STRUCT_ALIGN
+    #pragma options align=mac68k
+#elif PRAGMA_STRUCT_PACKPUSH
+    #pragma pack(push, 2)
+#elif PRAGMA_STRUCT_PACK
+    #pragma pack(2)
+#endif
+
+typedef struct {
+ unsigned short instruction;
+ void (*function)();
+} cdefRec, *cdefPtr, **cdefHandle;
+
+#if PRAGMA_STRUCT_ALIGN
+    #pragma options align=reset
+#elif PRAGMA_STRUCT_PACKPUSH
+    #pragma pack(pop)
+#elif PRAGMA_STRUCT_PACK
+    #pragma pack()
+#endif
+
 ControlActionUPP wxMacLiveScrollbarActionUPP = NULL ;
+wxControl *wxFindControlFromMacControl(ControlHandle inControl ) ;
 
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode ) ;
 pascal void wxMacLiveScrollbarActionProc( ControlHandle control , ControlPartCode partCode )
 {
-	if ( partCode != 0)
+    if ( partCode != 0)
+    {
+        wxControl*  wx = (wxControl*) GetControlReference( control ) ;
+        if ( wx )
+        {
+            wx->MacHandleControlClick( control , partCode ) ;
+        }
+    }
+}
+
+ControlColorUPP wxMacSetupControlBackgroundUPP = NULL ;
+ControlDefUPP wxMacControlActionUPP = NULL ;
+
+pascal SInt32  wxMacControlDefintion(SInt16 varCode, ControlRef theControl, ControlDefProcMessage message, SInt32 param)
+{
+    
+	wxControl*  wx = (wxControl*) wxFindControlFromMacControl( theControl ) ;
+	if ( wx != NULL && wx->IsKindOf( CLASSINFO( wxControl ) ) )
 	{
-		wxControl*	wx = (wxControl*) GetControlReference( control ) ;
-		if ( wx )
-		{
-			wx->MacHandleControlClick( control , partCode ) ;
-		}
+	    if( message == drawCntl )
+	    {
+	        wxMacWindowClipper clip( wx ) ;
+	        return InvokeControlDefUPP( varCode , theControl , message , param , (ControlDefUPP) wx->MacGetControlAction() ) ;
+	    }
+	    else
+	        return InvokeControlDefUPP( varCode , theControl , message , param , (ControlDefUPP) wx->MacGetControlAction() ) ;
 	}
+	return  NULL ;
 }
 
+pascal OSStatus wxMacSetupControlBackground( ControlRef iControl , SInt16 iMessage , SInt16 iDepth , Boolean iIsColor )
+{
+	OSStatus status = noErr ;
+	switch( iMessage )
+	{
+		case kControlMsgSetUpBackground :
+			{
+	        	wxControl*  wx = (wxControl*) GetControlReference( iControl ) ;
+	        	if ( wx != NULL && wx->IsKindOf( CLASSINFO( wxControl ) ) )
+	        	{
+	        		wxDC::MacSetupBackgroundForCurrentPort( wx->MacGetBackgroundBrush() ) ;
+	//				SetThemeBackground( iDepth , iIsColor ) ;
+				}
+				else
+				{
+					status = paramErr ;
+				}
+			}
+			break ;
+		default :
+			status = paramErr ;
+			break ;
+	}
+	return status ;
+}
+ 
 wxControl::wxControl()
 {
-	m_macControl = NULL ;
-	m_macHorizontalBorder = 0 ; // additional pixels around the real control
-	m_macVerticalBorder = 0 ;
+    m_macControl = NULL ;
+    m_macControlAction = NULL ;
+    m_macHorizontalBorder = 0 ; // additional pixels around the real control
+    m_macVerticalBorder = 0 ;
     m_backgroundColour = *wxWHITE;
     m_foregroundColour = *wxBLACK;
 #if WXWIN_COMPATIBILITY
   m_callback = 0;
 #endif // WXWIN_COMPATIBILITY
 
-	if ( wxMacLiveScrollbarActionUPP == NULL )
-	{
-#ifdef __UNIX__
+    if ( wxMacLiveScrollbarActionUPP == NULL )
+    {
+#if defined(UNIVERSAL_INTERFACES_VERSION) && (UNIVERSAL_INTERFACES_VERSION >= 0x0340)
         wxMacLiveScrollbarActionUPP = NewControlActionUPP( wxMacLiveScrollbarActionProc );
 #else
-		wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
+        wxMacLiveScrollbarActionUPP = NewControlActionProc( wxMacLiveScrollbarActionProc ) ;
 #endif
-	}
+    }
 }
 
 bool wxControl::Create(wxWindow *parent, wxWindowID id,
@@ -83,10 +158,16 @@ bool wxControl::Create(wxWindow *parent, wxWindowID id,
                        const wxValidator& validator,
                        const wxString& name)
 {
-	m_macControl = NULL ;
-	m_macHorizontalBorder = 0 ; // additional pixels around the real control
-	m_macVerticalBorder = 0 ;
+    m_macControl = NULL ;
+    m_macHorizontalBorder = 0 ; // additional pixels around the real control
+    m_macVerticalBorder = 0 ;
+    
     bool rval = wxWindow::Create(parent, id, pos, size, style, name);
+    if ( parent )
+    {
+        m_backgroundColour = parent->GetBackgroundColour() ;
+        m_foregroundColour = parent->GetForegroundColour() ;
+    }
     if (rval) {
 #if wxUSE_VALIDATORS
         SetValidator(validator);
@@ -98,48 +179,93 @@ bool wxControl::Create(wxWindow *parent, wxWindowID id,
 wxControl::~wxControl()
 {
     m_isBeingDeleted = TRUE;
+    wxRemoveMacControlAssociation( this ) ;
     // If we delete an item, we should initialize the parent panel,
     // because it could now be invalid.
-    wxPanel *panel = wxDynamicCast(GetParent(), wxPanel);
-    if ( panel )
+    wxWindow *parent = GetParent() ;
+    if ( parent )
     {
-        if (panel->GetDefaultItem() == (wxButton*) this)
-            panel->SetDefaultItem(NULL);
+        if (parent->GetDefaultItem() == (wxButton*) this)
+            parent->SetDefaultItem(NULL);
     }
-    if ( m_macControl )
+    if ( (ControlHandle) m_macControl )
     {
-    	UMADisposeControl( m_macControl ) ;
-    	m_macControl = NULL ;
+        ::DisposeControl( (ControlHandle) m_macControl ) ;
+        m_macControl = NULL ;
     }
 }
 
 void wxControl::SetLabel(const wxString& title)
 {
-	m_label = title ;
+    m_label = wxStripMenuCodes(title) ;
 
-	if ( m_macControl )
-	{
-		Str255 maclabel ;
-		wxString label ;
-	
-		if( wxApp::s_macDefaultEncodingIsPC )
-			label = wxMacMakeMacStringFromPC( title ) ;
-		else
-			label = title ;
-		
+    if ( (ControlHandle) m_macControl )
+    {
+        Str255 maclabel ;
+        wxString label ;
+    
+        if( wxApp::s_macDefaultEncodingIsPC )
+            label = wxMacMakeMacStringFromPC( m_label ) ;
+        else
+            label = m_label ;
+        
 #if TARGET_CARBON
-		c2pstrcpy( (StringPtr) maclabel , label ) ;
+        c2pstrcpy( (StringPtr) maclabel , label ) ;
 #else
-		strcpy( (char *) maclabel , label ) ;
-		c2pstr( (char *) maclabel ) ;
+        strcpy( (char *) maclabel , label ) ;
+        c2pstr( (char *) maclabel ) ;
 #endif
-		::SetControlTitle( m_macControl , maclabel ) ;
-	}
+        ::SetControlTitle( (ControlHandle) m_macControl , maclabel ) ;
+    }
+    Refresh() ;
 }
 
 wxSize wxControl::DoGetBestSize() const
 {
-    return wxSize(20, 20);
+    Rect    bestsize = { 0 , 0 , 0 , 0 } ;
+    short   baselineoffset ;
+    int bestWidth, bestHeight ;
+    ::GetBestControlRect( (ControlHandle) m_macControl , &bestsize , &baselineoffset ) ;
+
+    if ( EmptyRect( &bestsize ) )
+    {
+        baselineoffset = 0;
+        bestsize.left = bestsize.top = 0 ;
+        bestsize.right = 16 ;
+        bestsize.bottom = 16 ;
+        if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
+        {
+            bestsize.bottom = 16 ;
+        }
+        else if ( IsKindOf( CLASSINFO( wxSpinButton ) ) )
+        {
+            bestsize.bottom = 24 ; 
+        }
+    }
+
+    if ( IsKindOf( CLASSINFO( wxButton ) ) )
+    {
+        bestWidth = m_label.Length() * 8 + 12 ;
+        if ( bestWidth < 70 )
+          bestWidth = 70 ;
+    }
+    else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
+    {
+        bestWidth = m_label.Length() * 8 ;
+    }
+    else
+        bestWidth = bestsize.right - bestsize.left ;
+        
+    bestWidth += 2 * m_macHorizontalBorder ;
+
+    bestHeight = bestsize.bottom - bestsize.top ;
+    if ( bestHeight < 10 )
+        bestHeight = 13 ;
+
+    bestHeight += 2 * m_macVerticalBorder;
+
+        
+    return wxSize(bestWidth, bestHeight);
 }
 
 bool wxControl::ProcessCommand (wxCommandEvent & event)
@@ -188,18 +314,18 @@ void wxRemoveMacControlAssociation(wxControl *control)
 }
 
 void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString label , 
-			 const wxPoint& pos,
-			 const wxSize& size, long style,
-			 const wxValidator& validator,
-			 const wxString& name , Rect *outBounds , StringPtr maclabel ) 
+             const wxPoint& pos,
+             const wxSize& size, long style,
+             const wxValidator& validator,
+             const wxString& name , WXRECTPTR outBounds , unsigned char* maclabel ) 
 {
-	m_label = label ;
+    m_label = label ;
   SetName(name);
   if ( &validator )
-  	SetValidator(validator);
+    SetValidator(validator);
 
   m_windowStyle = style;
-  parent->AddChild((wxButton *)this);
+  parent->AddChild(this);
 
   m_backgroundColour = parent->GetBackgroundColour() ;
   m_foregroundColour = parent->GetForegroundColour() ;
@@ -209,364 +335,402 @@ void wxControl::MacPreControlCreate( wxWindow *parent, wxWindowID id, wxString l
   else
       m_windowId = id;
 
-	m_width = size.x ;
-	m_height = size.y ;
-	int x = pos.x ;
-	int y = pos.y ;
-	AdjustForParentClientOrigin(x, y, wxSIZE_USE_EXISTING);
-	m_x = x ;
-	m_y = y ;
-		
-
-	parent->MacClientToRootWindow( &x , &y ) ;
-	outBounds->top = y + m_macVerticalBorder ;
-	outBounds->left = x + m_macHorizontalBorder ;
-	outBounds->bottom = outBounds->top + m_height - 2 * m_macVerticalBorder;
-	outBounds->right = outBounds->left + m_width - 2 * m_macHorizontalBorder ;
-
-	char c_text[255];
-	strcpy( c_text , label ) ;
-	if( wxApp::s_macDefaultEncodingIsPC )
-	{
-		wxMacConvertFromPCForControls( c_text ) ;
-	}
+   // These sizes will be adjusted in MacPostControlCreate
+
+    m_width = size.x ;
+    m_height = size.y ;
+    m_x = pos.x ;
+    m_y = pos.y ;
+        
+    ((Rect*)outBounds)->top = -10;
+    ((Rect*)outBounds)->left = -10;
+    ((Rect*)outBounds)->bottom = 0;
+    ((Rect*)outBounds)->right = 0;
+
+    char c_text[255];
+    strcpy( c_text , label ) ;
+    if( wxApp::s_macDefaultEncodingIsPC )
+    {
+        wxMacConvertFromPCForControls( c_text ) ;
+    }
 
 #if TARGET_CARBON
-	c2pstrcpy( (StringPtr) maclabel , c_text ) ;
+    c2pstrcpy( (StringPtr) maclabel , c_text ) ;
 #else
-	strcpy( (char *) maclabel , c_text ) ;
-	c2pstr( (char *) maclabel ) ;
+    strcpy( (char *) maclabel , c_text ) ;
+    c2pstr( (char *) maclabel ) ;
 #endif
 }
 
 void wxControl::MacPostControlCreate()
 {
-	wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
-	
-	if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
-	{
-		// no font
-	}
-	else if ( IsKindOf( CLASSINFO( wxStaticBox ) ) || IsKindOf( CLASSINFO( wxRadioBox ) ) || IsKindOf( CLASSINFO( wxButton ) ) )
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , "No valid mac control" ) ;
+    
+    if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
+    {
+        // no font
+    }
+    else if ( !UMAHasAquaLayout() && (IsKindOf( CLASSINFO( wxStaticBox ) ) || IsKindOf( CLASSINFO( wxRadioBox ) ) || IsKindOf( CLASSINFO( wxButton ) ) ) )
+    {
+        ControlFontStyleRec     controlstyle ;
+        controlstyle.flags = kControlUseFontMask ;
+        controlstyle.font = kControlFontSmallBoldSystemFont ;
+        
+        ::SetControlFontStyle( (ControlHandle) m_macControl , &controlstyle ) ;
+    }
+    else
+    {
+        ControlFontStyleRec     controlstyle ;
+        controlstyle.flags = kControlUseFontMask ;
+        controlstyle.font = kControlFontSmallSystemFont ;
+        
+        ::SetControlFontStyle( (ControlHandle) m_macControl , &controlstyle ) ;
+    }
+    ControlHandle container = (ControlHandle) GetParent()->MacGetContainerForEmbedding() ;
+    wxASSERT_MSG( container != NULL , wxT("No valid mac container control") ) ;
+    ::EmbedControl( (ControlHandle) m_macControl , container ) ;
+    m_macControlIsShown  = true ;
+
+    wxAssociateControlWithMacControl( (ControlHandle) m_macControl , this ) ;
+	if ( wxMacSetupControlBackgroundUPP == NULL )
 	{
-		ControlFontStyleRec		controlstyle ;
-		controlstyle.flags = kControlUseFontMask ;
-		controlstyle.font = kControlFontSmallBoldSystemFont ;
-		
-		::UMASetControlFontStyle( m_macControl , &controlstyle ) ;
+		wxMacSetupControlBackgroundUPP = NewControlColorUPP( wxMacSetupControlBackground ) ;
 	}
-	else
+	if ( wxMacControlActionUPP == NULL )
 	{
-		ControlFontStyleRec		controlstyle ;
-		controlstyle.flags = kControlUseFontMask ;
-		controlstyle.font = kControlFontSmallSystemFont ;
-		
-		::UMASetControlFontStyle( m_macControl , &controlstyle ) ;
+	    wxMacControlActionUPP = NewControlDefUPP( wxMacControlDefintion ) ;
 	}
-	ControlHandle container = GetParent()->MacGetContainerForEmbedding() ;
-	wxASSERT_MSG( container != NULL , "No valid mac container control" ) ;
-	::UMAEmbedControl( m_macControl , container ) ;
-	m_macControlIsShown  = true ;
-	MacAdjustControlRect() ;
-	wxAssociateControlWithMacControl( m_macControl , this ) ;
+#if TARGET_CARBON
+/*
+    only working under classic carbon
+    m_macControlAction = *(**(ControlHandle)m_macControl).contrlDefProc ;
+    (**(ControlHandle)m_macControl).contrlDefProc = (Handle) &wxMacControlActionUPP ;
+*/
+#else
+    m_macControlAction = *(**(ControlHandle)m_macControl).contrlDefProc ;
+
+    cdefHandle cdef ;
+    cdef = (cdefHandle) NewHandle( sizeof(cdefRec) ) ;
+    if (  (**(ControlHandle)m_macControl).contrlDefProc != NULL )
+    {
+      (**cdef).instruction = 0x4EF9;  /* JMP instruction */
+      (**cdef).function = (void(*)()) wxMacControlActionUPP;
+      (**(ControlHandle)m_macControl).contrlDefProc = (Handle) cdef ;
+    }
+#endif
+	SetControlColorProc( (ControlHandle) m_macControl , wxMacSetupControlBackgroundUPP ) ;
+ 
+     // Adjust the controls size and position
+     wxPoint pos(m_x, m_y);
+     wxSize best_size( DoGetBestSize() );
+     wxSize new_size( m_width, m_height );
+ 
+     m_x = m_y = m_width = m_height = -1;  // Forces SetSize to move/size the control
+ 
+     if (new_size.x == -1) {
+        new_size.x = best_size.x;
+     }
+     if (new_size.y == -1) {
+         new_size.y = best_size.y;
+     }
+ 
+     SetSize(pos.x, pos.y, new_size.x, new_size.y);
+ 
+     UMAShowControl( (ControlHandle) m_macControl ) ;
+     Refresh() ;
 }
 
 void wxControl::MacAdjustControlRect() 
 {
-	wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
-	if ( m_width == -1 || m_height == -1 )
-	{
-		Rect 	bestsize = { 0 , 0 , 0 , 0 } ;
-		short	baselineoffset ;
-		
-		UMAGetBestControlRect( m_macControl , &bestsize , &baselineoffset ) ;
-
-		if ( EmptyRect( &bestsize ) )
-		{
-			baselineoffset = 0;
-			bestsize.left = bestsize.top = 0 ;
-			bestsize.right = 16 ;
-			bestsize.bottom = 16 ;
-			if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
-			{
-				bestsize.bottom = 16 ;
-			}
-			else if ( IsKindOf( CLASSINFO( wxSpinButton ) ) )
-			{
-				bestsize.bottom = 24 ; 
-			}
-		}
-
-		if ( m_width == -1 )
-		{
-			if ( IsKindOf( CLASSINFO( wxButton ) ) )
-			{
-				m_width = m_label.Length() * 8 + 12 ;
-			}
-			else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
-			{
-				m_width = m_label.Length() * 8 ;
-			}
-			else
-				m_width = bestsize.right - bestsize.left ;
-			
-			m_width += 2 * m_macHorizontalBorder ;
-		}
-		if ( m_height == -1 )
-		{
-			m_height = bestsize.bottom - bestsize.top ;
-			if ( m_height < 10 )
-				m_height = 13 ;
-
-			m_height += 2 * m_macVerticalBorder;
-		}
-		
-		wxMacDrawingHelper helper ( wxFindWinFromMacWindow( GetMacRootWindow() ) ) ;
-		if ( helper.Ok() )
-		{
-	    	UMASizeControl( m_macControl , m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
-  		}
-	}
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , wxT("No valid mac control") ) ;
+    if ( m_width == -1 || m_height == -1 )
+    {
+        Rect    bestsize = { 0 , 0 , 0 , 0 } ;
+        short   baselineoffset ;
+        
+        ::GetBestControlRect( (ControlHandle) m_macControl , &bestsize , &baselineoffset ) ;
+
+        if ( EmptyRect( &bestsize ) )
+        {
+            baselineoffset = 0;
+            bestsize.left = bestsize.top = 0 ;
+            bestsize.right = 16 ;
+            bestsize.bottom = 16 ;
+            if ( IsKindOf( CLASSINFO( wxScrollBar ) ) )
+            {
+                bestsize.bottom = 16 ;
+            }
+            else if ( IsKindOf( CLASSINFO( wxSpinButton ) ) )
+            {
+                bestsize.bottom = 24 ; 
+            }
+        }
+
+        if ( m_width == -1 )
+        {
+            if ( IsKindOf( CLASSINFO( wxButton ) ) )
+            {
+                m_width = m_label.Length() * 8 + 12 ;
+                if ( m_width < 70 )
+                  m_width = 70 ;
+            }
+            else if ( IsKindOf( CLASSINFO( wxStaticText ) ) )
+            {
+                m_width = m_label.Length() * 8 ;
+            }
+            else
+                m_width = bestsize.right - bestsize.left ;
+            
+            m_width += 2 * m_macHorizontalBorder + MacGetLeftBorderSize() + MacGetRightBorderSize() ;
+        }
+        if ( m_height == -1 )
+        {
+            m_height = bestsize.bottom - bestsize.top ;
+            if ( m_height < 10 )
+                m_height = 13 ;
+
+            m_height += 2 * m_macVerticalBorder + MacGetTopBorderSize() + MacGetBottomBorderSize() ;
+        }
+  		MacUpdateDimensions() ;      
+//        UMASizeControl( (ControlHandle) m_macControl , m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
+    }
 }
-ControlHandle wxControl::MacGetContainerForEmbedding() 
+
+WXWidget wxControl::MacGetContainerForEmbedding() 
 {
-	if ( m_macControl )
-		return m_macControl ;
+    if ( m_macControl )
+        return m_macControl ;
 
-	return wxWindow::MacGetContainerForEmbedding() ;
+    return wxWindow::MacGetContainerForEmbedding() ;
 }
 
-void wxControl::MacSuperChangedPosition() 
+void wxControl::MacUpdateDimensions() 
 {
-	if ( m_macControl )
-	{
-		Rect contrlRect ;		
-		GetControlBounds( m_macControl , &contrlRect ) ; 
-		int former_mac_x = contrlRect.left ;
-		int former_mac_y = contrlRect.top ;
-		int mac_x = m_x ;
-		int mac_y = m_y ;
-		GetParent()->MacClientToRootWindow( & mac_x , & mac_y ) ;
-		
-		WindowRef rootwindow = GetMacRootWindow() ;
-		wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
-		UMASetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
-		wxMacDrawingHelper focus( wxrootwindow ) ;
+	// actually in the current systems this should never be possible, but later reparenting
+	// may become a reality
 	
-		if ( mac_x != former_mac_x || mac_y != former_mac_y )
-		{
-			{
-				Rect inval = { former_mac_y , former_mac_x , former_mac_y + m_height , former_mac_x + m_width } ;
-				InvalWindowRect( rootwindow , &inval ) ;
-			}
-	  		UMAMoveControl( m_macControl , mac_x + m_macHorizontalBorder , mac_y + m_macVerticalBorder ) ;
-			{
-				Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-				InvalWindowRect( rootwindow , &inval ) ;
-			}
-		}
-		if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
+	if ( (ControlHandle) m_macControl == NULL )
+		return ;
+		
+	if ( GetParent() == NULL )
+		return ;
+		
+    WindowRef rootwindow = (WindowRef) MacGetRootWindow() ;
+    if ( rootwindow == NULL )
+    	return ;
+    	
+    Rect oldBounds ;       
+    GetControlBounds( (ControlHandle) m_macControl , &oldBounds ) ; 
+    
+    int new_x = m_x + MacGetLeftBorderSize() + m_macHorizontalBorder ;
+    int new_y = m_y + MacGetTopBorderSize() + m_macVerticalBorder ;
+    int new_width = m_width - MacGetLeftBorderSize() - MacGetRightBorderSize() - 2 * m_macHorizontalBorder ;
+    int new_height = m_height - MacGetTopBorderSize() - MacGetBottomBorderSize() - 2 * m_macVerticalBorder ;
+    
+    GetParent()->MacWindowToRootWindow( & new_x , & new_y ) ;
+    bool doMove = new_x != oldBounds.left || new_y != oldBounds.top ;
+    bool doResize =  ( oldBounds.right - oldBounds.left ) != new_width || (oldBounds.bottom - oldBounds.top ) != new_height ;
+	if ( doMove || doResize )
+	{
+		InvalWindowRect( rootwindow, &oldBounds ) ;
+		if ( doMove )
 		{
+			UMAMoveControl( (ControlHandle) m_macControl , new_x , new_y ) ;
 		}
-		else
+		if ( doResize )
 		{
-			UMASetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
+			UMASizeControl( (ControlHandle) m_macControl , new_width , new_height ) ;
 		}
 	}
+}
 
-	wxWindow::MacSuperChangedPosition() ;
+void wxControl::MacSuperChangedPosition() 
+{
+ 	MacUpdateDimensions() ;
+    wxWindow::MacSuperChangedPosition() ;
 }
 
 void wxControl::MacSuperEnabled( bool enabled ) 
 {
-/*
-	if ( m_macControl )
-	{
-		if ( UMAHasAppearance() )
-		{
-			if ( !enabled )
-			{
-				::DeactivateControl( m_macControl ) ; 
-			}
-			else
-			{
-				if ( m_macEnabled )
-					::ActivateControl( m_macControl ) ;
-			}
-		}
-		else
-		{
-			if ( !enabled )
-			{
-				::HiliteControl( m_macControl , 255 ) ;
-			}
-			else
-			{
-				if ( m_macEnabled )
-					::HiliteControl( m_macControl , 0 ) ;
-			}
-		}
-	}
-	wxWindow::MacSuperEnabled( enabled ) ;
-*/
+    Refresh(FALSE) ;
+    wxWindow::MacSuperEnabled( enabled ) ;
 }
 
 void  wxControl::MacSuperShown( bool show ) 
 {
-	if ( m_macControl )
-	{
-		if ( !show )
-		{
-			if ( m_macControlIsShown )
-			{
-				::UMAHideControl( m_macControl ) ;
-				m_macControlIsShown = false ;
-			}
-		}
-		else
-		{
-			if ( MacIsReallyShown() && !m_macControlIsShown )
-			{
-				::UMAShowControl( m_macControl ) ;
-				m_macControlIsShown = true ;
-			}
-		}
-	}
-		
-	wxWindow::MacSuperShown( show ) ;
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( !show )
+        {
+            if ( m_macControlIsShown )
+            {
+                ::UMAHideControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = false ;
+            }
+        }
+        else
+        {
+            if ( MacIsReallyShown() && !m_macControlIsShown )
+            {
+                ::UMAShowControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = true ;
+            }
+        }
+    }
+        
+    wxWindow::MacSuperShown( show ) ;
 }
 
 void  wxControl::DoSetSize(int x, int y,
             int width, int height,
             int sizeFlags )
 {
-	if ( m_macControl == NULL )
-	{
-		wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
-		return ;
-	}
-
-	WindowRef rootwindow = GetMacRootWindow() ;
-	wxWindow* wxrootwindow = wxFindWinFromMacWindow( rootwindow ) ;
-	UMASetThemeWindowBackground( rootwindow , kThemeBrushDialogBackgroundActive , false ) ;
-		
-	int former_x = m_x ;
-	int former_y = m_y ;
-	int former_w = m_width ;
-	int former_h = m_height ;
-	
-	Rect contrlRect ;		
-	GetControlBounds( m_macControl , &contrlRect ) ; 
-	int former_mac_x = contrlRect.left ;
-	int former_mac_y = contrlRect.top ;
-	
-  int currentX, currentY;
-  GetPosition(&currentX, &currentY);
-  int currentW,currentH;
-  GetSize(&currentW, &currentH);
-
-  int actualWidth = width;
-  int actualHeight = height;
-  int actualX = x;
-  int actualY = y;
-  if (x == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
-      actualX = currentX;
-  if (y == -1 && !(sizeFlags & wxSIZE_ALLOW_MINUS_ONE))
-      actualY = currentY;
-  if (width == -1)
-      actualWidth = currentW ;
-  if (height == -1)
-      actualHeight = currentH ;
-
-	if ( actualX == currentX && actualY == currentY && actualWidth == currentW && actualHeight == currentH)
-		return ;
-		
-	AdjustForParentClientOrigin(actualX, actualY, sizeFlags);
-	WindowRef macrootwindow = GetMacRootWindow() ;
-	wxMacDrawingHelper focus( wxFindWinFromMacWindow( macrootwindow ) ) ;
-
-	int mac_x = actualX ;
-	int mac_y = actualY ;
-	GetParent()->MacClientToRootWindow( & mac_x , & mac_y ) ;
-	
-	if ( mac_x != former_mac_x || mac_y != former_mac_y )
-	{
-		{
-			Rect inval = { former_mac_y , former_mac_x , former_mac_y + m_height , former_mac_x + m_width } ;
-			InvalWindowRect( macrootwindow, &inval ) ;
-		}
-  		UMAMoveControl( m_macControl , mac_x + m_macHorizontalBorder , mac_y  + m_macVerticalBorder ) ;
-		{
-			Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-			InvalWindowRect(macrootwindow, &inval ) ;
-		}
-	}
-
-	if ( actualX != former_x || actualY != former_y )
-	{
-		m_x = actualX ;
-		m_y = actualY ;
-
-		MacRepositionScrollBars() ;
-		// To consider -> should the parameters be the effective or the virtual coordinates (AdjustForParent..)
-    wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
-    event.SetEventObject(this);
-    GetEventHandler()->ProcessEvent(event);
-	}		
-	if ( actualWidth != former_w || actualHeight != former_h )
-	{
-		{
-			Rect inval = { mac_y , mac_x , mac_y + former_h , mac_x + former_w } ;
-			InvalWindowRect( macrootwindow, &inval ) ;
-		}
-		m_width = actualWidth ;
-		m_height = actualHeight ;
+    wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
+    return ;
+/*
 
-   		UMASizeControl( m_macControl ,  m_width - 2 * m_macHorizontalBorder, m_height -  2 * m_macVerticalBorder ) ;
-		{
-			Rect inval = { mac_y , mac_x , mac_y + m_height , mac_x + m_width } ;
-			InvalWindowRect( macrootwindow , &inval ) ;
-		}
+    if ( (ControlHandle) m_macControl == NULL )
+    {
+        wxWindow::DoSetSize( x , y ,width , height ,sizeFlags ) ;
+        return ;
+    }
 
-		MacRepositionScrollBars() ;
-    wxSizeEvent event(wxSize(m_width, m_height), m_windowId);
-    event.SetEventObject(this);
-    GetEventHandler()->ProcessEvent(event);
-	}
-	if ( wxrootwindow->IsKindOf( CLASSINFO( wxDialog ) ) )
-	{
-	}
-	else
-	{
-		UMASetThemeWindowBackground( rootwindow , kThemeBrushDocumentWindowBackground , false ) ;
-	}
+     Rect oldbounds;
+     int new_x, new_y, new_width, new_height;
+     int mac_x, mac_y;
+ 
+     new_x = m_x;
+     new_y = m_y;
+     new_width  = m_width;
+     new_height = m_height;
+ 
+     if (sizeFlags & wxSIZE_ALLOW_MINUS_ONE)
+     {
+         new_x      = x;
+         new_y      = y;
+         new_width  = width;
+         new_height = height;
+     }
+     else
+     {
+         if (x != -1)      new_x      = x;
+         if (y != -1)      new_y      = y;
+         if (width != -1)  new_width  = width;
+         if (height != -1) new_height = height;
+     }
+ 
+     if(sizeFlags & wxSIZE_AUTO)
+     {
+         wxSize size = GetBestSize();
+         if (sizeFlags & wxSIZE_AUTO_WIDTH)
+         {
+             if (width == -1)   new_width = size.x;
+         }
+         if (sizeFlags & wxSIZE_AUTO_HEIGHT)
+         {
+             if (height == -1)   new_height = size.y;
+         }
+     }
+     AdjustForParentClientOrigin(new_x, new_y, sizeFlags);
+ 
+     mac_x = new_x;
+     mac_y = new_y;
+     if(GetParent()) {
+         GetParent()->MacWindowToRootWindow(&mac_x, &mac_y);
+     }
+     GetControlBounds( (ControlHandle) m_macControl, &oldbounds);
+     oldbounds.right = oldbounds.left + m_width;
+     oldbounds.bottom = oldbounds.top + m_height;
+ 
+     bool doMove = false;
+     bool doResize = false;
+ 
+     if ( mac_x != (oldbounds.left - m_macHorizontalBorder) ||
+          mac_y != (oldbounds.top - m_macVerticalBorder) )
+     {
+         doMove = true ;
+     }
+     if ( new_width != oldbounds.right - oldbounds.left - 2 * m_macHorizontalBorder ||
+          new_height != oldbounds.bottom - oldbounds.top - 2 * m_macVerticalBorder)
+     {
+         doResize = true ;
+     }
+ 
+     if ( doMove || doResize )
+     {
+         Refresh() ;
+     
+         // Ensure resize is within constraints
+         if ((m_minWidth != -1) && (new_width < m_minWidth)) {
+             new_width = m_minWidth;
+         }
+         if ((m_minHeight != -1) && (new_height < m_minHeight)) {
+             new_height = m_minHeight;
+         }
+         if ((m_maxWidth != -1) && (new_width > m_maxWidth)) {
+             new_width = m_maxWidth;
+         }
+         if ((m_maxHeight != -1) && (new_height > m_maxHeight)) {
+             new_height = m_maxHeight;
+         }
+ 
+         if ( doMove )
+         {
+             m_x = new_x;
+             m_y = new_y;
+ 
+             UMAMoveControl( (ControlHandle) m_macControl,
+                            mac_x + m_macHorizontalBorder, mac_y + m_macVerticalBorder);
+
+             wxMoveEvent event(wxPoint(m_x, m_y), m_windowId);
+             event.SetEventObject(this);
+             GetEventHandler()->ProcessEvent(event) ;
+         }
+         if ( doResize )
+         {
+             m_width = new_width;
+             m_height = new_height;
+ 
+             UMASizeControl( (ControlHandle) m_macControl,
+                             m_width - 2 * m_macHorizontalBorder,
+                             m_height -  2 * m_macVerticalBorder ) ;
+ 
+ 
+             wxSizeEvent event(wxSize(m_width, m_height), m_windowId);
+             event.SetEventObject(this);
+             GetEventHandler()->ProcessEvent(event);
+         }
+ 
+          Refresh() ;
+     }
+*/
 }
 
 bool  wxControl::Show(bool show) 
 {
-	if ( !wxWindow::Show( show ) )
-		return FALSE ;
-		
-	if ( m_macControl )
-	{
-		if ( !show )
-		{
-			if ( m_macControlIsShown )
-			{
-				::UMAHideControl( m_macControl ) ;
-				m_macControlIsShown = false ;
-			}
-		}
-		else
-		{
-			if ( MacIsReallyShown() && !m_macControlIsShown )
-			{
-				::UMAShowControl( m_macControl ) ;
-				m_macControlIsShown = true ;
-			}
-		}
-	}
-	return TRUE ;
+    if ( !wxWindow::Show( show ) )
+        return FALSE ;
+        
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( !show )
+        {
+            if ( m_macControlIsShown )
+            {
+                ::UMAHideControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = false ;
+            }
+        }
+        else
+        {
+            if ( MacIsReallyShown() && !m_macControlIsShown )
+            {
+                ::UMAShowControl( (ControlHandle) m_macControl ) ;
+                m_macControlIsShown = true ;
+            }
+        }
+    }
+    return TRUE ;
 }
 
 bool  wxControl::Enable(bool enable) 
@@ -574,227 +738,151 @@ bool  wxControl::Enable(bool enable)
     if ( !wxWindow::Enable(enable) )
         return FALSE;
 
-	if ( m_macControl )
-	{
-		
-		if ( UMAHasAppearance() )
-		{
-			if ( enable )
-				::ActivateControl( m_macControl ) ;
-			else
-				::DeactivateControl( m_macControl ) ;
-		}
-		else
-		{
-			if ( enable )
-				::HiliteControl( m_macControl , 0 ) ;
-			else
-				::HiliteControl( m_macControl , 255 ) ;
-		}
-	}
-	return TRUE ;
+    if ( (ControlHandle) m_macControl )
+    {
+        if ( enable )
+            UMAActivateControl( (ControlHandle) m_macControl ) ;
+        else
+            UMADeactivateControl( (ControlHandle) m_macControl ) ;
+    }
+    return TRUE ;
 }
 
 void wxControl::Refresh(bool eraseBack, const wxRect *rect)
 {
-  if ( m_macControl )
-  {
-  	wxWindow::Refresh( eraseBack , rect ) ;
-	}
-  else
-  {
-  	wxWindow::Refresh( eraseBack , rect ) ;
-  }
+    wxWindow::Refresh( eraseBack , rect ) ;
 }
 
 void wxControl::MacRedrawControl()
 {
-	if ( m_macControl )
-	{
-		WindowRef window = GetMacRootWindow() ;
-		if ( window )
-		{
-			wxWindow* win = wxFindWinFromMacWindow( window ) ;
-			if ( win )
-			{
-				wxMacDrawingHelper help( win ) ;
-				// the mac control manager always assumes to have the origin at 0,0
-				SetOrigin( 0 , 0 ) ;
-				
-				bool			hasTabBehind = false ;
-				wxWindow* parent = GetParent() ;
-				while ( parent )
-				{
-					if( parent->MacGetWindowData() )
-					{
-						UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
-						break ;
-					}
-					
-					if( parent->IsKindOf( CLASSINFO( wxNotebook ) ) ||  parent->IsKindOf( CLASSINFO( wxTabCtrl ) ))
-					{
-						if ( ((wxControl*)parent)->m_macControl )
-							SetUpControlBackground( ((wxControl*)parent)->m_macControl , -1 , true ) ;
-						break ;
-					}
-					
-					parent = parent->GetParent() ;
-				} 
-				
-				UMADrawControl( m_macControl ) ;
-				UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-				wxDC::MacInvalidateSetup() ;
-			}
-		}
-	}
+    if ( (ControlHandle) m_macControl && MacGetRootWindow() )
+    {
+        wxClientDC dc(this) ;
+        wxMacPortSetter helper(&dc) ;
+        wxMacWindowClipper clipper(this) ;
+        wxDC::MacSetupBackgroundForCurrentPort( MacGetBackgroundBrush() ) ;
+        UMADrawControl( (ControlHandle) m_macControl ) ;
+    }
 }
 
 void wxControl::OnPaint(wxPaintEvent& event)
 {
-	if ( m_macControl )
-	{
-		WindowRef window = GetMacRootWindow() ;
-		if ( window )
-		{
-			wxWindow* win = wxFindWinFromMacWindow( window ) ;
-			if ( win )
-			{
-				wxMacDrawingHelper help( win ) ;
-				// the mac control manager always assumes to have the origin at 0,0
-				SetOrigin( 0 , 0 ) ;
-				
-				bool			hasTabBehind = false ;
-				wxWindow* parent = GetParent() ;
-				while ( parent )
-				{
-					if( parent->MacGetWindowData() )
-					{
-						UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , kThemeBrushDialogBackgroundActive , false ) ;
-						break ;
-					}
-					
-					if( parent->IsKindOf( CLASSINFO( wxNotebook ) ) ||  parent->IsKindOf( CLASSINFO( wxTabCtrl ) ))
-					{
-						if ( ((wxControl*)parent)->m_macControl )
-							SetUpControlBackground( ((wxControl*)parent)->m_macControl , -1 , true ) ;
-						break ;
-					}
-					
-					parent = parent->GetParent() ;
-				} 
-				
-				UMADrawControl( m_macControl ) ;
-				UMASetThemeWindowBackground( win->MacGetWindowData()->m_macWindow , win->MacGetWindowData()->m_macWindowBackgroundTheme , false ) ;
-				wxDC::MacInvalidateSetup() ;
-			}
-		}
-	}
-	else
-	{
-		// wxWindow::OnPaint( event ) ;
-	}
+    if ( (ControlHandle) m_macControl )
+    {
+        wxPaintDC dc(this) ;
+        wxMacPortSetter helper(&dc) ;
+        wxMacWindowClipper clipper(this) ;
+        wxDC::MacSetupBackgroundForCurrentPort( MacGetBackgroundBrush() ) ;
+        UMADrawControl( (ControlHandle) m_macControl ) ;
+    }
+    else
+    {
+      event.Skip() ;
+    }
 }
 void wxControl::OnEraseBackground(wxEraseEvent& event)
 {
-    // In general, you don't want to erase the background of a control,
-    // or you'll get a flicker.
-    // TODO: move this 'null' function into each control that
-    // might flicker.
+    wxWindow::OnEraseBackground( event ) ; 
 }
 
 
 void  wxControl::OnKeyDown( wxKeyEvent &event ) 
 {
-	if ( m_macControl == NULL )
-		return ;
-	
-	EventRecord *ev = wxTheApp->MacGetCurrentEvent() ;
-	short keycode ;
-	short keychar ;
-	keychar = short(ev->message & charCodeMask);
-	keycode = short(ev->message & keyCodeMask) >> 8 ;
-
-	UMAHandleControlKey( m_macControl , keycode , keychar , ev->modifiers ) ;
+    if ( (ControlHandle) m_macControl == NULL )
+        return ;
+    
+    EventRecord *ev = (EventRecord*) wxTheApp->MacGetCurrentEvent() ;
+    short keycode ;
+    short keychar ;
+    keychar = short(ev->message & charCodeMask);
+    keycode = short(ev->message & keyCodeMask) >> 8 ;
+
+    ::HandleControlKey( (ControlHandle) m_macControl , keycode , keychar , ev->modifiers ) ;
 }
 
 void  wxControl::OnMouseEvent( wxMouseEvent &event ) 
 {
-	if ( m_macControl == NULL )
-	{
-		event.Skip() ;
-		return ;
-	}
-		
-	if (event.GetEventType() == wxEVT_LEFT_DOWN || event.GetEventType() == wxEVT_LEFT_DCLICK )
-	{
-			
-		int x = event.m_x ;
-		int y = event.m_y ;
-		
-		MacClientToRootWindow( &x , &y ) ;
-			
-		ControlHandle	control ;
-		Point		localwhere ;
-		GrafPtr		port ;
-		SInt16		controlpart ;
-		WindowRef	window = GetMacRootWindow() ;
-		
-		localwhere.h = x ;
-		localwhere.v = y ;
-	
-		short modifiers = 0;
-		
-		if ( !event.m_leftDown && !event.m_rightDown )
-			modifiers  |= btnState ;
-	
-		if ( event.m_shiftDown )
-			modifiers |= shiftKey ;
-			
-		if ( event.m_controlDown )
-			modifiers |= controlKey ;
-	
-		if ( event.m_altDown )
-			modifiers |= optionKey ;
-	
-		if ( event.m_metaDown )
-			modifiers |= cmdKey ;
-	
-		controlpart = FindControl( localwhere , window , &control ) ;
-		{
-			if ( AcceptsFocus() && FindFocus() != this )
-			{
-				SetFocus() ;
-			}
-			if ( control && UMAIsControlActive( control ) )
-			{
-				{
-					if ( controlpart == kControlIndicatorPart && !UMAHasAppearance() )
-						controlpart = UMAHandleControlClick( control , localwhere , modifiers , (ControlActionUPP) NULL ) ;
-					else
-						controlpart = UMAHandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
-					wxTheApp->s_lastMouseDown = 0 ;
-					if ( controlpart && ! ( ( UMAHasAppearance() || (controlpart != kControlIndicatorPart) ) 
-						&& (IsKindOf( CLASSINFO( wxScrollBar ) ) ) ) ) // otherwise we will get the event twice
-					{
-						MacHandleControlClick( control , controlpart ) ;
-					}
-				}
-			}
-		}
-	}
+    if ( (ControlHandle) m_macControl == NULL )
+    {
+        event.Skip() ;
+        return ;
+    }
+        
+    if (event.GetEventType() == wxEVT_LEFT_DOWN || event.GetEventType() == wxEVT_LEFT_DCLICK )
+    {
+            
+        int x = event.m_x ;
+        int y = event.m_y ;
+        
+        MacClientToRootWindow( &x , &y ) ;
+            
+        ControlHandle   control ;
+        Point       localwhere ;
+        SInt16      controlpart ;
+        WindowRef   window = (WindowRef) MacGetRootWindow() ;
+        
+        localwhere.h = x ;
+        localwhere.v = y ;
+    
+        short modifiers = 0;
+        
+        if ( !event.m_leftDown && !event.m_rightDown )
+            modifiers  |= btnState ;
+    
+        if ( event.m_shiftDown )
+            modifiers |= shiftKey ;
+            
+        if ( event.m_controlDown )
+            modifiers |= controlKey ;
+    
+        if ( event.m_altDown )
+            modifiers |= optionKey ;
+    
+        if ( event.m_metaDown )
+            modifiers |= cmdKey ;
+/*
+#if TARGET_CARBON
+        control = FindControlUnderMouse( localwhere , window , &controlpart ) ;
+#else
+        controlpart = FindControl( localwhere , window , &control ) ;
+#endif
+*/
+        {
+        /*
+            if ( AcceptsFocus() && FindFocus() != this )
+            {
+                SetFocus() ;
+            }
+        */
+            control = (ControlHandle) m_macControl ;
+            if ( control && ::IsControlActive( control ) )
+            {
+                {
+                    controlpart = ::HandleControlClick( control , localwhere , modifiers , (ControlActionUPP) -1 ) ;
+                    wxTheApp->s_lastMouseDown = 0 ;
+                    if ( control && controlpart != kControlNoPart && 
+                        ! IsKindOf( CLASSINFO( wxScrollBar ) ) 
+                    ) // otherwise we will get the event twice for scrollbar
+                    {
+                        MacHandleControlClick( control , controlpart ) ;
+                    }
+                }
+            }
+        }
+    }
 }
 
 bool wxControl::MacCanFocus() const
 {
-		{ if ( m_macControl == NULL ) 
-				return true ; 
-			else
-			return false ; 
-		}
+        { if ( (ControlHandle) m_macControl == NULL ) 
+                return true ; 
+            else
+            return false ; 
+        }
 }
 
-void wxControl::MacHandleControlClick( ControlHandle control , SInt16 controlpart ) 
+void wxControl::MacHandleControlClick( WXWidget control , wxInt16 controlpart ) 
 {
-	wxASSERT_MSG( m_macControl != NULL , "No valid mac control" ) ;
+    wxASSERT_MSG( (ControlHandle) m_macControl != NULL , "No valid mac control" ) ;
 }