Fixed a long-standing issue where wxSlider controls with a hardcoded size would mispl...

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

diff --git a/src/x11/region.cpp b/src/x11/region.cpp
index 6ed3767c1147e1fbc3ccf550c0fa1e0d55d983da..78be8597084ab2b73f918d2c64f0cf95438d0fba 100644 (file)
--- a/src/x11/region.cpp
+++ b/src/x11/region.cpp
@@ -1,14 +1,14 @@
 /////////////////////////////////////////////////////////////////////////////
 // File:      region.cpp
 // Purpose:   Region class
 /////////////////////////////////////////////////////////////////////////////
 // File:      region.cpp
 // Purpose:   Region class

-// Author:    Markus Holzem, Julian Smart, Robert Roebling
+// Author:    Julian Smart, Robert Roebling

 // Created:   Fri Oct 24 10:46:34 MET 1997
 // Created:   Fri Oct 24 10:46:34 MET 1997

-// RCS-ID:      $Id$
-// Copyright: (c) 1997 Markus Holzem, Julian Smart, Robert Roebling
+// RCS-ID:    $Id$
+// Copyright: (c) 1997 Julian Smart, Robert Roebling

 // Licence:   wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
 // Licence:   wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 

-#ifdef __GNUG__
+#if defined(__GNUG__) && !defined(NO_GCC_PRAGMA)

 #pragma implementation "region.h"
 #endif
 
 #pragma implementation "region.h"
 #endif
 


@@ -149,6 +149,12 @@ void wxRegion::Clear()
 
 bool wxRegion::Union( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
 {
 
 bool wxRegion::Union( wxCoord x, wxCoord y, wxCoord width, wxCoord height )
 {

+    // work around for XUnionRectWithRegion() bug: taking a union with an empty
+    // rect results in an empty region (at least XFree 3.3.6 and 4.0 have this
+    // problem)
+    if ( !width || !height )
+        return TRUE;
+

     XRectangle rect;
     rect.x = x;
     rect.y = y;
     XRectangle rect;
     rect.x = x;
     rect.y = y;


@@ -434,13 +440,13 @@ public:
 
 wxRIRefData::~wxRIRefData()
 {
 
 wxRIRefData::~wxRIRefData()
 {

-    delete m_rects;
+    delete [] m_rects;

 }
 
 void wxRIRefData::CreateRects( const wxRegion& region )
 {
     if (m_rects)
 }
 
 void wxRIRefData::CreateRects( const wxRegion& region )
 {
     if (m_rects)

-      delete m_rects;
+      delete [] m_rects;

 
     m_rects = 0;
     m_numRects = 0;
 
     m_rects = 0;
     m_numRects = 0;


@@ -450,6 +456,16 @@ void wxRIRefData::CreateRects( const wxRegion& region )
     Region r = (Region) region.GetX11Region();
     if (r)
     {
     Region r = (Region) region.GetX11Region();
     if (r)
     {

+#if wxUSE_NANOX
+        GR_RECT rect;
+        GrGetRegionBox(r, & rect);
+        m_numRects = 1;
+        m_rects = new wxRect[1];
+        m_rects[0].x = rect.x;
+        m_rects[0].y = rect.y;
+        m_rects[0].width = rect.width;
+        m_rects[0].height = rect.height;
+#else

         m_numRects = r->numRects;
         if (m_numRects)
         {
         m_numRects = r->numRects;
         if (m_numRects)
         {


@@ -464,6 +480,7 @@ void wxRIRefData::CreateRects( const wxRegion& region )
                 wr.height = xr.y2-xr.y1;
             }
         }
                 wr.height = xr.y2-xr.y1;
             }
         }

+#endif

     }
 }
 
     }
 }