warning fix after huge file patch

[wxWidgets.git] / src / common / valgen.cpp

diff --git a/src/common/valgen.cpp b/src/common/valgen.cpp
index 091c2b9db4de22a0887995f455e48185db3566b5..f7daf5556d3c09526aa4e83c7a6d79dd01a60f29 100644 (file)
--- a/src/common/valgen.cpp
+++ b/src/common/valgen.cpp
@@ -4,12 +4,12 @@
 // Author:      Kevin Smith
 // Modified by:
 // Created:     Jan 22 1999
 // Author:      Kevin Smith
 // Modified by:
 // Created:     Jan 22 1999

-// RCS-ID:      
+// RCS-ID:

 // Copyright:   (c) 1999 Kevin Smith
 // Copyright:   (c) 1999 Kevin Smith

-// Licence:    wxWindows licence
+// Licence:           wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "valgen.h"
 #endif
 
 #pragma implementation "valgen.h"
 #endif
 


@@ -17,62 +17,87 @@
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__
 #include "wx/wxprec.h"
 
 #ifdef __BORLANDC__

-#pragma hdrstop
+  #pragma hdrstop

 #endif
 
 #ifndef WX_PRECOMP
 #endif
 
 #ifndef WX_PRECOMP

-#include "wx/utils.h"
-#include "wx/intl.h"
-#include "wx/wx.h"
-#include "wx/dynarray.h"
+  #include "wx/defs.h"

 #endif
 
 #endif
 

-#ifndef __WIN16__
-#include "wx/spinbutt.h"
-#include "wx/checklst.h"
+#if wxUSE_VALIDATORS
+
+#ifndef WX_PRECOMP
+  #include "wx/utils.h"
+  #include "wx/intl.h"
+  #include "wx/dynarray.h"
+  #include "wx/choice.h"
+  #include "wx/combobox.h"
+  #include "wx/radiobox.h"
+  #include "wx/radiobut.h"
+  #include "wx/checkbox.h"
+  #include "wx/scrolbar.h"
+  #include "wx/gauge.h"
+  #include "wx/stattext.h"
+  #include "wx/textctrl.h"
+  #include "wx/button.h"
+  #include "wx/listbox.h"
+  #include "wx/slider.h"
+#endif
+
+
+  #include "wx/spinctrl.h"
+
+#if wxUSE_SPINBTN
+  #include "wx/spinbutt.h"
+#endif
+#if wxUSE_CHECKLISTBOX
+  #include "wx/checklst.h"

 #endif
 
 #include "wx/valgen.h"
 
 #endif
 
 #include "wx/valgen.h"
 

+IMPLEMENT_CLASS(wxGenericValidator, wxValidator)
+

 wxGenericValidator::wxGenericValidator(bool *val)
 {
 wxGenericValidator::wxGenericValidator(bool *val)
 {

-  Initialize();
-  m_pBool = val;
+    Initialize();
+    m_pBool = val;

 }
 
 wxGenericValidator::wxGenericValidator(int *val)
 {
 }
 
 wxGenericValidator::wxGenericValidator(int *val)
 {

-  Initialize();
-  m_pInt = val;
+    Initialize();
+    m_pInt = val;

 }
 
 wxGenericValidator::wxGenericValidator(wxString *val)
 {
 }
 
 wxGenericValidator::wxGenericValidator(wxString *val)
 {

-  Initialize();
-  m_pString = val;
+    Initialize();
+    m_pString = val;

 }
 
 wxGenericValidator::wxGenericValidator(wxArrayInt *val)
 {
 }
 
 wxGenericValidator::wxGenericValidator(wxArrayInt *val)
 {

-  Initialize();
-  m_pArrayInt = val;
+    Initialize();
+    m_pArrayInt = val;

 }
 
 wxGenericValidator::wxGenericValidator(const wxGenericValidator& val)
 }
 
 wxGenericValidator::wxGenericValidator(const wxGenericValidator& val)

+    : wxValidator()

 {
 {

-  Copy(val);
+    Copy(val);

 }
 
 bool wxGenericValidator::Copy(const wxGenericValidator& val)
 {
 }
 
 bool wxGenericValidator::Copy(const wxGenericValidator& val)
 {

-  wxValidator::Copy(val);
+    wxValidator::Copy(val);

 
 

-  m_pBool = val.m_pBool;
-  m_pInt = val.m_pInt;
-  m_pString = val.m_pString;
-  m_pArrayInt = val.m_pArrayInt;
+    m_pBool = val.m_pBool;
+    m_pInt = val.m_pInt;
+    m_pString = val.m_pString;
+    m_pArrayInt = val.m_pArrayInt;

 
 

-  return TRUE;
+    return true;

 }
 
 wxGenericValidator::~wxGenericValidator()
 }
 
 wxGenericValidator::~wxGenericValidator()


@@ -82,307 +107,451 @@ wxGenericValidator::~wxGenericValidator()
 // Called to transfer data to the window
 bool wxGenericValidator::TransferToWindow(void)
 {
 // Called to transfer data to the window
 bool wxGenericValidator::TransferToWindow(void)
 {

-  if ( !m_validatorWindow )
-    return FALSE;
+    if ( !m_validatorWindow )
+        return false;

 
 

-  // bool controls
-  if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckBox)) )
-  {
-    wxCheckBox* pControl = (wxCheckBox*) m_validatorWindow;
-       if (m_pBool)
+    // bool controls
+#if wxUSE_CHECKBOX
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckBox)) )

     {
     {

-      pControl->SetValue(*m_pBool) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioButton)) )
-  {
-    wxRadioButton* pControl = (wxRadioButton*) m_validatorWindow;
-       if (m_pBool)
+        wxCheckBox* pControl = (wxCheckBox*) m_validatorWindow;
+        if (m_pBool)
+        {
+            pControl->SetValue(*m_pBool);
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_RADIOBTN
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioButton)) )

     {
     {

-      pControl->SetValue(*m_pBool) ;
-      return TRUE;
-    }
-  }
-  // int controls
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxGauge)) )
-  {
-    wxGauge* pControl = (wxGauge*) m_validatorWindow;
-       if (m_pInt)
+        wxRadioButton* pControl = (wxRadioButton*) m_validatorWindow;
+        if (m_pBool)
+        {
+            pControl->SetValue(*m_pBool) ;
+            return true;
+        }
+    } else
+#endif
+
+    // int controls
+#if wxUSE_GAUGE
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxGauge)) )

     {
     {

-      pControl->SetValue(*m_pInt) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioBox)) )
-  {
-    wxRadioBox* pControl = (wxRadioBox*) m_validatorWindow;
-       if (m_pInt)
+        wxGauge* pControl = (wxGauge*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetValue(*m_pInt);
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_RADIOBOX
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioBox)) )

     {
     {

-      pControl->SetSelection(*m_pInt) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxScrollBar)) )
-  {
-    wxScrollBar* pControl = (wxScrollBar*) m_validatorWindow;
-       if (m_pInt)
+        wxRadioBox* pControl = (wxRadioBox*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetSelection(*m_pInt) ;
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_SCROLLBAR
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxScrollBar)) )

     {
     {

-      pControl->SetThumbPosition(*m_pInt) ;
-      return TRUE;
-    }
-  }
-#ifndef __WIN16__
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinButton)) )
-  {
-    wxSpinButton* pControl = (wxSpinButton*) m_validatorWindow;
-       if (m_pInt)
+        wxScrollBar* pControl = (wxScrollBar*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetThumbPosition(*m_pInt) ;
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_SPINCTRL && !defined(__WXMOTIF__)
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinCtrl)) )

     {
     {

-      pControl->SetValue(*m_pInt) ;
-      return TRUE;
-    }
-  }
+        wxSpinCtrl* pControl = (wxSpinCtrl*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetValue(*m_pInt);
+            return true;
+        }
+    } else

 #endif
 #endif

-  // string controls
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxButton)) )
-  {
-    wxButton* pControl = (wxButton*) m_validatorWindow;
-       if (m_pString)
+#if wxUSE_SPINBTN
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinButton)) )

     {
     {

-      pControl->SetLabel(*m_pString) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxComboBox)) )
-  {
-    wxComboBox* pControl = (wxComboBox*) m_validatorWindow;
-       if (m_pString)
+        wxSpinButton* pControl = (wxSpinButton*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetValue(*m_pInt) ;
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_SLIDER
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxSlider)) )

     {
     {

-      pControl->SetValue(*m_pString) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxChoice)) )
-  {
-    wxChoice* pControl = (wxChoice*) m_validatorWindow;
-       if (m_pInt)
+        wxSlider* pControl = (wxSlider*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetValue(*m_pInt) ;
+            return true;
+        }
+    } else
+#endif
+
+    // string controls
+#if wxUSE_BUTTON
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxButton)) )

     {
     {

-      pControl->SetSelection(*m_pInt) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxStaticText)) )
-  {
-    wxStaticText* pControl = (wxStaticText*) m_validatorWindow;
-       if (m_pString)
+        wxButton* pControl = (wxButton*) m_validatorWindow;
+        if (m_pString)
+        {
+            pControl->SetLabel(*m_pString) ;
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_COMBOBOX
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxComboBox)) )

     {
     {

-      pControl->SetLabel(*m_pString) ;
-      return TRUE;
-    }
-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxTextCtrl)) )
-  {
-    wxTextCtrl* pControl = (wxTextCtrl*) m_validatorWindow;
-       if (m_pString)
+        wxComboBox* pControl = (wxComboBox*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetSelection(*m_pInt) ;
+            return true;
+        }
+        else if (m_pString)
+        {
+            if (pControl->FindString(* m_pString) != wxNOT_FOUND)
+            {
+                pControl->SetStringSelection(* m_pString);
+            }
+            else
+            {
+                pControl->SetValue(* m_pString);
+            }
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_CHOICE
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxChoice)) )

     {
     {

-      pControl->SetValue(*m_pString) ;
-      return TRUE;
-    }
-  }
-#ifndef __WIN16__
-  // array controls
-  // NOTE: wxCheckListBox isa wxListBox, so wxCheckListBox
-  // MUST come first:
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckListBox)) )
-  {
-    wxCheckListBox* pControl = (wxCheckListBox*) m_validatorWindow;
-       if (m_pArrayInt)
-    {
-      // clear all selections
-      int i;
-      for (i = 0 ; i < pControl->Number(); ++i)
-        pControl->Check(i, FALSE);
-      // select each item in our array
-      unsigned u;
-      for (u = 0; u < m_pArrayInt->Count(); ++u)
-        pControl->Check(m_pArrayInt->Item(u));
-      return TRUE;
-    }
-       else
-               return FALSE;
-  }
+        wxChoice* pControl = (wxChoice*) m_validatorWindow;
+        if (m_pInt)
+        {
+            pControl->SetSelection(*m_pInt) ;
+            return true;
+        }
+        else if (m_pString)
+        {
+            if (pControl->FindString(* m_pString) != wxNOT_FOUND)
+            {
+                pControl->SetStringSelection(* m_pString);
+            }
+            return true;
+        }
+    } else

 #endif
 #endif

-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxListBox)) )
-  {
-    wxListBox* pControl = (wxListBox*) m_validatorWindow;
-       if (m_pArrayInt)
-    {
-      // clear all selections
-      int i;
-      for (i = 0 ; i < pControl->Number(); ++i)
-        pControl->Deselect(i);
-      // select each item in our array
-      unsigned u;
-      for (u = 0; u < m_pArrayInt->Count(); ++u)
-        pControl->SetSelection(m_pArrayInt->Item(u));
-      return TRUE;
-    }
-  }
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxStaticText)) )
+    {
+        wxStaticText* pControl = (wxStaticText*) m_validatorWindow;
+        if (m_pString)
+        {
+            pControl->SetLabel(*m_pString) ;
+            return true;
+        }
+    } else
+#if wxUSE_TEXTCTRL
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxTextCtrl)) )
+    {
+        wxTextCtrl* pControl = (wxTextCtrl*) m_validatorWindow;
+        if (m_pString)
+        {
+            pControl->SetValue(*m_pString) ;
+            return true;
+        }
+        else if (m_pInt)
+        {
+            wxString str;
+            str.Printf(wxT("%d"), *m_pInt);
+            pControl->SetValue(str);
+            return true;
+        }
+    } else
+#endif
+    // array controls
+#if wxUSE_CHECKLISTBOX
+    // NOTE: wxCheckListBox is a wxListBox, so wxCheckListBox MUST come first:
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckListBox)) )
+    {
+        wxCheckListBox* pControl = (wxCheckListBox*) m_validatorWindow;
+        if (m_pArrayInt)
+        {
+            // clear all selections
+            size_t i,
+                   count = pControl->GetCount();
+            for ( i = 0 ; i < count; i++ )
+                pControl->Check(i, false);
+
+            // select each item in our array
+            count = m_pArrayInt->GetCount();
+            for ( i = 0 ; i < count; i++ )
+                pControl->Check(m_pArrayInt->Item(i));
+
+            return true;
+        }
+        else
+            return false;
+    } else
+#endif
+#if wxUSE_LISTBOX
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxListBox)) )
+    {
+        wxListBox* pControl = (wxListBox*) m_validatorWindow;
+        if (m_pArrayInt)
+        {
+            // clear all selections
+            size_t i,
+                   count = pControl->GetCount();
+            for ( i = 0 ; i < count; i++ )
+                pControl->Deselect(i);
+
+            // select each item in our array
+            count = m_pArrayInt->GetCount();
+            for ( i = 0 ; i < count; i++ )
+                pControl->SetSelection(m_pArrayInt->Item(i));
+
+            return true;
+        }
+    } else
+#endif
+        ;   // to match the last 'else' above

 
   // unrecognized control, or bad pointer
 
   // unrecognized control, or bad pointer

-  return FALSE;
+  return false;

 }
 
 }
 

-// Called to transfer data to the window
+// Called to transfer data from the window

 bool wxGenericValidator::TransferFromWindow(void)
 {
   if ( !m_validatorWindow )
 bool wxGenericValidator::TransferFromWindow(void)
 {
   if ( !m_validatorWindow )

-    return FALSE;
+    return false;

 
   // bool controls
 
   // bool controls

+#if wxUSE_CHECKBOX

   if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckBox)) )
   {
     wxCheckBox* pControl = (wxCheckBox*) m_validatorWindow;
   if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckBox)) )
   {
     wxCheckBox* pControl = (wxCheckBox*) m_validatorWindow;

-       if (m_pBool)
+        if (m_pBool)

     {
       *m_pBool = pControl->GetValue() ;
     {
       *m_pBool = pControl->GetValue() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioButton)) )
+  } else
+#endif
+#if wxUSE_RADIOBTN
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioButton)) )

   {
     wxRadioButton* pControl = (wxRadioButton*) m_validatorWindow;
   {
     wxRadioButton* pControl = (wxRadioButton*) m_validatorWindow;

-       if (m_pBool)
+        if (m_pBool)

     {
       *m_pBool = pControl->GetValue() ;
     {
       *m_pBool = pControl->GetValue() ;

-      return TRUE;
+      return true;

     }
     }

-  }
+  } else
+#endif

   // int controls
   // int controls

-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxGauge)) )
+#if wxUSE_GAUGE
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxGauge)) )

   {
     wxGauge* pControl = (wxGauge*) m_validatorWindow;
   {
     wxGauge* pControl = (wxGauge*) m_validatorWindow;

-       if (m_pInt)
+        if (m_pInt)

     {
       *m_pInt = pControl->GetValue() ;
     {
       *m_pInt = pControl->GetValue() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioBox)) )
+  } else
+#endif
+#if wxUSE_RADIOBOX
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxRadioBox)) )

   {
     wxRadioBox* pControl = (wxRadioBox*) m_validatorWindow;
   {
     wxRadioBox* pControl = (wxRadioBox*) m_validatorWindow;

-       if (m_pInt)
+        if (m_pInt)

     {
       *m_pInt = pControl->GetSelection() ;
     {
       *m_pInt = pControl->GetSelection() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxScrollBar)) )
+  } else
+#endif
+#if wxUSE_SCROLLBAR
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxScrollBar)) )

   {
     wxScrollBar* pControl = (wxScrollBar*) m_validatorWindow;
   {
     wxScrollBar* pControl = (wxScrollBar*) m_validatorWindow;

-       if (m_pInt)
+        if (m_pInt)

     {
       *m_pInt = pControl->GetThumbPosition() ;
     {
       *m_pInt = pControl->GetThumbPosition() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-#ifndef __WIN16__
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinButton)) )
+  } else
+#endif
+#if wxUSE_SPINCTRL && !defined(__WXMOTIF__)
+    if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinCtrl)) )
+    {
+        wxSpinCtrl* pControl = (wxSpinCtrl*) m_validatorWindow;
+        if (m_pInt)
+        {
+            *m_pInt=pControl->GetValue();
+            return true;
+        }
+    } else
+#endif
+#if wxUSE_SPINBTN
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxSpinButton)) )

   {
     wxSpinButton* pControl = (wxSpinButton*) m_validatorWindow;
   {
     wxSpinButton* pControl = (wxSpinButton*) m_validatorWindow;

-       if (m_pInt)
+        if (m_pInt)
+    {
+      *m_pInt = pControl->GetValue() ;
+      return true;
+    }
+  } else
+#endif
+#if wxUSE_SLIDER
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxSlider)) )
+  {
+    wxSlider* pControl = (wxSlider*) m_validatorWindow;
+    if (m_pInt)

     {
       *m_pInt = pControl->GetValue() ;
     {
       *m_pInt = pControl->GetValue() ;

-      return TRUE;
+      return true;

     }
     }

-  }
+  } else

 #endif
   // string controls
 #endif
   // string controls

-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxButton)) )
+#if wxUSE_BUTTON
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxButton)) )

   {
     wxButton* pControl = (wxButton*) m_validatorWindow;
   {
     wxButton* pControl = (wxButton*) m_validatorWindow;

-       if (m_pString)
+        if (m_pString)

     {
       *m_pString = pControl->GetLabel() ;
     {
       *m_pString = pControl->GetLabel() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxComboBox)) )
+  } else
+#endif
+#if wxUSE_COMBOBOX
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxComboBox)) )

   {
     wxComboBox* pControl = (wxComboBox*) m_validatorWindow;
   {
     wxComboBox* pControl = (wxComboBox*) m_validatorWindow;

-       if (m_pString)
+    if (m_pInt)

     {
     {

-      *m_pString = pControl->GetValue() ;
-      return TRUE;
+      *m_pInt = pControl->GetSelection() ;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxChoice)) )
+    else if (m_pString)
+    {
+        *m_pString = pControl->GetValue();
+        return true;
+    }
+  } else
+#endif
+#if wxUSE_CHOICE
+ if (m_validatorWindow->IsKindOf(CLASSINFO(wxChoice)) )

   {
     wxChoice* pControl = (wxChoice*) m_validatorWindow;
   {
     wxChoice* pControl = (wxChoice*) m_validatorWindow;

-       if (m_pInt)
+        if (m_pInt)

     {
       *m_pInt = pControl->GetSelection() ;
     {
       *m_pInt = pControl->GetSelection() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxStaticText)) )
+    else if (m_pString)
+    {
+        *m_pString = pControl->GetStringSelection();
+        return true;
+    }
+  } else
+#endif
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxStaticText)) )

   {
     wxStaticText* pControl = (wxStaticText*) m_validatorWindow;
   {
     wxStaticText* pControl = (wxStaticText*) m_validatorWindow;

-       if (m_pString)
+        if (m_pString)

     {
       *m_pString = pControl->GetLabel() ;
     {
       *m_pString = pControl->GetLabel() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxTextCtrl)) )
+  } else
+#if wxUSE_TEXTCTRL
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxTextCtrl)) )

   {
     wxTextCtrl* pControl = (wxTextCtrl*) m_validatorWindow;
   {
     wxTextCtrl* pControl = (wxTextCtrl*) m_validatorWindow;

-       if (m_pString)
+        if (m_pString)

     {
       *m_pString = pControl->GetValue() ;
     {
       *m_pString = pControl->GetValue() ;

-      return TRUE;
+      return true;

     }
     }

-  }
-#ifndef __WIN16__
+    else if (m_pInt)
+    {
+        *m_pInt = wxAtoi(pControl->GetValue());
+        return true;
+    }
+  } else
+#endif

   // array controls
   // array controls

-  // NOTE: wxCheckListBox isa wxListBox, so wxCheckListBox
-  // MUST come first:
-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckListBox)) )
+#if wxUSE_CHECKLISTBOX
+  // NOTE: wxCheckListBox isa wxListBox, so wxCheckListBox MUST come first:
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxCheckListBox)) )

   {
     wxCheckListBox* pControl = (wxCheckListBox*) m_validatorWindow;
   {
     wxCheckListBox* pControl = (wxCheckListBox*) m_validatorWindow;

-       if (m_pArrayInt)
+    if (m_pArrayInt)

     {
       // clear our array
       m_pArrayInt->Clear();
     {
       // clear our array
       m_pArrayInt->Clear();

+

       // add each selected item to our array
       // add each selected item to our array

-      int i;
-      for (i = 0 ; i < pControl->Number(); ++i)
+      size_t i,
+             count = pControl->GetCount();
+      for ( i = 0; i < count; i++ )
+      {

         if (pControl->IsChecked(i))
           m_pArrayInt->Add(i);
         if (pControl->IsChecked(i))
           m_pArrayInt->Add(i);

-      return TRUE;
+      }
+
+      return true;

     }
     }

-       else
-         return FALSE;
-  }
+    else
+      return false;
+  } else

 #endif
 #endif

-  else if (m_validatorWindow->IsKindOf(CLASSINFO(wxListBox)) )
+#if wxUSE_LISTBOX
+  if (m_validatorWindow->IsKindOf(CLASSINFO(wxListBox)) )

   {
     wxListBox* pControl = (wxListBox*) m_validatorWindow;
   {
     wxListBox* pControl = (wxListBox*) m_validatorWindow;

-       if (m_pArrayInt)
+    if (m_pArrayInt)

     {
       // clear our array
       m_pArrayInt->Clear();
     {
       // clear our array
       m_pArrayInt->Clear();

+

       // add each selected item to our array
       // add each selected item to our array

-      int i;
-      for (i = 0 ; i < pControl->Number(); ++i)
+      size_t i,
+             count = pControl->GetCount();
+      for ( i = 0; i < count; i++ )
+      {

         if (pControl->Selected(i))
           m_pArrayInt->Add(i);
         if (pControl->Selected(i))
           m_pArrayInt->Add(i);

-      return TRUE;
+      }
+
+      return true;

     }
     }

-  }
+  } else
+#endif

 
   // unrecognized control, or bad pointer
 
   // unrecognized control, or bad pointer

-  return FALSE;
+    return false;
+  return false;

 }
 
 /*
 }
 
 /*


@@ -390,9 +559,12 @@ bool wxGenericValidator::TransferFromWindow(void)
 */
 void wxGenericValidator::Initialize()
 {
 */
 void wxGenericValidator::Initialize()
 {

-  m_pBool = 0;
-  m_pInt = 0;
-  m_pString = 0;
-  m_pArrayInt = 0;
+    m_pBool = 0;
+    m_pInt = 0;
+    m_pString = 0;
+    m_pArrayInt = 0;

 }
 
 }
 

+#endif
+  // wxUSE_VALIDATORS
+