added wxConvFileName which is a variable, not define as wxConvFile; added comments...

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

diff --git a/include/wx/os2/colour.h b/include/wx/os2/colour.h
index b82c55433e3c0b653de840d908960ff411cf3b08..889a8ef798de35e833b743dcfd3fd9a3fea4aa3c 100644 (file)
--- a/include/wx/os2/colour.h
+++ b/include/wx/os2/colour.h
@@ -35,11 +35,13 @@ public:
              ,unsigned char cBlue
             );
 
              ,unsigned char cBlue
             );
 

+    wxColour( unsigned long colRGB ) { Set(colRGB); }
+

     //
     // Implicit conversion from the colour name
     //
     wxColour(const wxString& rColourName) { InitFromName(rColourName); }
     //
     // Implicit conversion from the colour name
     //
     wxColour(const wxString& rColourName) { InitFromName(rColourName); }

-    wxColour(const char* zColourName) { InitFromName(zColourName); }
+    wxColour(const wxChar *zColourName) { InitFromName(zColourName); }

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


@@ -81,16 +83,6 @@ public:
     //
     bool Ok(void) const {return m_bIsInit; }
 
     //
     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
-

     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; }


@@ -100,17 +92,24 @@ public:
     //
     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; };
 

+    void InitFromName(const wxString& rCol);
+

 private:
 private:

+
+    // Helper function
+    void Init();
+

     bool                            m_bIsInit;
     unsigned char                   m_cRed;
     unsigned char                   m_cBlue;
     bool                            m_bIsInit;
     unsigned char                   m_cRed;
     unsigned char                   m_cBlue;