check that the version of __sync_sub_and_fetch that returns a value is supported...

[wxWidgets.git] / src / mac / carbon / scrolbar.cpp

diff --git a/src/mac/carbon/scrolbar.cpp b/src/mac/carbon/scrolbar.cpp
index 2be39020bdd1038d0d3d7d9aa2780b4b48be2f01..c7c1263451387280267f40ea5cfe4fdc9071b4c7 100644 (file)
--- a/src/mac/carbon/scrolbar.cpp
+++ b/src/mac/carbon/scrolbar.cpp
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        scrolbar.cpp
+// Name:        src/mac/carbon/scrolbar.cpp

 // Purpose:     wxScrollBar
 // Author:      Stefan Csomor
 // Modified by:
 // Purpose:     wxScrollBar
 // Author:      Stefan Csomor
 // Modified by:


@@ -11,12 +11,14 @@
 
 #include "wx/wxprec.h"
 
 
 #include "wx/wxprec.h"
 

+#include "wx/scrolbar.h"
+

 #ifndef WX_PRECOMP
     #include "wx/intl.h"
     #include "wx/log.h"
 #ifndef WX_PRECOMP
     #include "wx/intl.h"
     #include "wx/log.h"

-#endif // WX_PRECOMP
+    #include "wx/settings.h"
+#endif

 
 

-#include "wx/scrolbar.h"

 #include "wx/mac/uma.h"
 
 IMPLEMENT_DYNAMIC_CLASS(wxScrollBar, wxControl)
 #include "wx/mac/uma.h"
 
 IMPLEMENT_DYNAMIC_CLASS(wxScrollBar, wxControl)


@@ -25,26 +27,30 @@ BEGIN_EVENT_TABLE(wxScrollBar, wxControl)
 END_EVENT_TABLE()
 
 
 END_EVENT_TABLE()
 
 

-bool wxScrollBar::Create(wxWindow *parent,
+bool wxScrollBar::Create( wxWindow *parent,

     wxWindowID id,
     const wxPoint& pos,
     const wxSize& size,
     long style,
     const wxValidator& validator,
     wxWindowID id,
     const wxPoint& pos,
     const wxSize& size,
     long style,
     const wxValidator& validator,

-    const wxString& name)
+    const wxString& name )

 {
 {

-    m_macIsUserPane = false ;
+    m_macIsUserPane = false;

 
 

-    if ( !wxControl::Create(parent, id, pos, size, style, validator, name) )
+    if ( !wxControl::Create( parent, id, pos, size, style, validator, name ) )

         return false;
 
         return false;
 

-    Rect bounds = wxMacGetBoundsForControl( this , pos , size ) ;
+    Rect bounds = wxMacGetBoundsForControl( this, pos, size );

 
 

-    m_peer = new wxMacControl(this) ;
-    verify_noerr( CreateScrollBarControl( MAC_WXHWND(parent->MacGetTopLevelWindowRef()) , &bounds ,
-    0 , 0 , 100 , 1 , true /* liveTracking */ , GetwxMacLiveScrollbarActionProc() , m_peer->GetControlRefAddr() ) );
+    m_peer = new wxMacControl( this );
+    OSStatus err = CreateScrollBarControl(
+        MAC_WXHWND(parent->MacGetTopLevelWindowRef()), &bounds,
+        0, 0, 100, 1, true /* liveTracking */,
+        GetwxMacLiveScrollbarActionProc(),
+        m_peer->GetControlRefAddr() );
+    verify_noerr( err );

 
 

-    MacPostControlCreate( pos, size ) ;
+    MacPostControlCreate( pos, size );

 
     return true;
 }
 
     return true;
 }


@@ -53,82 +59,86 @@ wxScrollBar::~wxScrollBar()
 {
 }
 
 {
 }
 

-void wxScrollBar::SetThumbPosition(int viewStart)
+void wxScrollBar::SetThumbPosition( int viewStart )

 {
 {

-    m_peer->SetValue( viewStart ) ;
+    m_peer->SetValue( viewStart );

 }
 
 int wxScrollBar::GetThumbPosition() const
 {
 }
 
 int wxScrollBar::GetThumbPosition() const
 {

-    return m_peer->GetValue() ;
+    return m_peer->GetValue();

 }
 
 }
 

-void wxScrollBar::SetScrollbar(int position, int thumbSize, int range, int pageSize, bool refresh)
+void wxScrollBar::SetScrollbar( int position,
+                                int thumbSize,
+                                int range,
+                                int pageSize,
+                                bool WXUNUSED(refresh) )

 {
     m_pageSize = pageSize;
     m_viewSize = thumbSize;
     m_objectSize = range;
 
 {
     m_pageSize = pageSize;
     m_viewSize = thumbSize;
     m_objectSize = range;
 

-   int range1 = wxMax((m_objectSize - m_viewSize), 0) ;
+   int range1 = wxMax( (m_objectSize - m_viewSize), 0 );

 
 

-    m_peer->SetMaximum( range1 ) ;
-    m_peer->SetMinimum( 0 ) ;
-    m_peer->SetValue( position ) ;
-    m_peer->SetViewSize( m_viewSize ) ;
+    m_peer->SetMinimum( 0 );
+    m_peer->SetMaximum( range1 );
+    m_peer->SetValue( position );
+    m_peer->SetViewSize( m_viewSize );

 }
 
 }
 

-void wxScrollBar::Command(wxCommandEvent& event)
+void wxScrollBar::Command( wxCommandEvent& event )

 {
 {

-    SetThumbPosition(event.GetInt());
-    ProcessCommand(event);
+    SetThumbPosition( event.GetInt() );
+    ProcessCommand( event );

 }
 
 }
 

-void wxScrollBar::MacHandleControlClick( WXWidget control , wxInt16 controlpart , bool mouseStillDown )
+void wxScrollBar::MacHandleControlClick( WXWidget WXUNUSED(control), wxInt16 controlpart, bool mouseStillDown )

 {
 {

-    int position = m_peer->GetValue() ;
-    int minPos = m_peer->GetMinimum() ;
-    int maxPos = m_peer->GetMaximum() ;
+    int position = m_peer->GetValue();
+    int minPos = m_peer->GetMinimum();
+    int maxPos = m_peer->GetMaximum();

 
     wxEventType scrollEvent = wxEVT_NULL;
     int nScrollInc = 0;
 
     // all events have already been reported during mouse down, except for THUMBRELEASE
     if ( !mouseStillDown && controlpart != kControlIndicatorPart )
 
     wxEventType scrollEvent = wxEVT_NULL;
     int nScrollInc = 0;
 
     // all events have already been reported during mouse down, except for THUMBRELEASE
     if ( !mouseStillDown && controlpart != kControlIndicatorPart )

-        return ;
+        return;

 
     switch ( controlpart )
     {
 
     switch ( controlpart )
     {

-    case kControlUpButtonPart :
+    case kControlUpButtonPart:

         nScrollInc = -1;
         scrollEvent = wxEVT_SCROLL_LINEUP;
         nScrollInc = -1;
         scrollEvent = wxEVT_SCROLL_LINEUP;

-        break ;
+        break;

 
 

-    case kControlDownButtonPart :
+    case kControlDownButtonPart:

         nScrollInc = 1;
         scrollEvent = wxEVT_SCROLL_LINEDOWN;
         nScrollInc = 1;
         scrollEvent = wxEVT_SCROLL_LINEDOWN;

-        break ;
+        break;

 
 

-    case kControlPageUpPart :
+    case kControlPageUpPart:

         nScrollInc = -m_pageSize;
         scrollEvent = wxEVT_SCROLL_PAGEUP;
         nScrollInc = -m_pageSize;
         scrollEvent = wxEVT_SCROLL_PAGEUP;

-        break ;
+        break;

 
 

-    case kControlPageDownPart :
+    case kControlPageDownPart:

         nScrollInc = m_pageSize;
         scrollEvent = wxEVT_SCROLL_PAGEDOWN;
         nScrollInc = m_pageSize;
         scrollEvent = wxEVT_SCROLL_PAGEDOWN;

-        break ;
+        break;

 
 

-    case kControlIndicatorPart :
-        nScrollInc = 0 ;
+    case kControlIndicatorPart:
+        nScrollInc = 0;

         if ( mouseStillDown )
             scrollEvent = wxEVT_SCROLL_THUMBTRACK;
         else
             scrollEvent = wxEVT_SCROLL_THUMBRELEASE;
         if ( mouseStillDown )
             scrollEvent = wxEVT_SCROLL_THUMBTRACK;
         else
             scrollEvent = wxEVT_SCROLL_THUMBRELEASE;

-        break ;
+        break;

 
 

-    default :
-        wxFAIL_MSG(wxT("illegal scrollbar selector"));
-        break ;
+    default:
+        wxFAIL_MSG(wxT("unknown scrollbar selector"));
+        break;

     }
 
     int new_pos = position + nScrollInc;
     }
 
     int new_pos = position + nScrollInc;


@@ -139,51 +149,53 @@ void wxScrollBar::MacHandleControlClick( WXWidget control , wxInt16 controlpart
         new_pos = maxPos;
 
     if ( nScrollInc )
         new_pos = maxPos;
 
     if ( nScrollInc )

-        SetThumbPosition(new_pos);
+        SetThumbPosition( new_pos );

 
 

-    wxScrollEvent event(scrollEvent, m_windowId);
+    wxScrollEvent event( scrollEvent, m_windowId );

     if ( m_windowStyle & wxHORIZONTAL )
     if ( m_windowStyle & wxHORIZONTAL )

-        event.SetOrientation( wxHORIZONTAL ) ;
+        event.SetOrientation( wxHORIZONTAL );

     else
     else

-        event.SetOrientation( wxVERTICAL ) ;
+        event.SetOrientation( wxVERTICAL );

 
 

-    event.SetPosition(new_pos);
+    event.SetPosition( new_pos );

     event.SetEventObject( this );
 
     event.SetEventObject( this );
 

-    wxWindow* window = GetParent() ;
-    if (window && window->MacIsWindowScrollbar(this) )
+    wxWindow* window = GetParent();
+    if (window && window->MacIsWindowScrollbar( this ))

         // this is hardcoded
         // this is hardcoded

-        window->MacOnScroll(event);
+        window->MacOnScroll( event );

     else
     else

-        GetEventHandler()->ProcessEvent(event);
+        HandleWindowEvent( event );

 }
 
 }
 

-wxInt32 wxScrollBar::MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF mevent )
+wxInt32 wxScrollBar::MacControlHit( WXEVENTHANDLERREF WXUNUSED(handler), WXEVENTREF mevent )

 {
 {

-    int position = m_peer->GetValue() ;
-    int minPos = m_peer->GetMinimum() ;
-    int maxPos = m_peer->GetMaximum() ;
+    int position = m_peer->GetValue();
+    int minPos = m_peer->GetMinimum();
+    int maxPos = m_peer->GetMaximum();

 
     wxEventType scrollEvent = wxEVT_NULL;
     int nScrollInc = 0;
 
 
     wxEventType scrollEvent = wxEVT_NULL;
     int nScrollInc = 0;
 

-    wxMacCarbonEvent cEvent( (EventRef) mevent ) ;
-    ControlPartCode controlpart = cEvent.GetParameter<ControlPartCode>(kEventParamControlPart, typeControlPartCode) ;
+    wxMacCarbonEvent cEvent( (EventRef)mevent );
+    ControlPartCode controlpart = cEvent.GetParameter<ControlPartCode>(kEventParamControlPart, typeControlPartCode);

 
     // all events have already been reported during mouse down, except for THUMBRELEASE
 
     // all events have already been reported during mouse down, except for THUMBRELEASE

+    // NB: this may need to be reviewed in light of the fact that scroll wheel events
+    // aren't being handled properly

     if ( controlpart != kControlIndicatorPart )
     if ( controlpart != kControlIndicatorPart )

-        return eventNotHandledErr ;
+        return eventNotHandledErr;

 
     switch ( controlpart )
     {
 
     switch ( controlpart )
     {

-    case kControlIndicatorPart :
-        nScrollInc = 0 ;
+    case kControlIndicatorPart:
+        nScrollInc = 0;

         scrollEvent = wxEVT_SCROLL_THUMBRELEASE;
         scrollEvent = wxEVT_SCROLL_THUMBRELEASE;

-        break ;
+        break;

 
 

-    default :
-        wxFAIL_MSG(wxT("illegal scrollbar selector"));
-        break ;
+    default:
+        wxFAIL_MSG(wxT("unknown scrollbar selector"));
+        break;

     }
 
     int new_pos = position + nScrollInc;
     }
 
     int new_pos = position + nScrollInc;


@@ -194,22 +206,42 @@ wxInt32 wxScrollBar::MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF meven
         new_pos = maxPos;
 
     if ( nScrollInc )
         new_pos = maxPos;
 
     if ( nScrollInc )

-        SetThumbPosition(new_pos);
+        SetThumbPosition( new_pos );

 
 

-    wxScrollEvent event(scrollEvent, m_windowId);
+    wxScrollEvent event( scrollEvent, m_windowId );

     if ( m_windowStyle & wxHORIZONTAL )
         event.SetOrientation( wxHORIZONTAL );
     else
         event.SetOrientation( wxVERTICAL );
 
     if ( m_windowStyle & wxHORIZONTAL )
         event.SetOrientation( wxHORIZONTAL );
     else
         event.SetOrientation( wxVERTICAL );
 

-    event.SetPosition(new_pos);
+    event.SetPosition( new_pos );

     event.SetEventObject( this );
     event.SetEventObject( this );

-    wxWindow* window = GetParent() ;
-    if (window && window->MacIsWindowScrollbar(this) )
+    wxWindow* window = GetParent();
+    if (window && window->MacIsWindowScrollbar( this ))

         // this is hardcoded
         // this is hardcoded

-        window->MacOnScroll(event);
+        window->MacOnScroll( event );
+    else
+        HandleWindowEvent( event );
+
+    return noErr;
+}
+
+
+wxSize wxScrollBar::DoGetBestSize() const
+{
+    int w = 100;
+    int h = 100;
+
+    if ( IsVertical() )
+    {
+        w = wxSystemSettings::GetMetric(wxSYS_VSCROLL_X);
+    }

     else
     else

-        GetEventHandler()->ProcessEvent(event);
+    {
+        h = wxSystemSettings::GetMetric(wxSYS_HSCROLL_Y);
+    }

 
 

-    return noErr ;
+    wxSize best(w, h);
+    CacheBestSize(best);
+    return best;

 }
 }