some != NULL checks

[wxWidgets.git] / src / mac / region.cpp

diff --git a/src/mac/region.cpp b/src/mac/region.cpp
index c71d4c605a45a15004c399c66b2a1577e4c6e716..5c01ddd438b31c848d782db2e7d34e2c1d59fcc2 100644 (file)
--- a/src/mac/region.cpp
+++ b/src/mac/region.cpp
@@ -15,10 +15,8 @@
 #include "wx/region.h"
 #include "wx/gdicmn.h"
 
-#if !USE_SHARED_LIBRARY
        IMPLEMENT_DYNAMIC_CLASS(wxRegion, wxGDIObject)
        IMPLEMENT_DYNAMIC_CLASS(wxRegionIterator, wxObject)
-#endif
 
 //-----------------------------------------------------------------------------
 // wxRegionRefData implementation
@@ -28,19 +26,24 @@ class WXDLLEXPORT wxRegionRefData : public wxGDIRefData {
 public:
        wxRegionRefData()
        {
+               m_macRgn = NewRgn() ;
        }
 
        wxRegionRefData(const wxRegionRefData& data)
        {
-        // TODO
+               m_macRgn = NewRgn() ;
+        CopyRgn( data.m_macRgn , m_macRgn ) ;
        }
 
        ~wxRegionRefData()
        {
-        // TODO
+        DisposeRgn( m_macRgn ) ;
        }
+       RgnHandle       m_macRgn ;
 };
 
+#define M_REGION (((wxRegionRefData*)m_refData)->m_macRgn)
+#define OTHER_M_REGION(a) (((wxRegionRefData*)(a.m_refData))->m_macRgn)
 
 //-----------------------------------------------------------------------------
 // wxRegion
@@ -52,25 +55,30 @@ public:
 wxRegion::wxRegion()
 {
     m_refData = new wxRegionRefData;
-    // TODO create empty region
+}
+
+wxRegion::wxRegion(WXHRGN hRegion )
+{
+    m_refData = new wxRegionRefData;
+    CopyRgn( hRegion , M_REGION ) ;
 }
 
 wxRegion::wxRegion(long x, long y, long w, long h)
 {
     m_refData = new wxRegionRefData;
-    // TODO create rect region
+    SetRectRgn( M_REGION , x , y , x+w , y+h ) ;
 }
 
 wxRegion::wxRegion(const wxPoint& topLeft, const wxPoint& bottomRight)
 {
     m_refData = new wxRegionRefData;
-    // TODO create rect region
+    SetRectRgn( M_REGION , topLeft.x , topLeft.y , bottomRight.x , bottomRight.y ) ;
 }
 
 wxRegion::wxRegion(const wxRect& rect)
 {
     m_refData = new wxRegionRefData;
-    // TODO create rect region
+    SetRectRgn( M_REGION , rect.x , rect.y , rect.x+rect.width , rect.y+rect.height ) ;
 }
 
 /*!
@@ -95,41 +103,42 @@ void wxRegion::Clear()
 bool wxRegion::Combine(long x, long y, long width, long height, wxRegionOp op)
 {
        // Don't change shared data
-       if (!m_refData) {
+       if (!m_refData) 
+       {
                m_refData = new wxRegionRefData();
-       } else if (m_refData->GetRefCount() > 1) {
+       } 
+       else if (m_refData->GetRefCount() > 1) 
+       {
                wxRegionRefData* ref = (wxRegionRefData*)m_refData;
                UnRef();
                m_refData = new wxRegionRefData(*ref);
        }
-    // If ref count is 1, that means it's 'ours' anyway so no action.
-
-    // TODO create rect region
-
-    int mode = 0; // TODO platform-specific code
+    RgnHandle rgn = NewRgn() ;
+               SetRectRgn( rgn , x , y, x+width,y + height ) ;
+               
     switch (op)
     {
         case wxRGN_AND:
-            // TODO
+            SectRgn( M_REGION , rgn , M_REGION ) ;
             break ;
         case wxRGN_OR:
-            // TODO
+            UnionRgn( M_REGION , rgn , M_REGION ) ;
             break ;
         case wxRGN_XOR:
-            // TODO
+             XorRgn( M_REGION , rgn , M_REGION ) ;
             break ;
         case wxRGN_DIFF:
-            // TODO
+            DiffRgn( M_REGION , rgn , M_REGION ) ;
             break ;
         case wxRGN_COPY:
         default:
-            // TODO
+                               CopyRgn( rgn ,M_REGION ) ;
             break ;
     }
 
-    // TODO do combine region
+               DisposeRgn( rgn ) ;
 
-    return FALSE;
+    return TRUE;
 }
 
 //! Union /e region with this.
@@ -141,36 +150,35 @@ bool wxRegion::Combine(const wxRegion& region, wxRegionOp op)
        // Don't change shared data
        if (!m_refData) {
                m_refData = new wxRegionRefData();
-       } else  if (m_refData->GetRefCount() > 1) {
+       } 
+       else    if (m_refData->GetRefCount() > 1) 
+       {
                wxRegionRefData* ref = (wxRegionRefData*)m_refData;
                UnRef();
                m_refData = new wxRegionRefData(*ref);
        }
 
-    int mode = 0; // TODO platform-specific code
     switch (op)
     {
         case wxRGN_AND:
-            // TODO
+            SectRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
         case wxRGN_OR:
-            // TODO
+            UnionRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
         case wxRGN_XOR:
-            // TODO
+             XorRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
         case wxRGN_DIFF:
-            // TODO
+            DiffRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
         case wxRGN_COPY:
         default:
-            // TODO
+                               CopyRgn( OTHER_M_REGION(region) ,M_REGION ) ;
             break ;
     }
 
-    // TODO combine region
-
-       return FALSE;
+       return TRUE;
 }
 
 bool wxRegion::Combine(const wxRect& rect, wxRegionOp op)
@@ -185,9 +193,16 @@ bool wxRegion::Combine(const wxRect& rect, wxRegionOp op)
 // Outer bounds of region
 void wxRegion::GetBox(long& x, long& y, long&w, long &h) const
 {
-       if (m_refData) {
-        // TODO get box
-       } else {
+       if (m_refData) 
+       {
+               Rect box = (**M_REGION).rgnBBox ;
+        x = box.left ;
+        y = box.top ;
+        w = box.right - box.left ;
+        h = box.bottom - box.top ;
+       } 
+       else 
+       {
                x = y = w = h = 0;
        }
 }
@@ -202,8 +217,12 @@ wxRect wxRegion::GetBox() const
 // Is region empty?
 bool wxRegion::Empty() const
 {
-    // TODO
-    return FALSE;
+    return EmptyRgn( M_REGION ) ;
+}
+
+const WXHRGN wxRegion::GetWXHRGN() const
+{
+       return M_REGION ;
 }
 
 //-----------------------------------------------------------------------------
@@ -228,11 +247,11 @@ wxRegionContain wxRegion::Contains(const wxPoint& pt) const
        if (!m_refData)
                return wxOutRegion;
 
-    // TODO. Return wxInRegion if within region.
-    if (0)
+    Point p = { pt.y , pt.x } ;
+    if (PtInRgn( p , M_REGION ) )
         return wxInRegion;
-    else
-        return wxOutRegion;
+        
+    return wxOutRegion;
 }
 
 // Does the region contain the rectangle (x, y, w, h)?
@@ -241,8 +260,8 @@ wxRegionContain wxRegion::Contains(long x, long y, long w, long h) const
        if (!m_refData)
                return wxOutRegion;
 
-    // TODO. Return wxInRegion if within region.
-    if (0)
+    Rect rect = { y , x , y + h , x + w } ;
+    if (RectInRgn( &rect , M_REGION ) )
         return wxInRegion;
     else
         return wxOutRegion;
@@ -308,8 +327,14 @@ void wxRegionIterator::Reset(const wxRegion& region)
                m_numRects = 0;
        else
     {
-        // TODO create m_rects and fill with rectangles for this region
-        m_numRects = 0;
+       // we cannot dissolve it into rects on mac
+        m_rects = new wxRect[1];
+        Rect rect = (**OTHER_M_REGION( region )).rgnBBox ;
+        m_rects[0].x = rect.left;
+        m_rects[0].y = rect.top;
+        m_rects[0].width = rect.right - rect.left;
+        m_rects[0].height = rect.bottom - rect.top;
+        m_numRects = 1;
     }
 }