X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/dcb44466c13262aad45c82f13373069dac8ba932..28d52eed36a2b4ee6ea36b3c5d4861b3ea292074:/include/wx/gdicmn.h?ds=sidebyside

diff --git a/include/wx/gdicmn.h b/include/wx/gdicmn.h
index a94d0ee228..a3f585fd2d 100644
--- a/include/wx/gdicmn.h
+++ b/include/wx/gdicmn.h
@@ -149,7 +149,9 @@ enum wxStockCursor
 class WXDLLEXPORT wxSize
 {
 public:
-    // members are public for compatibility
+    // members are public for compatibility (don't use them directly,
+    // especially that there names were chosen very unfortunately - they should
+    // have been called width and height)
     long x;
     long y;
 
@@ -158,15 +160,21 @@ public:
     wxSize(long xx, long yy) { Set(xx, yy); }
 
     // no copy ctor or assignment operator - the defaults are ok
-
     bool operator==(const wxSize& sz) const { return x == sz.x && y == sz.y; }
 
     // FIXME are these really useful? If they're, we should have += &c as well
     wxSize operator+(const wxSize& sz) { return wxSize(x + sz.x, y + sz.y); }
     wxSize operator-(const wxSize& sz) { return wxSize(x - sz.x, y - sz.y); }
 
+    // accessors
     void Set(long xx, long yy) { x = xx; y = yy; }
+    void SetWidth(long w) { x = w; }
+    void SetHeight(long h) { y = h; }
+
+    long GetWidth() const { return x; }
+    long GetHeight() const { return y; }
 
+    // compatibility
     long GetX() const { return x; }
     long GetY() const { return y; }
 };
@@ -258,13 +266,14 @@ public:
     long GetBottom() const { return y + height; }
     long GetRight()  const { return x + width; }
 
-    bool Inside(int, int) const;
 
     bool operator==(const wxRect& rect) const;
     bool operator!=(const wxRect& rect) const { return !(*this == rect); }
 
+    bool Inside(int cx, int cy) const;
     wxRect operator + (const wxRect& rect) const;
     const wxRect& operator += (const wxRect& rect);
+    
 public:
     long x, y, width, height;
 };