Sorry, should be a __UNIX__ test

[wxWidgets.git] / include / wx / valgen.h

diff --git a/include/wx/valgen.h b/include/wx/valgen.h
index f80ced14c3f9b2a127bd52bd7d588574250a385b..395f06a7c012aa224d57e2dc23c923250831c853 100644 (file)
--- a/include/wx/valgen.h
+++ b/include/wx/valgen.h
@@ -12,7 +12,7 @@
 #ifndef _WX_VALGENH__
 #define _WX_VALGENH__
 
-#if defined(__GNUG__) && !defined(__APPLE__)
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
 #pragma interface "valgen.h"
 #endif
 
@@ -30,7 +30,7 @@ public:
   wxGenericValidator(wxArrayInt* val);
   wxGenericValidator(const wxGenericValidator& copyFrom);
 
-  ~wxGenericValidator();
+  ~wxGenericValidator(){}
 
   // Make a clone of this validator (or return NULL) - currently necessary
   // if you're passing a reference to a validator.
@@ -41,7 +41,7 @@ public:
 
   // Called when the value in the window must be validated.
   // This function can pop up an error message.
-  virtual bool Validate(wxWindow * WXUNUSED(parent)) { return TRUE; }
+  virtual bool Validate(wxWindow * WXUNUSED(parent)) { return true; }
 
   // Called to transfer data to the window
   virtual bool TransferToWindow();
@@ -56,10 +56,18 @@ protected:
   int*        m_pInt;
   wxString*   m_pString;
   wxArrayInt* m_pArrayInt;
+
+private:
+// Cannot use
+//  DECLARE_NO_COPY_CLASS(wxGenericValidator)
+// because copy constructor is explicitly declared above;
+// but no copy assignment operator is defined, so declare
+// it private to prevent the compiler from defining it:
+    wxGenericValidator& operator=(const wxGenericValidator&);
 };
 
 #endif
   // wxUSE_VALIDATORS
 
-#endif    
+#endif
   // _WX_VALGENH__