fixed bug with the caret positioning after SetValue() introduced by the last commit

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

diff --git a/include/wx/msw/colour.h b/include/wx/msw/colour.h
index c4632bde4d3a69cd7b6ba5e592b03a6d1bc03cb7..56ef566d8585c1c629674291e8d9c5681d04469b 100644 (file)
--- a/include/wx/msw/colour.h
+++ b/include/wx/msw/colour.h
@@ -5,67 +5,92 @@
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$
 // Modified by:
 // Created:     01/02/97
 // RCS-ID:      $Id$

-// Copyright:   (c) Julian Smart and Markus Holzem
-// Licence:    wxWindows licence
+// Copyright:   (c) Julian Smart
+// Licence:     wxWindows licence

 /////////////////////////////////////////////////////////////////////////////
 
 /////////////////////////////////////////////////////////////////////////////
 

-#ifndef __COLOURH__
-#define __COLOURH__
+#ifndef _WX_COLOUR_H_
+#define _WX_COLOUR_H_

 
 #ifdef __GNUG__
 #pragma interface "colour.h"
 #endif
 
 #ifdef __GNUG__
 #pragma interface "colour.h"
 #endif

+#include "wx/object.h"

 
 // Colour
 class WXDLLEXPORT wxColour: public wxObject
 {
 
 // Colour
 class WXDLLEXPORT wxColour: public wxObject
 {

-  DECLARE_DYNAMIC_CLASS(wxColour)

 public:
 public:

-  wxColour(void);
-  wxColour(const unsigned char r, const unsigned char g, const unsigned char b);
-  wxColour(unsigned long colRGB) { Set(colRGB); }
-  wxColour(const wxColour& col);
-  wxColour(const wxString& col);
-  ~wxColour(void) ;
-  wxColour& operator =(const wxColour& src) ;
-  wxColour& operator =(const wxString& src) ;
-  inline int Ok(void) const { return (m_isInit) ; }
-
-  void Set(unsigned char r, unsigned char g, unsigned char b);
-  void Set(unsigned long colRGB) 
-  { 
+  // ctors
+    // default
+  wxColour();
+    // from RGB
+  wxColour( unsigned char red, unsigned char green, unsigned char blue );
+  wxColour( unsigned long colRGB ) { Set(colRGB); }
+
+    // implicit conversion from the colour name
+  wxColour( const wxString &colourName ) { InitFromName(colourName); }
+  wxColour( const wxChar *colourName ) { InitFromName(colourName); }
+
+
+    // copy ctors and assignment operators
+  wxColour( const wxColour& col );
+  wxColour& operator = ( const wxColour& col );
+
+    // dtor
+  ~wxColour();
+
+  // Set() functions
+  void Set( unsigned char red, unsigned char green, unsigned char blue );
+  void Set(unsigned long colRGB)
+  {

     // 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
     // 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)colRGB, 
+    Set((unsigned char)colRGB,

         (unsigned char)(colRGB >> 8),
         (unsigned char)(colRGB >> 8),

-        (unsigned char)(colRGB >> 16)); 
+        (unsigned char)(colRGB >> 16));

   }
 
   }
 

+  // accessors
+  bool Ok() const {return m_isInit; }
+

   // Let's remove this inelegant function
 #if WXWIN_COMPATIBILITY
   void Get(unsigned char *r, unsigned char *g, unsigned char *b) const;
 #endif
 
   // Let's remove this inelegant function
 #if WXWIN_COMPATIBILITY
   void Get(unsigned char *r, unsigned char *g, unsigned char *b) const;
 #endif
 

-  inline unsigned char Red(void) const { return m_red; }
-  inline unsigned char Green(void) const { return m_green; }
-  inline unsigned char Blue(void) const { return m_blue; }
+  unsigned char Red() const { return m_red; }
+  unsigned char Green() const { return m_green; }
+  unsigned char Blue() const { return m_blue; }

 
 

-  inline bool operator == (const wxColour& colour) { return (m_red == colour.m_red && m_green == colour.m_green && m_blue == colour.m_blue); }
+  // comparison
+  bool operator==(const wxColour& colour) const
+  {
+    return m_isInit == colour.m_isInit &&
+           m_red == colour.m_red &&
+           m_green == colour.m_green &&
+           m_blue == colour.m_blue;
+  }
+
+  bool operator != (const wxColour& colour) const { return !(*this == colour); }

 
 

-  inline bool operator != (const wxColour& colour) { return (!(m_red == colour.m_red && m_green == colour.m_green && m_blue == colour.m_blue)); }
+  WXCOLORREF GetPixel() const { return m_pixel; };

 
 

-  WXCOLORREF GetPixel(void) const { return m_pixel; };
+public:
+  WXCOLORREF m_pixel;

 
 

- private:
-  bool                         m_isInit;
+private:
+  bool          m_isInit;

   unsigned char m_red;
   unsigned char m_blue;
   unsigned char m_green;
   unsigned char m_red;
   unsigned char m_blue;
   unsigned char m_green;

- public:
-  WXCOLORREF m_pixel ;
-};

 
 

-#define wxColor wxColour
+  // helper func
+  void InitFromName(const wxString& colourName);
+
+private:
+  DECLARE_DYNAMIC_CLASS(wxColour)
+};

 
 #endif
 
 #endif

-       // __COLOURH__
+        // _WX_COLOUR_H_