don't leave two conscutive separators after "moving" a standard menu item to the...

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

diff --git a/include/wx/valgen.h b/include/wx/valgen.h
index 1dbcdb62ecb527db508df3795955f2c879226874..4b165afab43ce42f8eecc78c163a908c79d34ebe 100644 (file)
--- a/include/wx/valgen.h
+++ b/include/wx/valgen.h
@@ -12,7 +12,7 @@
 #ifndef _WX_VALGENH__
 #define _WX_VALGENH__
 
-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)
 #pragma interface "valgen.h"
 #endif
 
@@ -22,6 +22,7 @@
 
 class WXDLLEXPORT wxGenericValidator: public wxValidator
 {
+DECLARE_CLASS(wxGenericValidator)
 public:
   wxGenericValidator(bool* val);
   wxGenericValidator(int* val);
@@ -40,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();
@@ -55,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__