X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/2b5f62a0b2db198609b45dec622a018dae37008e..e4b712e2455dfc6e5303c428119b435d8d6182ed:/include/wx/msw/pen.h?ds=sidebyside diff --git a/include/wx/msw/pen.h b/include/wx/msw/pen.h index a653d39db0..07325f94a4 100644 --- a/include/wx/msw/pen.h +++ b/include/wx/msw/pen.h @@ -18,6 +18,7 @@ #include "wx/gdiobj.h" #include "wx/bitmap.h" +#include "wx/colour.h" typedef WXDWORD wxMSWDash; @@ -41,9 +42,18 @@ protected: wxDash * m_dash ; wxColour m_colour; WXHPEN m_hPen; + +private: +// Cannot use +// DECLARE_NO_COPY_CLASS(wxPenRefData) +// 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: + wxPenRefData& operator=(const wxPenRefData&); }; #define M_PENDATA ((wxPenRefData *)m_refData) +#define wxPENDATA(x) ((wxPenRefData *)(x).m_refData) // Pen class WXDLLEXPORT wxPen: public wxGDIObject @@ -57,8 +67,28 @@ public: ~wxPen(); inline wxPen& operator = (const wxPen& pen) { if (*this == pen) return (*this); Ref(pen); return *this; } - inline bool operator == (const wxPen& pen) const { return m_refData == pen.m_refData; } - inline bool operator != (const wxPen& pen) const { return m_refData != pen.m_refData; } + inline bool operator == (const wxPen& pen) const + { + // It is impossible to know if the user dashes have changed, + // so we must assume that they have + if ( m_refData && pen.m_refData ) + { + if ( M_PENDATA->m_nbDash != 0 || wxPENDATA(pen)->m_nbDash != 0 ) + return false; + } + return m_refData == pen.m_refData; + } + inline bool operator != (const wxPen& pen) const + { + // It is impossible to know if the user dashes have changed, + // so we must assume that they have + if ( m_refData && pen.m_refData ) + { + if ( M_PENDATA->m_nbDash != 0 || wxPENDATA(pen)->m_nbDash != 0 ) + return true; + } + return m_refData != pen.m_refData; + } virtual bool Ok() const { return (m_refData != NULL) ; }