cleanup - reformatting

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

diff --git a/src/mac/carbon/slider.cpp b/src/mac/carbon/slider.cpp
index d7613a3edddc166794ceaa3ff0fa4476dc734213..559b37eabaa0155c2b0c11c8da2f09b2187c2e75 100644 (file)
--- a/src/mac/carbon/slider.cpp
+++ b/src/mac/carbon/slider.cpp
@@ -21,7 +21,7 @@ IMPLEMENT_DYNAMIC_CLASS(wxSlider, wxControl)
 BEGIN_EVENT_TABLE(wxSlider, wxControl)
 END_EVENT_TABLE()
 
 BEGIN_EVENT_TABLE(wxSlider, wxControl)
 END_EVENT_TABLE()
 

- // The dimensions of the different styles of sliders (From Aqua document)
+ // The dimensions of the different styles of sliders (from Aqua document)

 #define wxSLIDER_DIMENSIONACROSS 15
 #define wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS 24
 #define wxSLIDER_DIMENSIONACROSS_ARROW 18
 #define wxSLIDER_DIMENSIONACROSS 15
 #define wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS 24
 #define wxSLIDER_DIMENSIONACROSS_ARROW 18


@@ -29,21 +29,26 @@ END_EVENT_TABLE()
 // Distance between slider and text
 #define wxSLIDER_BORDERTEXT 5
 
 // Distance between slider and text
 #define wxSLIDER_BORDERTEXT 5
 

-/* NB!  The default orientation for a slider is horizontal however if the user specifies
- * some slider styles but dosen't specify the orientation we have to assume he wants a
- * horizontal one.  Therefore in this file when testing for the sliders orientation
- * vertical is tested for if this is not set then we use the horizontal one
- * eg.  if(GetWindowStyle() & wxSL_VERTICAL) {}  else { horizontal case }>
- */
+/*
+* NB! The default orientation for a slider is horizontal however if the user specifies
+* some slider styles but doesn't specify the orientation we have to assume he wants a
+* horizontal one.  Therefore in this file when testing for the slider's orientation
+* vertical is tested for if this is not set then we use the horizontal one
+* e.g., if (GetWindowStyle() & wxSL_VERTICAL) {}  else { horizontal case }.
+*/

 
 

- // Slider
- wxSlider::wxSlider()
+
+wxSlider::wxSlider()

 {
     m_pageSize = 1;
     m_lineSize = 1;
     m_rangeMax = 0;
     m_rangeMin = 0;
     m_tickFreq = 0;
 {
     m_pageSize = 1;
     m_lineSize = 1;
     m_rangeMax = 0;
     m_rangeMin = 0;
     m_tickFreq = 0;

+
+    m_macMinimumStatic = NULL;
+    m_macMaximumStatic = NULL;
+    m_macValueStatic = NULL;

 }
 
 bool wxSlider::Create(wxWindow *parent, wxWindowID id,
 }
 
 bool wxSlider::Create(wxWindow *parent, wxWindowID id,


@@ -55,6 +60,18 @@ bool wxSlider::Create(wxWindow *parent, wxWindowID id,
 {
     m_macIsUserPane = false ;
 
 {
     m_macIsUserPane = false ;
 

+    m_macMinimumStatic = NULL ;
+    m_macMaximumStatic = NULL ;
+    m_macValueStatic = NULL ;
+
+    m_lineSize = 1;
+    m_tickFreq = 0;
+
+    m_rangeMax = maxValue;
+    m_rangeMin = minValue;
+
+    m_pageSize = (int)((maxValue-minValue)/10);
+

     // our styles are redundant: wxSL_LEFT/RIGHT imply wxSL_VERTICAL and
     // wxSL_TOP/BOTTOM imply wxSL_HORIZONTAL, but for backwards compatibility
     // reasons we can't really change it, instead try to infer the orientation
     // our styles are redundant: wxSL_LEFT/RIGHT imply wxSL_VERTICAL and
     // wxSL_TOP/BOTTOM imply wxSL_HORIZONTAL, but for backwards compatibility
     // reasons we can't really change it, instead try to infer the orientation


@@ -72,13 +89,13 @@ bool wxSlider::Create(wxWindow *parent, wxWindowID id,
             break;
 
         case 0:
             break;
 
         case 0:

+        default:

             // no specific direction, do we have at least the orientation?
             if ( !(style & (wxSL_HORIZONTAL | wxSL_VERTICAL)) )
             // no specific direction, do we have at least the orientation?
             if ( !(style & (wxSL_HORIZONTAL | wxSL_VERTICAL)) )

-            {
-                // no, choose default
+                // no: choose default

                 style |= wxSL_BOTTOM | wxSL_HORIZONTAL;
                 style |= wxSL_BOTTOM | wxSL_HORIZONTAL;

-            }
-    };
+            break;
+    }

 
     wxASSERT_MSG( !(style & wxSL_VERTICAL) || !(style & wxSL_HORIZONTAL),
         _T("incompatible slider direction and orientation") );
 
     wxASSERT_MSG( !(style & wxSL_VERTICAL) || !(style & wxSL_HORIZONTAL),
         _T("incompatible slider direction and orientation") );


@@ -86,55 +103,44 @@ bool wxSlider::Create(wxWindow *parent, wxWindowID id,
     if ( !wxControl::Create(parent, id, pos, size, style, validator, name) )
         return false;
 
     if ( !wxControl::Create(parent, id, pos, size, style, validator, name) )
         return false;
 

-    m_macMinimumStatic = NULL ;
-    m_macMaximumStatic = NULL ;
-    m_macValueStatic = NULL ;
-
-    m_lineSize = 1;
-    m_tickFreq = 0;
-
-    m_rangeMax = maxValue;
-    m_rangeMin = minValue;
-
-    m_pageSize = (int)((maxValue-minValue)/10);
-

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

-    //

     // NB: (RN) Ticks here are sometimes off in the GUI if there
     // is not as many ticks as there are values
     //
     // NB: (RN) Ticks here are sometimes off in the GUI if there
     // is not as many ticks as there are values
     //

-    UInt16 tickMarks = 0 ;
+    int tickMarks = 0 ;

     if ( style & wxSL_AUTOTICKS )
         tickMarks = (maxValue - minValue) + 1; //+1 for the 0 value
 
     if ( style & wxSL_AUTOTICKS )
         tickMarks = (maxValue - minValue) + 1; //+1 for the 0 value
 

+    // keep the number of tickmarks from becoming unwieldly, therefore below it is ok to cast
+    // it to a UInt16

     while (tickMarks > 20)
     while (tickMarks > 20)

-        tickMarks /= 5; //keep the number of tickmarks from becoming unwieldly
+        tickMarks /= 5;

 
 

-    m_peer = new wxMacControl(this) ;
+    m_peer = new wxMacControl( this );

     verify_noerr ( CreateSliderControl( MAC_WXHWND(parent->MacGetTopLevelWindowRef()) , &bounds ,
     verify_noerr ( CreateSliderControl( MAC_WXHWND(parent->MacGetTopLevelWindowRef()) , &bounds ,

-        value , minValue , maxValue , kControlSliderPointsDownOrRight , tickMarks , true /* liveTracking */ ,
+        value , minValue , maxValue , kControlSliderPointsDownOrRight , (UInt16) tickMarks , true /* liveTracking */ ,

         GetwxMacLiveScrollbarActionProc() , m_peer->GetControlRefAddr() ) );
 
         GetwxMacLiveScrollbarActionProc() , m_peer->GetControlRefAddr() ) );
 

-    if(style & wxSL_VERTICAL) {
+    if (style & wxSL_VERTICAL)

         SetSizeHints(10, -1, 10, -1);  // Forces SetSize to use the proper width
         SetSizeHints(10, -1, 10, -1);  // Forces SetSize to use the proper width

-    }
-    else {
+    else

         SetSizeHints(-1, 10, -1, 10);  // Forces SetSize to use the proper height
         SetSizeHints(-1, 10, -1, 10);  // Forces SetSize to use the proper height

-    }
+

     // NB!  SetSizeHints is overloaded by wxSlider and will substitute 10 with the
     // proper dimensions, it also means other people cannot bugger the slider with
     // other values
 
     // NB!  SetSizeHints is overloaded by wxSlider and will substitute 10 with the
     // proper dimensions, it also means other people cannot bugger the slider with
     // other values
 

-    if(style & wxSL_LABELS)
+    if (style & wxSL_LABELS)

     {
         m_macMinimumStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );
         m_macMaximumStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );
         m_macValueStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );
     {
         m_macMinimumStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );
         m_macMaximumStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );
         m_macValueStatic = new wxStaticText( parent, wxID_ANY, wxEmptyString );

-        SetRange(minValue, maxValue);
-        SetValue(value);

     }
 
     }
 

+    SetRange(minValue, maxValue);
+    SetValue(value);
+

     MacPostControlCreate(pos,size) ;
 
     return true;
     MacPostControlCreate(pos,size) ;
 
     return true;


@@ -144,7 +150,7 @@ wxSlider::~wxSlider()
 {
     // this is a special case, as we had to add windows as siblings we are
     // responsible for their disposal, but only if we are not part of a DestroyAllChildren
 {
     // this is a special case, as we had to add windows as siblings we are
     // responsible for their disposal, but only if we are not part of a DestroyAllChildren

-    if ( m_parent && m_parent->IsBeingDeleted() == false )
+    if ( m_parent && !m_parent->IsBeingDeleted() )

     {
         delete m_macMinimumStatic ;
         delete m_macMaximumStatic ;
     {
         delete m_macMinimumStatic ;
         delete m_macMaximumStatic ;


@@ -160,10 +166,12 @@ int wxSlider::GetValue() const
 
 void wxSlider::SetValue(int value)
 {
 
 void wxSlider::SetValue(int value)
 {

-    wxString valuestring ;
-    valuestring.Printf( wxT("%d") , value ) ;

     if ( m_macValueStatic )
     if ( m_macValueStatic )

-        m_macValueStatic->SetLabel( valuestring ) ;
+    {
+        wxString valuestring;
+        valuestring.Printf( wxT("%d") , value );
+        m_macValueStatic->SetLabel( valuestring );
+    }

 
     // We only invert for the setting of the actual native widget
     m_peer->SetValue( ValueInvertOrNot ( value ) ) ;
 
     // We only invert for the setting of the actual native widget
     m_peer->SetValue( ValueInvertOrNot ( value ) ) ;


@@ -179,14 +187,18 @@ void wxSlider::SetRange(int minValue, int maxValue)
     m_peer->SetMinimum( m_rangeMin);
     m_peer->SetMaximum( m_rangeMax);
 
     m_peer->SetMinimum( m_rangeMin);
     m_peer->SetMaximum( m_rangeMax);
 

-    if(m_macMinimumStatic) {
+    if (m_macMinimumStatic)
+    {

         value.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMin ) );
         m_macMinimumStatic->SetLabel(value);
     }
         value.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMin ) );
         m_macMinimumStatic->SetLabel(value);
     }

-    if(m_macMaximumStatic) {
+
+    if (m_macMaximumStatic)
+    {

         value.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         m_macMaximumStatic->SetLabel(value);
     }
         value.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         m_macMaximumStatic->SetLabel(value);
     }

+

     SetValue(m_rangeMin);
 }
 
     SetValue(m_rangeMin);
 }
 


@@ -265,15 +277,15 @@ void wxSlider::SetTick(int tickPos)
 
 void wxSlider::Command (wxCommandEvent & event)
 {
 
 void wxSlider::Command (wxCommandEvent & event)
 {

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

 }
 
 void wxSlider::MacHandleControlClick( WXWidget control , wxInt16 controlpart, bool mouseStillDown )
 {
     // Whatever the native value is, we may need to invert it for calling
     // SetValue and putting the possibly inverted value in the event
 }
 
 void wxSlider::MacHandleControlClick( WXWidget control , wxInt16 controlpart, bool mouseStillDown )
 {
     // Whatever the native value is, we may need to invert it for calling
     // SetValue and putting the possibly inverted value in the event

-    SInt16 value = ValueInvertOrNot ( m_peer->GetValue() ) ;
+    int value = ValueInvertOrNot ( m_peer->GetValue() ) ;

 
     SetValue( value ) ;
 
 
     SetValue( value ) ;
 


@@ -297,7 +309,7 @@ wxInt32 wxSlider::MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF mevent )
 {
     // Whatever the native value is, we may need to invert it for calling
     // SetValue and putting the possibly inverted value in the event
 {
     // Whatever the native value is, we may need to invert it for calling
     // SetValue and putting the possibly inverted value in the event

-    SInt16 value = ValueInvertOrNot ( m_peer->GetValue() ) ;
+    int value = ValueInvertOrNot ( m_peer->GetValue() ) ;

 
     SetValue( value ) ;
 
 
     SetValue( value ) ;
 


@@ -315,34 +327,37 @@ wxInt32 wxSlider::MacControlHit( WXEVENTHANDLERREF handler , WXEVENTREF mevent )
     cevent.SetEventObject( this );
 
     GetEventHandler()->ProcessEvent( cevent );
     cevent.SetEventObject( this );
 
     GetEventHandler()->ProcessEvent( cevent );

-    return noErr ;
+
+    return noErr;

 }
 
 }
 

-/* This is overloaded in wxSlider so that the proper width/height will always be used
-* for the slider different values would cause redrawing and mouse detection problems */
+// This is overloaded in wxSlider so that the proper width/height will always be used
+// for the slider different values would cause redrawing and mouse detection problems
+//

 void wxSlider::DoSetSizeHints( int minW, int minH,
                                int maxW , int maxH ,
                                int incW , int incH )
 {
     wxSize size = GetBestSize();
 
 void wxSlider::DoSetSizeHints( int minW, int minH,
                                int maxW , int maxH ,
                                int incW , int incH )
 {
     wxSize size = GetBestSize();
 

-    if(GetWindowStyle() & wxSL_VERTICAL) {
+    if (GetWindowStyle() & wxSL_VERTICAL)

         wxWindow::DoSetSizeHints(size.x, minH, size.x, maxH, incW, incH);
         wxWindow::DoSetSizeHints(size.x, minH, size.x, maxH, incW, incH);

-    }
-    else {
+    else

         wxWindow::DoSetSizeHints(minW, size.y, maxW, size.y, incW, incH);
         wxWindow::DoSetSizeHints(minW, size.y, maxW, size.y, incW, incH);

-    }

 }
 
 wxSize wxSlider::DoGetBestSize() const
 {
     wxSize size;
 }
 
 wxSize wxSlider::DoGetBestSize() const
 {
     wxSize size;

-    int textwidth = 0;
-    int textheight = 0;
+    int textwidth, textheight;

     int mintwidth, mintheight;
     int maxtwidth, maxtheight;
 
     int mintwidth, mintheight;
     int maxtwidth, maxtheight;
 

-    if(GetWindowStyle() & wxSL_LABELS)
+    textwidth = textheight = 0;
+    mintwidth = mintheight = 0;
+    maxtwidth = maxtheight = 0;
+
+    if (GetWindowStyle() & wxSL_LABELS)

     {
         wxString text;
 
     {
         wxString text;
 


@@ -351,64 +366,60 @@ wxSize wxSlider::DoGetBestSize() const
         GetTextExtent(text, &mintwidth, &mintheight);
         text.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         GetTextExtent(text, &maxtwidth, &maxtheight);
         GetTextExtent(text, &mintwidth, &mintheight);
         text.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         GetTextExtent(text, &maxtwidth, &maxtheight);

-        if(maxtheight > mintheight) {
+
+        if (maxtheight > mintheight)

             textheight = maxtheight;
             textheight = maxtheight;

-        }
-        else {
+        else

             textheight = mintheight;
             textheight = mintheight;

-        }
-        if (maxtwidth > mintwidth) {
+
+        if (maxtwidth > mintwidth)

             textwidth = maxtwidth;
             textwidth = maxtwidth;

-        }
-        else {
+        else

             textwidth = mintwidth;
             textwidth = mintwidth;

-        }

     }
 
     }
 

-    if(GetWindowStyle() & wxSL_VERTICAL)
+    if (GetWindowStyle() & wxSL_VERTICAL)

     {
     {

-        if(GetWindowStyle() & wxSL_AUTOTICKS) {
+        if (GetWindowStyle() & wxSL_AUTOTICKS)

             size.x = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;
             size.x = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;

-        }
-        else {
+        else

             size.x = wxSLIDER_DIMENSIONACROSS_ARROW;
             size.x = wxSLIDER_DIMENSIONACROSS_ARROW;

-        }
-        if(GetWindowStyle() & wxSL_LABELS) {
+
+        if (GetWindowStyle() & wxSL_LABELS)

             size.x += textwidth + wxSLIDER_BORDERTEXT;
             size.x += textwidth + wxSLIDER_BORDERTEXT;

-        }
+

         size.y = 150;
     }
     else
     {
         size.y = 150;
     }
     else
     {

-        if(GetWindowStyle() & wxSL_AUTOTICKS) {
+        if (GetWindowStyle() & wxSL_AUTOTICKS)

             size.y = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;
             size.y = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;

-        }
-        else {
+        else

             size.y = wxSLIDER_DIMENSIONACROSS_ARROW;
             size.y = wxSLIDER_DIMENSIONACROSS_ARROW;

-        }

 
         size.x = 150;
 
 
         size.x = 150;
 

-        if(GetWindowStyle() & wxSL_LABELS) {
+        if (GetWindowStyle() & wxSL_LABELS)
+        {

             size.y += textheight + wxSLIDER_BORDERTEXT;
             size.y += textheight + wxSLIDER_BORDERTEXT;

-            size.x += (mintwidth/2) + (maxtwidth/2);
+            size.x += (mintwidth / 2) + (maxtwidth / 2);

         }
     }
         }
     }

+

     return size;
 }
 
 void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
 {
     return size;
 }
 
 void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
 {

-    int  xborder, yborder;
-    int  minValWidth, maxValWidth, textheight;
-    int  sliderBreadth;
+    int xborder, yborder;
+    int minValWidth, maxValWidth, textheight;
+    int sliderBreadth;

     int width = w;
 
     xborder = yborder = 0;
 
     if (GetWindowStyle() & wxSL_LABELS)
     {
     int width = w;
 
     xborder = yborder = 0;
 
     if (GetWindowStyle() & wxSL_LABELS)
     {

-

         wxString text;
         int ht, valValWidth;
 
         wxString text;
         int ht, valValWidth;
 


@@ -418,62 +429,55 @@ void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
         text.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         GetTextExtent(text, &maxValWidth, &ht);
 
         text.Printf(wxT("%d"), ValueInvertOrNot( m_rangeMax ) );
         GetTextExtent(text, &maxValWidth, &ht);
 

-        if(ht > textheight) {
+        if (ht > textheight)

             textheight = ht;
             textheight = ht;

-        }

 
 

-        if(GetWindowStyle() & wxSL_HORIZONTAL)
+        if (GetWindowStyle() & wxSL_HORIZONTAL)

         {
         {

-          if ( m_macMinimumStatic ) {
-            w-=minValWidth/2;
-            x+=minValWidth/2;
-        }
-          if ( m_macMaximumStatic ) {
-            w-=maxValWidth/2;
-          }
-        }
+            if ( m_macMinimumStatic )
+            {
+                w -= minValWidth / 2;
+                x += minValWidth / 2;
+            }

 
 

+            if ( m_macMaximumStatic )
+                w -= maxValWidth / 2;
+        }

 
 

-        //Labels have this control's parent as their parent
-        //so if this control is not at 0,0 relative to the parent
-        //the labels need to know the position of this control
-        //relative to its parent in order to size properly, so
-        //move the control first so we can use GetPosition()
-        wxControl::DoSetSize( x, y , w , h ,sizeFlags ) ;
+        // Labels have this control's parent as their parent
+        // so if this control is not at 0,0 relative to the parent
+        // the labels need to know the position of this control
+        // relative to its parent in order to size properly, so
+        // move the control first so we can use GetPosition()
+        wxControl::DoSetSize( x, y , w , h , sizeFlags );

 
 

-        // If vertical, use current value
-        if(GetWindowStyle() & wxSL_VERTICAL)
-        {
+        if (GetWindowStyle() & wxSL_VERTICAL)
+            // If vertical, use current value

             text.Printf(wxT("%d"), (int)m_peer->GetValue());
             text.Printf(wxT("%d"), (int)m_peer->GetValue());

-        }
-        // Use max so that the current value doesn't drift as centering would need to change

         else
         else

-        {
+            // Use max so that the current value doesn't drift as centering would need to change

             text.Printf(wxT("%d"), m_rangeMax);
             text.Printf(wxT("%d"), m_rangeMax);

-        }

 
         GetTextExtent(text, &valValWidth, &ht);
 
         yborder = textheight + wxSLIDER_BORDERTEXT;
 
         // Get slider breadth
 
         GetTextExtent(text, &valValWidth, &ht);
 
         yborder = textheight + wxSLIDER_BORDERTEXT;
 
         // Get slider breadth

-        if(GetWindowStyle() & wxSL_AUTOTICKS) {
+        if (GetWindowStyle() & wxSL_AUTOTICKS)

             sliderBreadth = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;
             sliderBreadth = wxSLIDER_DIMENSIONACROSS_WITHTICKMARKS;

-        }
-        else {
+        else

             sliderBreadth = wxSLIDER_DIMENSIONACROSS_ARROW;
             sliderBreadth = wxSLIDER_DIMENSIONACROSS_ARROW;

-        }

 
 

-        if(GetWindowStyle() & wxSL_VERTICAL)
+        if (GetWindowStyle() & wxSL_VERTICAL)

         {
         {

-            h = h - yborder ;
+            h = h - yborder;

 
             if ( m_macMinimumStatic )
                 m_macMinimumStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + h - yborder);
             if ( m_macMaximumStatic )
                 m_macMaximumStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + 0);
             if ( m_macValueStatic )
 
             if ( m_macMinimumStatic )
                 m_macMinimumStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + h - yborder);
             if ( m_macMaximumStatic )
                 m_macMaximumStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + 0);
             if ( m_macValueStatic )

-                m_macValueStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + (h/2) - (ht/2));
+                m_macValueStatic->Move(GetPosition().x + sliderBreadth + wxSLIDER_BORDERTEXT, GetPosition().y + (h / 2) - (ht / 2));

         }
         else
         {
         }
         else
         {


@@ -482,7 +486,7 @@ void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
             if ( m_macMaximumStatic )
                  m_macMaximumStatic->Move(GetPosition().x + w - maxValWidth, GetPosition().y + sliderBreadth + wxSLIDER_BORDERTEXT);
             if ( m_macValueStatic )
             if ( m_macMaximumStatic )
                  m_macMaximumStatic->Move(GetPosition().x + w - maxValWidth, GetPosition().y + sliderBreadth + wxSLIDER_BORDERTEXT);
             if ( m_macValueStatic )

-                m_macValueStatic->Move(GetPosition().x + (w/2) - (valValWidth/2), GetPosition().y + sliderBreadth + wxSLIDER_BORDERTEXT);
+                m_macValueStatic->Move(GetPosition().x + (w / 2) - (valValWidth / 2), GetPosition().y + sliderBreadth + wxSLIDER_BORDERTEXT);

         }
     }
 
         }
     }
 


@@ -495,8 +499,7 @@ void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
 
     // TODO: Can all of this code go in the conditional wxSL_LABELS block?
 
 
     // TODO: Can all of this code go in the conditional wxSL_LABELS block?
 

-    int minWidth;
-    minWidth = m_minWidth;
+    int minWidth = m_minWidth;

 
     if (GetWindowStyle() & wxSL_LABELS)
     {
 
     if (GetWindowStyle() & wxSL_LABELS)
     {


@@ -504,35 +507,40 @@ void wxSlider::DoSetSize(int x, int y, int w, int h, int sizeFlags)
         if (width == GetSize().x)
             m_minWidth = -1;
     }
         if (width == GetSize().x)
             m_minWidth = -1;
     }

-    //If the control has labels, we still need to call this again because
-    //the labels alter the control's w and h values.
-    wxControl::DoSetSize( x, y , w , h ,sizeFlags ) ;
+
+    // If the control has labels, we still need to call this again because
+    // the labels alter the control's w and h values.
+    wxControl::DoSetSize( x, y, w, h, sizeFlags );

 
     m_minWidth = minWidth;
 }
 
 void wxSlider::DoMoveWindow(int x, int y, int width, int height)
 {
 
     m_minWidth = minWidth;
 }
 
 void wxSlider::DoMoveWindow(int x, int y, int width, int height)
 {

-    wxControl::DoMoveWindow(x,y,width,height) ;
+    wxControl::DoMoveWindow( x, y, width, height );

 }
 
 // Common processing to invert slider values based on wxSL_INVERSE
 int wxSlider::ValueInvertOrNot(int value) const
 {
 }
 
 // Common processing to invert slider values based on wxSL_INVERSE
 int wxSlider::ValueInvertOrNot(int value) const
 {

+    int result = 0;
+

     if (m_windowStyle & wxSL_VERTICAL)
     {
         // The reason for the backwards logic is that Mac's vertical sliders are
         // inverted compared to Windows and GTK, hence we want inversion to be the
         // default, and if wxSL_INVERSE is set, then we do not invert (use native)
         if (m_windowStyle & wxSL_INVERSE)
     if (m_windowStyle & wxSL_VERTICAL)
     {
         // The reason for the backwards logic is that Mac's vertical sliders are
         // inverted compared to Windows and GTK, hence we want inversion to be the
         // default, and if wxSL_INVERSE is set, then we do not invert (use native)
         if (m_windowStyle & wxSL_INVERSE)

-            return value;
+            result = value;

         else
         else

-            return (m_rangeMax + m_rangeMin) - value;
+            result = (m_rangeMax + m_rangeMin) - value;

     }
     else // normal logic applies to HORIZONTAL sliders
     {
     }
     else // normal logic applies to HORIZONTAL sliders
     {

-        return wxSliderBase::ValueInvertOrNot(value);
+        result = wxSliderBase::ValueInvertOrNot(value);

     }
     }

+
+    return result;

 }
 
 #endif // wxUSE_SLIDER
 }
 
 #endif // wxUSE_SLIDER