wxWindow::Fit() infinite loop bug fixed

[wxWidgets.git] / src / msw / spinbutt.cpp

diff --git a/src/msw/spinbutt.cpp b/src/msw/spinbutt.cpp
index 03657deaa0d710c27ceece62de130f82127e11ec..e1efced86ed5b434a3e148b7337fda7ca534760a 100644 (file)
--- a/src/msw/spinbutt.cpp
+++ b/src/msw/spinbutt.cpp
@@ -6,22 +6,22 @@
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem
 // Created:     04/01/98
 // RCS-ID:      $Id$
 // Copyright:   (c) Julian Smart and Markus Holzem

-// Licence:    wxWindows license
+// Licence:     wxWindows license

 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__
 /////////////////////////////////////////////////////////////////////////////
 
 #ifdef __GNUG__

-#pragma implementation "spinbutt.h"
+    #pragma implementation "spinbutt.h"

 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 #endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+    #pragma hdrstop

 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP

-#include "wx/wx.h"
+    #include "wx/wx.h"

 #endif
 
 // Can't resolve reference to CreateUpDownControl in
 #endif
 
 // Can't resolve reference to CreateUpDownControl in


@@ -33,17 +33,17 @@
 #include "wx/msw/private.h"
 
 #if !defined(__GNUWIN32__) || defined(__TWIN32__)
 #include "wx/msw/private.h"
 
 #if !defined(__GNUWIN32__) || defined(__TWIN32__)

-#include <commctrl.h>
+    #include <commctrl.h>

 #endif
 
 #if !USE_SHARED_LIBRARY
 #endif
 
 #if !USE_SHARED_LIBRARY

-IMPLEMENT_DYNAMIC_CLASS(wxSpinButton, wxControl)
+    IMPLEMENT_DYNAMIC_CLASS(wxSpinButton, wxControl)

 #endif
 
 #endif
 

-wxSpinButton::wxSpinButton(void)
+wxSpinButton::wxSpinButton()

 {
 {

-       m_min = 0;
-       m_max = 100;
+    m_min = 0;
+    m_max = 100;

 }
 
 bool wxSpinButton::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos, const wxSize& size,
 }
 
 bool wxSpinButton::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos, const wxSize& size,


@@ -81,11 +81,11 @@ bool wxSpinButton::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos, c
   DWORD wstyle = WS_VISIBLE | WS_CHILD | WS_TABSTOP;
   
   if ( m_windowStyle & wxSP_HORIZONTAL )
   DWORD wstyle = WS_VISIBLE | WS_CHILD | WS_TABSTOP;
   
   if ( m_windowStyle & wxSP_HORIZONTAL )

-       wstyle |= UDS_HORZ;
+    wstyle |= UDS_HORZ;

   if ( m_windowStyle & wxSP_ARROW_KEYS )
   if ( m_windowStyle & wxSP_ARROW_KEYS )

-       wstyle |= UDS_ARROWKEYS;
+    wstyle |= UDS_ARROWKEYS;

   if ( m_windowStyle & wxSP_WRAP )
   if ( m_windowStyle & wxSP_WRAP )

-       wstyle |= UDS_WRAP;
+    wstyle |= UDS_WRAP;

 
   // Create the ListView control.
   HWND hWndListControl = CreateUpDownControl(wstyle,
 
   // Create the ListView control.
   HWND hWndListControl = CreateUpDownControl(wstyle,


@@ -94,177 +94,106 @@ bool wxSpinButton::Create(wxWindow *parent, wxWindowID id, const wxPoint& pos, c
     m_windowId,
     wxGetInstance(),
     0,
     m_windowId,
     wxGetInstance(),
     0,

-       m_min, m_max, m_min);
+    m_min, m_max, m_min);

 
   m_hWnd = (WXHWND) hWndListControl;
   if (parent) parent->AddChild(this);
 
   // TODO: have this for all controls.
   if ( !m_hWnd )
 
   m_hWnd = (WXHWND) hWndListControl;
   if (parent) parent->AddChild(this);
 
   // TODO: have this for all controls.
   if ( !m_hWnd )

-       return FALSE;
+    return FALSE;

   
   SubclassWin((WXHWND) m_hWnd);
 
   return TRUE;
 }
 
   
   SubclassWin((WXHWND) m_hWnd);
 
   return TRUE;
 }
 

-wxSpinButton::~wxSpinButton(void)
+wxSpinButton::~wxSpinButton()

 {
 }
 
 // Attributes
 ////////////////////////////////////////////////////////////////////////////
 
 {
 }
 
 // Attributes
 ////////////////////////////////////////////////////////////////////////////
 

-int wxSpinButton::GetValue(void) const
+int wxSpinButton::GetValue() const

 {
 {

-       return (int) ::SendMessage((HWND) GetHWND(), UDM_GETPOS, 0, 0);
+    return (int) ::SendMessage((HWND) GetHWND(), UDM_GETPOS, 0, 0);

 }
 
 void wxSpinButton::SetValue(int val)
 {
 }
 
 void wxSpinButton::SetValue(int val)
 {

-       ::SendMessage((HWND) GetHWND(), UDM_SETPOS, 0, (LPARAM) MAKELONG((short) val, 0));
+    ::SendMessage((HWND) GetHWND(), UDM_SETPOS, 0, (LPARAM) MAKELONG((short) val, 0));

 }
 
 void wxSpinButton::SetRange(int minVal, int maxVal)
 {
 }
 
 void wxSpinButton::SetRange(int minVal, int maxVal)
 {

-       m_min = minVal;
-       m_max = maxVal;
-       ::SendMessage((HWND) GetHWND(), UDM_SETRANGE, 0,
+    m_min = minVal;
+    m_max = maxVal;
+    ::SendMessage((HWND) GetHWND(), UDM_SETRANGE, 0,

                    (LPARAM) MAKELONG((short)maxVal, (short)minVal));
 }
 
                    (LPARAM) MAKELONG((short)maxVal, (short)minVal));
 }
 

-void wxSpinButton::MSWOnVScroll(WXWORD wParam, WXWORD pos, WXHWND control)
+bool wxSpinButton::MSWOnScroll(int orientation, WXWORD wParam,
+                               WXWORD pos, WXHWND control)

 {
 {

-  if (control)
-  {
+    if ( !control )
+        return FALSE;
+

     wxSpinEvent event(wxEVT_NULL, m_windowId);
     event.SetPosition(pos);
     wxSpinEvent event(wxEVT_NULL, m_windowId);
     event.SetPosition(pos);

-    event.SetOrientation(wxVERTICAL);
-    event.SetEventObject( this );
+    event.SetOrientation(orientation);
+    event.SetEventObject(this);

 
 

-       switch ( wParam )
-       {
-               case SB_TOP:
-                       event.m_eventType = wxEVT_SCROLL_TOP;
-                       break;
+    switch ( wParam )
+    {
+        case SB_TOP:
+            event.m_eventType = wxEVT_SCROLL_TOP;
+            break;

 
 

-               case SB_BOTTOM:
-                       event.m_eventType = wxEVT_SCROLL_BOTTOM;
-                       break;
+        case SB_BOTTOM:
+            event.m_eventType = wxEVT_SCROLL_BOTTOM;
+            break;

 
 

-               case SB_LINEUP:
-                       event.m_eventType = wxEVT_SCROLL_LINEUP;
-                       break;
+        case SB_LINEUP:
+            event.m_eventType = wxEVT_SCROLL_LINEUP;
+            break;

 
 

-               case SB_LINEDOWN:
-                       event.m_eventType = wxEVT_SCROLL_LINEDOWN;
-                       break;
+        case SB_LINEDOWN:
+            event.m_eventType = wxEVT_SCROLL_LINEDOWN;
+            break;

 
 

-               case SB_PAGEUP:
-                        event.m_eventType = wxEVT_SCROLL_PAGEUP;
-                       break;
+        case SB_PAGEUP:
+            event.m_eventType = wxEVT_SCROLL_PAGEUP;
+            break;

 
 

-               case SB_PAGEDOWN:
-                        event.m_eventType = wxEVT_SCROLL_PAGEDOWN;
-                       break;
+        case SB_PAGEDOWN:
+            event.m_eventType = wxEVT_SCROLL_PAGEDOWN;
+            break;

 
         case SB_THUMBTRACK:
         case SB_THUMBPOSITION:
 
         case SB_THUMBTRACK:
         case SB_THUMBPOSITION:

-                        event.m_eventType = wxEVT_SCROLL_THUMBTRACK;
-                       break;
-
-               default:
-                        return;
-                        break;
-       }
-    if (!GetEventHandler()->ProcessEvent(event))
-      Default();
-  }
-}
-
-void wxSpinButton::MSWOnHScroll( WXWORD wParam, WXWORD pos, WXHWND control)
-{
-  if (control)
-  {
-    wxSpinEvent event(wxEVT_NULL, m_windowId);
-    event.SetPosition(pos);
-    event.SetOrientation(wxHORIZONTAL);
-    event.SetEventObject( this );
-
-       switch ( wParam )
-       {
-               case SB_TOP:
-                       event.m_eventType = wxEVT_SCROLL_TOP;
-                       break;
-
-               case SB_BOTTOM:
-                       event.m_eventType = wxEVT_SCROLL_BOTTOM;
-                       break;
-
-               case SB_LINEUP:
-                       event.m_eventType = wxEVT_SCROLL_LINEUP;
-                       break;
+            event.m_eventType = wxEVT_SCROLL_THUMBTRACK;
+            break;

 
 

-               case SB_LINEDOWN:
-               event.m_eventType = wxEVT_SCROLL_LINEDOWN;
-                       break;
+        default:
+            return FALSE;
+    }

 
 

-               case SB_PAGEUP:
-                        event.m_eventType = wxEVT_SCROLL_PAGEUP;
-                       break;
-
-               case SB_PAGEDOWN:
-                        event.m_eventType = wxEVT_SCROLL_PAGEDOWN;
-                       break;
-
-        case SB_THUMBTRACK:
-        case SB_THUMBPOSITION:
-                        event.m_eventType = wxEVT_SCROLL_THUMBTRACK;
-                       break;
-
-               default:
-                        return;
-                        break;
-       }
-    if (!GetEventHandler()->ProcessEvent(event))
-       Default();
-  }
+    return GetEventHandler()->ProcessEvent(event);

 }
 
 bool wxSpinButton::MSWCommand(WXUINT cmd, WXWORD id)
 {
 }
 
 bool wxSpinButton::MSWCommand(WXUINT cmd, WXWORD id)
 {

-  // No command messages
-  return FALSE;
-}
-
-bool wxSpinButton::MSWNotify(WXWPARAM wParam, WXLPARAM lParam, WXLPARAM* result)
-{
-       NMHDR* hdr1 = (NMHDR*) lParam;
-       switch ( hdr1->code )
-       {
-        /* We don't process this message, currently */
-               case UDN_DELTAPOS:
-
-               default :
-                       return wxControl::MSWNotify(wParam, lParam, result);
-                       break;
-       }
-/*
-       event.eventObject = this;
-       event.SetEventType(eventType);
-
-       if ( !GetEventHandler()->ProcessEvent(event) )
-               return FALSE;
-*/
-       return TRUE;
+    // No command messages
+    return FALSE;

 }
 
 // Spin event
 IMPLEMENT_DYNAMIC_CLASS(wxSpinEvent, wxScrollEvent)
 
 }
 
 // Spin event
 IMPLEMENT_DYNAMIC_CLASS(wxSpinEvent, wxScrollEvent)
 

-wxSpinEvent::wxSpinEvent(wxEventType commandType, int id):
-  wxScrollEvent(commandType, id)
+wxSpinEvent::wxSpinEvent(wxEventType commandType, int id)
+           : wxScrollEvent(commandType, id)

 {
 }
 
 {
 }
 

-#endif
+#endif // __WIN95__