No real changes, just use wxString::clear() instead of assignment.

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

diff --git a/src/msw/spinctrl.cpp b/src/msw/spinctrl.cpp
index 4625a25b4914556010ee83db975037e498621764..a034cda2ee9a83553cf23343920b3bcff7434787 100644 (file)
--- a/src/msw/spinctrl.cpp
+++ b/src/msw/spinctrl.cpp
@@ -399,12 +399,11 @@ bool wxSpinCtrl::Create(wxWindow *parent,
     if ( value.ToLong(&initialFromText) )
         initial = initialFromText;
 
     if ( value.ToLong(&initialFromText) )
         initial = initialFromText;
 

-    SetValue(initial);
-
-    m_oldValue = initial;
-
-    // Set the range in the native control
+    // Set the range in the native control: notice that we must do it before
+    // calling SetValue() to use the correct validity checks for the initial
+    // value.

     SetRange(min, max);
     SetRange(min, max);

+    SetValue(initial);

 
     // Also set the text part of the control if it was specified independently
     // but don't generate an event for this, it would be unexpected.
 
     // Also set the text part of the control if it was specified independently
     // but don't generate an event for this, it would be unexpected.