no real changes, just some cleanup

[wxWidgets.git] / include / wx / os2 / colour.h

diff --git a/include/wx/os2/colour.h b/include/wx/os2/colour.h
index b82c55433e3c0b653de840d908960ff411cf3b08..f0538eab36e6d8025ddd9e9eceb49982d3c390cb 100644 (file)
--- a/include/wx/os2/colour.h
+++ b/include/wx/os2/colour.h
@@ -1,5 +1,5 @@
 /////////////////////////////////////////////////////////////////////////////
 /////////////////////////////////////////////////////////////////////////////

-// Name:        colour.h
+// Name:        wx/os2/colour.h

 // Purpose:     wxColour class
 // Author:      David Webster
 // Modified by:
 // Purpose:     wxColour class
 // Author:      David Webster
 // Modified by:


@@ -15,111 +15,64 @@
 #include "wx/object.h"
 
 // Colour
 #include "wx/object.h"
 
 // Colour

-class WXDLLEXPORT wxColour: public wxObject
+class WXDLLEXPORT wxColour: public wxColourBase

 {
 public:
 {
 public:

-    //
-    // Ctors
-    //
+    // constructors
+    // ------------

 
 

-    //
-    // Default
-    //
+    // default

     wxColour();
     wxColour();

+    DEFINE_STD_WXCOLOUR_CONSTRUCTORS

 
 

-    //
-    // from RGB
-    //
-    wxColour( unsigned char cRed
-             ,unsigned char cGreen
-             ,unsigned char cBlue
-            );
-
-    //
-    // Implicit conversion from the colour name
-    //
-    wxColour(const wxString& rColourName) { InitFromName(rColourName); }
-    wxColour(const char* zColourName) { InitFromName(zColourName); }
-
-    //

     // Copy ctors and assignment operators
     // Copy ctors and assignment operators

-    //

     wxColour(const wxColour& rCol);
     wxColour(const wxColour* pCol);
     wxColour&operator = (const wxColour& rCol);
 
     wxColour(const wxColour& rCol);
     wxColour(const wxColour* pCol);
     wxColour&operator = (const wxColour& rCol);
 

-    //

     // Dtor
     // Dtor

-    //
-    ~wxColour();
-
-    //
-    // Set functions
-    //
-    void Set( unsigned char cRed
-             ,unsigned char cGreen
-             ,unsigned char cBlue
-            );
-    void Set(unsigned long lColRGB)
-    {
-        //
-        // We don't need to know sizeof(long) here because we assume that the three
-        // least significant bytes contain the R, G and B values
-        //
-        Set( (unsigned char)lColRGB
-            ,(unsigned char)(lColRGB >> 8)
-            ,(unsigned char)(lColRGB >> 16)
-           );
-    }
-    void Set(const wxString& rsColour)
-    {
-        InitFromName(rsColour);
-    }
+    virtual ~wxColour();

 
 

-    //

     // Accessors
     // Accessors

-    //
-    bool Ok(void) const {return m_bIsInit; }
-
-    //
-    // Let's remove this inelegant function
-    //
-#if WXWIN_COMPATIBILITY
-    void Get( unsigned char* pRed
-             ,unsigned char* pGreen
-             ,unsigned char* pBlue
-            ) const;
-#endif
+    bool Ok() const { return IsOk(); }
+    bool IsOk(void) const {return m_bIsInit; }

 
     unsigned char Red(void) const { return m_cRed; }
     unsigned char Green(void) const { return m_cGreen; }
     unsigned char Blue(void) const { return m_cBlue; }
 
 
     unsigned char Red(void) const { return m_cRed; }
     unsigned char Green(void) const { return m_cGreen; }
     unsigned char Blue(void) const { return m_cBlue; }
 

-    //

     // Comparison
     // Comparison

-    //

     bool operator == (const wxColour& rColour) const
     {
     bool operator == (const wxColour& rColour) const
     {

-        return (m_cRed   == rColour.m_cRed &&
-                m_cGreen == rColour.m_cGreen &&
-                m_cBlue  == rColour.m_cBlue
+        return (m_bIsInit == rColour.m_bIsInit
+                && m_cRed   == rColour.m_cRed
+                && m_cGreen == rColour.m_cGreen
+                && m_cBlue  == rColour.m_cBlue

                );
     }
                );
     }

+

     bool operator != (const wxColour& rColour) const { return !(*this == rColour); }
 
     bool operator != (const wxColour& rColour) const { return !(*this == rColour); }
 

-    void       InitFromName(const wxString& rCol);

     WXCOLORREF GetPixel(void) const { return m_vPixel; };
 
     WXCOLORREF GetPixel(void) const { return m_vPixel; };
 

+

 private:
 private:

-    bool                            m_bIsInit;
-    unsigned char                   m_cRed;
-    unsigned char                   m_cBlue;
-    unsigned char                   m_cGreen;
+
+    // Helper function
+    void Init();
+
+    bool           m_bIsInit;
+    unsigned char  m_cRed;
+    unsigned char  m_cBlue;
+    unsigned char  m_cGreen;
+
+    virtual void
+    InitRGBA(unsigned char r, unsigned char g, unsigned char b, unsigned char a);

 
 public:
     WXCOLORREF                      m_vPixel ;
 private:
 
 public:
     WXCOLORREF                      m_vPixel ;
 private:

-  DECLARE_DYNAMIC_CLASS(wxColour)
+    DECLARE_DYNAMIC_CLASS(wxColour)

 }; // end of class wxColour
 
 #endif
 }; // end of class wxColour
 
 #endif