wxGetUserHome() may return const val.

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

diff --git a/src/common/valtext.cpp b/src/common/valtext.cpp
index 259b201326f81b6295cd5ea2abb49879276bd3ad..4f1740a7499d2cfe82306e4d74345eeb13d63fed 100644 (file)
--- a/src/common/valtext.cpp
+++ b/src/common/valtext.cpp
@@ -75,14 +75,14 @@ bool wxTextValidator::Copy(const wxTextValidator& val)
     wxNode *node = val.m_includeList.First() ;
     while ( node )
     {
-        char *s = (char *)node->Data();
+        wxChar *s = (wxChar *)node->Data();
         m_includeList.Add(s);
         node = node->Next();
     }
     node = val.m_excludeList.First() ;
     while ( node )
     {
-        char *s = (char *)node->Data();
+        wxChar *s = (wxChar *)node->Data();
         m_excludeList.Add(s);
         node = node->Next();
     }
@@ -128,52 +128,52 @@ bool wxTextValidator::Validate(wxWindow *parent)
 
     wxTextCtrl *control = (wxTextCtrl *) m_validatorWindow ;
 
-    // If window is disabled, don't validate
+    // If window is disabled, simply return
     if ( !control->Enabled() )
-        return FALSE;
+        return TRUE;
 
     wxString val(control->GetValue());
 
-    bool ok = true;
+    bool ok = TRUE;
 
     // this format string should contian exactly one '%s'
-    const char *errormsg = _("'%s' is invalid");
+    const wxChar *errormsg = _("'%s' is invalid");
 
     if ( m_validatorStyle & wxFILTER_INCLUDE_LIST )
     {
         if ( !m_includeList.Member(val) )
         {
-            ok = false;
+            ok = FALSE;
         }
     }
     else if ( m_validatorStyle & wxFILTER_EXCLUDE_LIST )
     {
         if ( m_excludeList.Member(val) )
         {
-            ok = false;
+            ok = FALSE;
         }
     }
     else if ( (m_validatorStyle & wxFILTER_ASCII) && !val.IsAscii() )
     {
-        ok = false;
+        ok = FALSE;
 
         errormsg = _("'%s' should only contain ASCII characters.");
     }
     else if ( (m_validatorStyle & wxFILTER_ALPHA) && !wxIsAlpha(val) )
     {
-        ok = false;
+        ok = FALSE;
 
         errormsg = _("'%s' should only contain alphabetic characters.");
     }
     else if ( (m_validatorStyle & wxFILTER_ALPHANUMERIC) && !wxIsAlphaNumeric(val))
     {
-        ok = false;
+        ok = FALSE;
 
         errormsg = _("'%s' should only contain alphabetic or numeric characters.");
     }
     else if ( (m_validatorStyle & wxFILTER_NUMERIC) && !wxIsNumeric(val))
     {
-        ok = false;
+        ok = FALSE;
 
         errormsg = _("'%s' should be numeric.");
     }
@@ -236,7 +236,7 @@ void wxTextValidator::SetIncludeList(const wxStringList& list)
     wxNode *node = list.First() ;
     while ( node )
     {
-        char *s = (char *)node->Data();
+        wxChar *s = (wxChar *)node->Data();
         m_includeList.Add(s);
         node = node->Next();
     }
@@ -254,7 +254,7 @@ void wxTextValidator::SetExcludeList(const wxStringList& list)
     wxNode *node = list.First() ;
     while ( node )
     {
-        char *s = (char *)node->Data();
+        wxChar *s = (wxChar *)node->Data();
         m_excludeList.Add(s);
         node = node->Next();
     }