cleanup - reformat

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@36781 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/mac/carbon/region.cpp

@@ -21,12 +21,11 @@ IMPLEMENT_DYNAMIC_CLASS(wxRegionIterator, wxObject)
 // wxRegionRefData implementation
 //-----------------------------------------------------------------------------
 
-class WXDLLEXPORT wxRegionRefData : public wxGDIRefData {
+class WXDLLEXPORT wxRegionRefData : public wxGDIRefData
+{
 public:
     wxRegionRefData()
-    {
-        m_macRgn = NewRgn() ;
-    }
+    { m_macRgn = NewRgn(); }
 
     wxRegionRefData(const wxRegionRefData& data)
         : wxGDIRefData()
@@ -36,9 +35,8 @@ public:
     }
 
     ~wxRegionRefData()
-    {
-        DisposeRgn( m_macRgn ) ;
-    }
+    { DisposeRgn( m_macRgn ); }
+
     RgnHandle    m_macRgn;
 };
 
@@ -95,7 +93,7 @@ wxRegion::wxRegion(size_t n, const wxPoint *points, int WXUNUSED(fillStyle))
     Rect destRect = { 0, 0, 1, 1 };
 
     ::GetGWorld( &oldWorld, &oldGDHandle );
-    err = ::NewGWorld( &gWorld, 32, &destRect, nil, nil, 0 );
+    err = ::NewGWorld( &gWorld, 32, &destRect, NULL, NULL, 0 );
     if ( err == noErr )
     {
         ::SetGWorld( gWorld, GetGDevice() );
@@ -105,6 +103,7 @@ wxRegion::wxRegion(size_t n, const wxPoint *points, int WXUNUSED(fillStyle))
         wxCoord x1, x2 , y1 , y2 ;
         x2 = x1 = points[0].x ;
         y2 = y1 = points[0].y ;
+
         ::MoveTo( x1, y1 );
         for (size_t i = 1; i < n; i++)
         {
@@ -112,19 +111,17 @@ wxRegion::wxRegion(size_t n, const wxPoint *points, int WXUNUSED(fillStyle))
             y2 = points[i].y ;
             ::LineTo( x2, y2 );
         }
+
         // close the polyline if necessary
         if ( x1 != x2 || y1 != y2 )
-        {
             ::LineTo( x1, y1 ) ;
-        }
+
         CloseRgn( M_REGION ) ;
+
         ::SetGWorld( oldWorld, oldGDHandle );
     }
 }
 
-/*!
- * Destroy the region.
- */
 wxRegion::~wxRegion()
 {
     // m_refData unrefed in ~wxObject
@@ -146,12 +143,11 @@ bool wxRegion::Offset(wxCoord x, wxCoord y)
     wxCHECK_MSG( M_REGION, false, _T("invalid wxRegion") );
 
     if ( !x && !y )
-    {
         // nothing to do
         return true;
-    }
 
     OffsetRgn( M_REGION , x , y ) ;
+
     return true ;
 }
 
@@ -170,6 +166,7 @@ bool wxRegion::Combine(long x, long y, long width, long height, wxRegionOp op)
         UnRef();
         m_refData = new wxRegionRefData( *ref );
     }
+
     RgnHandle rgn = NewRgn() ;
     SetRectRgn( rgn , x , y, x + width, y + height ) ;
 
@@ -178,15 +175,19 @@ bool wxRegion::Combine(long x, long y, long width, long height, wxRegionOp op)
         case wxRGN_AND:
             SectRgn( M_REGION , rgn , M_REGION ) ;
             break ;
+
         case wxRGN_OR:
             UnionRgn( M_REGION , rgn , M_REGION ) ;
             break ;
+
         case wxRGN_XOR:
              XorRgn( M_REGION , rgn , M_REGION ) ;
             break ;
+
         case wxRGN_DIFF:
             DiffRgn( M_REGION , rgn , M_REGION ) ;
             break ;
+
         case wxRGN_COPY:
         default:
             CopyRgn( rgn , M_REGION ) ;
@@ -195,17 +196,18 @@ bool wxRegion::Combine(long x, long y, long width, long height, wxRegionOp op)
 
     DisposeRgn( rgn ) ;
 
-    return TRUE;
+    return true;
 }
 
 //! Union /e region with this.
 bool wxRegion::Combine(const wxRegion& region, wxRegionOp op)
 {
     if (region.Empty())
-        return FALSE;
+        return false;
 
     // Don't change shared data
-    if (!m_refData) {
+    if (!m_refData)
+    {
         m_refData = new wxRegionRefData();
     }
     else if (m_refData->GetRefCount() > 1)
@@ -220,22 +222,26 @@ bool wxRegion::Combine(const wxRegion& region, wxRegionOp op)
         case wxRGN_AND:
             SectRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
+
         case wxRGN_OR:
             UnionRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
+
         case wxRGN_XOR:
              XorRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
+
         case wxRGN_DIFF:
             DiffRgn( M_REGION , OTHER_M_REGION(region) , M_REGION ) ;
             break ;
+
         case wxRGN_COPY:
         default:
             CopyRgn( OTHER_M_REGION(region) , M_REGION ) ;
             break ;
     }
 
-    return TRUE;
+    return true;
 }
 
 bool wxRegion::Combine(const wxRect& rect, wxRegionOp op)
@@ -269,6 +275,7 @@ wxRect wxRegion::GetBox() const
 {
     wxCoord x, y, w, h;
     GetBox(x, y, w, h);
+
     return wxRect(x, y, w, h);
 }
 
@@ -294,12 +301,13 @@ wxRegionContain wxRegion::Contains(long x, long y) const
         return wxOutRegion;
 
     // TODO. Return wxInRegion if within region.
-    if (0)
-        return wxInRegion;
+//    if (0)
+//        return wxInRegion;
+
     return wxOutRegion;
 }
 
-// Does the region contain the point pt?
+// Does the region contain the point?
 wxRegionContain wxRegion::Contains(const wxPoint& pt) const
 {
     if (!m_refData)
@@ -332,10 +340,12 @@ wxRegionContain wxRegion::Contains(const wxRect& rect) const
         return wxOutRegion;
 
     long x, y, w, h;
+
     x = rect.x;
     y = rect.y;
     w = rect.GetWidth();
     h = rect.GetHeight();
+
     return Contains(x, y, w, h);
 }
 
@@ -355,7 +365,8 @@ wxRegionIterator::wxRegionIterator()
 
 wxRegionIterator::~wxRegionIterator()
 {
-    if (m_rects) {
+    if (m_rects)
+    {
         delete [] m_rects;
         m_rects = NULL;
     }
@@ -374,6 +385,7 @@ wxRegionIterator& wxRegionIterator::operator=(const wxRegionIterator& iterator)
 {
     m_current  = iterator.m_current;
     SetRects(iterator.m_numRects, iterator.m_rects);
+
     return *this;
 }
 
@@ -382,17 +394,21 @@ wxRegionIterator& wxRegionIterator::operator=(const wxRegionIterator& iterator)
  */
 void wxRegionIterator::SetRects(long numRects, wxRect *rects)
 {
-    if (m_rects) {
+    if (m_rects)
+    {
         delete [] m_rects;
         m_rects = NULL;
     }
-    if (rects)
+
+    if (rects && (numRects > 0))
     {
         int i;
+
         m_rects = new wxRect[numRects];
         for (i = 0; i < numRects; i++)
             m_rects[i] = rects[i];
     }
+
     m_numRects = numRects;
 }
 
@@ -422,6 +438,7 @@ OSStatus wxMacRegionToRectsCounterCallback (
     {
         (*m_numRects) += 1 ;
     }
+
     return noErr;
 }
 
@@ -440,6 +457,7 @@ OSStatus wxMacRegionToRectsSetterCallback (
         RegionToRectsCallbackData *cb = (RegionToRectsCallbackData*) data ;
         cb->m_rects[cb->m_current++] = wxRect( rect->left , rect->top , rect->right - rect->left , rect->bottom - rect->top ) ;
     }
+
     return noErr;
 }
 
@@ -448,13 +466,16 @@ void wxRegionIterator::Reset(const wxRegion& region)
     m_current = 0;
     m_region = region;
 
-    if (m_rects) {
+    if (m_rects)
+    {
         delete [] m_rects;
         m_rects = NULL;
     }
 
     if (m_region.Empty())
+    {
         m_numRects = 0;
+    }
     else
     {
         RegionToRectsUPP proc = NewRegionToRectsUPP( wxMacRegionToRectsCounterCallback );
@@ -475,6 +496,7 @@ void wxRegionIterator::Reset(const wxRegion& region)
         {
             m_numRects = 0;
         }
+
         DisposeRegionToRectsUPP( proc );
     }
 }
@@ -487,6 +509,7 @@ wxRegionIterator& wxRegionIterator::operator ++ ()
 {
     if (m_current < m_numRects)
         ++m_current;
+    
     return *this;
 }
 
@@ -508,6 +531,7 @@ long wxRegionIterator::GetX() const
 {
     if (m_current < m_numRects)
         return m_rects[m_current].x;
+
     return 0;
 }
 
@@ -515,6 +539,7 @@ long wxRegionIterator::GetY() const
 {
     if (m_current < m_numRects)
         return m_rects[m_current].y;
+
     return 0;
 }
 
@@ -522,6 +547,7 @@ long wxRegionIterator::GetW() const
 {
     if (m_current < m_numRects)
         return m_rects[m_current].width ;
+
     return 0;
 }
 
@@ -529,6 +555,7 @@ long wxRegionIterator::GetH() const
 {
     if (m_current < m_numRects)
         return m_rects[m_current].height;
+
     return 0;
 }