]> git.saurik.com Git - wxWidgets.git/commitdiff
Document and test behaviour of wxRegion methods when it is invalid.
authorVadim Zeitlin <vadim@wxwidgets.org>
Tue, 18 Oct 2011 21:56:48 +0000 (21:56 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Tue, 18 Oct 2011 21:56:48 +0000 (21:56 +0000)
Document which wxRegion methods can and can't be used when the region itself
is invalid.

Apply the minor changes to wxGTK (Xor() didn't do the right thing, Offset()
didn't assert) and wxOSX (Offset() crashed) to make their behaviour consistent
with wxMSW.

Add a (trivial, so far, but to be extended later) wxRegion unit test checking
that the methods do indeed behave as documented.

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

14 files changed:
interface/wx/region.h
src/gtk/region.cpp
src/osx/carbon/region.cpp
tests/Makefile.in
tests/geometry/region.cpp [new file with mode: 0644]
tests/makefile.bcc
tests/makefile.gcc
tests/makefile.vc
tests/makefile.wat
tests/test.bkl
tests/test_test_gui.dsp
tests/test_vc7_test_gui.vcproj
tests/test_vc8_test_gui.vcproj
tests/test_vc9_test_gui.vcproj

index 7430c592e598ea6dd05470808edc83bc96b2b618..9685c82f24b12ef9f2a4252aec92827c74b34ab8 100644 (file)
@@ -157,6 +157,9 @@ class wxRegion : public wxGDIObject
 public:
     /**
         Default constructor.
+
+        This constructor creates an invalid, or null, object, i.e. calling
+        IsOk() on it returns @false and IsNull() returns @true.
     */
     wxRegion();
     /**
@@ -202,18 +205,26 @@ public:
 
     /**
         Clears the current region.
+
+        The object becomes invalid, or null, after being cleared.
     */
     virtual void Clear();
 
     /**
         Returns a value indicating whether the given point is contained within the region.
 
+        This method always returns @c wxOutRegion for an invalid region but
+        may, nevertheless, be safely called in this case.
+
         @return The return value is one of @c wxOutRegion and @c wxInRegion.
     */
     wxRegionContain Contains(wxCoord x, wxCoord y) const;
     /**
         Returns a value indicating whether the given point is contained within the region.
 
+        This method always returns @c wxOutRegion for an invalid region but
+        may, nevertheless, be safely called in this case.
+
         @return The return value is one of @c wxOutRegion and @c wxInRegion.
     */
     wxRegionContain Contains(const wxPoint& pt) const;
@@ -221,6 +232,9 @@ public:
         Returns a value indicating whether the given rectangle is contained within the
         region.
 
+        This method always returns @c wxOutRegion for an invalid region but
+        may, nevertheless, be safely called in this case.
+
         @return One of ::wxOutRegion, ::wxPartRegion or ::wxInRegion.
 
         @note On Windows, only ::wxOutRegion and ::wxInRegion are returned; a value
@@ -232,6 +246,9 @@ public:
         Returns a value indicating whether the given rectangle is contained within the
         region.
 
+        This method always returns @c wxOutRegion for an invalid region but
+        may, nevertheless, be safely called in this case.
+
         @return One of ::wxOutRegion, ::wxPartRegion or ::wxInRegion.
 
         @note On Windows, only ::wxOutRegion and ::wxInRegion are returned; a value
@@ -243,12 +260,16 @@ public:
     /**
         Convert the region to a black and white bitmap with the white pixels
         being inside the region.
+
+        This method can't be used for invalid region.
     */
     wxBitmap ConvertToBitmap() const;
 
     //@{
     /**
         Returns the outer bounds of the region.
+
+        This method returns 0-sized bounding box for invalid regions.
     */
     void GetBox(wxCoord& x, wxCoord& y, wxCoord& width,
                 wxCoord& height) const;
@@ -259,6 +280,9 @@ public:
         Finds the intersection of this region and another, rectangular region,
         specified using position and size.
 
+        This method always fails, i.e. returns @false, if this region is
+        invalid but may nevertheless be safely used even in this case.
+
         @return @true if successful, @false otherwise.
 
         @remarks Creates the intersection of the two regions, that is, the parts
@@ -270,6 +294,9 @@ public:
     /**
         Finds the intersection of this region and another, rectangular region.
 
+        This method always fails, i.e. returns @false, if this region is
+        invalid but may nevertheless be safely used even in this case.
+
         @return @true if successful, @false otherwise.
 
         @remarks Creates the intersection of the two regions, that is, the parts
@@ -280,6 +307,9 @@ public:
     /**
         Finds the intersection of this region and another region.
 
+        This method always fails, i.e. returns @false, if this region is
+        invalid but may nevertheless be safely used even in this case.
+
         @return @true if successful, @false otherwise.
 
         @remarks Creates the intersection of the two regions, that is, the parts
@@ -290,6 +320,8 @@ public:
 
     /**
         Returns @true if the region is empty, @false otherwise.
+
+        Always returns @true if the region is invalid.
     */
     virtual bool IsEmpty() const;
 
@@ -297,8 +329,8 @@ public:
         Returns @true if the region is equal to, i.e. covers the same area as,
         another one.
 
-        @note If both this region and @a region are invalid, they are
-              considered to be equal.
+        If both this region and @a region are both invalid, they are considered
+        to be equal.
     */
     bool IsEqual(const wxRegion& region) const;
 
@@ -307,6 +339,10 @@ public:
         Moves the region by the specified offsets in horizontal and vertical
         directions.
 
+        This method can't be called if the region is invalid as it doesn't make
+        sense to offset it then. Attempts to do it will result in assert
+        failure.
+
         @return @true if successful, @false otherwise (the region is unchanged
                  then).
     */
@@ -317,6 +353,9 @@ public:
     /**
         Subtracts a rectangular region from this region.
 
+        This method always fails, i.e. returns @false, if this region is
+        invalid but may nevertheless be safely used even in this case.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation combines the parts of 'this' region that are not
@@ -327,6 +366,9 @@ public:
     /**
         Subtracts a region from this region.
 
+        This method always fails, i.e. returns @false, if this region is
+        invalid but may nevertheless be safely used even in this case.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation combines the parts of 'this' region that are not
@@ -339,6 +381,10 @@ public:
         Finds the union of this region and another, rectangular region, specified using
         position and size.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given rectangle.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
@@ -349,6 +395,10 @@ public:
     /**
         Finds the union of this region and another, rectangular region.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given rectangle.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
@@ -359,6 +409,10 @@ public:
     /**
         Finds the union of this region and another region.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given @a region.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
@@ -396,6 +450,10 @@ public:
         Finds the Xor of this region and another, rectangular region, specified using
         position and size.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given rectangle.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
@@ -406,6 +464,10 @@ public:
     /**
         Finds the Xor of this region and another, rectangular region.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given rectangle.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
@@ -416,6 +478,10 @@ public:
     /**
         Finds the Xor of this region and another region.
 
+        This method can be used even if this region is invalid and has the
+        natural behaviour in this case, i.e. makes this region equal to the
+        given @a region.
+
         @return @true if successful, @false otherwise.
 
         @remarks This operation creates a region that combines all of this region
index 669dc09424f026927098f1c4f7a42d95a39c17d0..cf1d5d363bec152a98610d8ace9f75db54d30137 100644 (file)
@@ -183,16 +183,14 @@ bool wxRegion::DoUnionWithRegion( const wxRegion& region )
 
     if (!m_refData)
     {
-        m_refData = new wxRegionRefData();
-        M_REGIONDATA->m_region = gdk_region_new();
+        m_refData = new wxRegionRefData(*M_REGIONDATA_OF(region));
     }
     else
     {
         AllocExclusive();
+        gdk_region_union( M_REGIONDATA->m_region, region.GetRegion() );
     }
 
-    gdk_region_union( M_REGIONDATA->m_region, region.GetRegion() );
-
     return true;
 }
 
@@ -236,20 +234,23 @@ bool wxRegion::DoXor( const wxRegion& region )
 
     if (!m_refData)
     {
-        return false;
+        // XOR-ing with an invalid region is the same as XOR-ing with an empty
+        // one, i.e. it is simply a copy.
+        m_refData = new wxRegionRefData(*M_REGIONDATA_OF(region));
     }
+    else
+    {
+        AllocExclusive();
 
-    AllocExclusive();
-
-    gdk_region_xor( M_REGIONDATA->m_region, region.GetRegion() );
+        gdk_region_xor( M_REGIONDATA->m_region, region.GetRegion() );
+    }
 
     return true;
 }
 
 bool wxRegion::DoOffset( wxCoord x, wxCoord y )
 {
-    if (!m_refData)
-        return false;
+    wxCHECK_MSG( m_refData, false, wxS("invalid region") );
 
     AllocExclusive();
 
index ff12238509786e3bf88e01895b88d587fe175d5e..83a2e15e12c369776d1b5ae96ef50c750177f185 100644 (file)
@@ -178,7 +178,7 @@ void wxRegion::Clear()
 // Move the region
 bool wxRegion::DoOffset(wxCoord x, wxCoord y)
 {
-    wxCHECK_MSG( M_REGION, false, wxT("invalid wxRegion") );
+    wxCHECK_MSG( m_refData, false, wxT("invalid wxRegion") );
 
     if ( !x && !y )
         // nothing to do
index 1fda4594c764c14d10b74aaeaea96274be55c28c..83e0719b6dc1074cdda8eff723b646d097ebf971 100644 (file)
@@ -151,6 +151,7 @@ TEST_GUI_OBJECTS =  \
        test_gui_rect.o \
        test_gui_size.o \
        test_gui_point.o \
+       test_gui_region.o \
        test_gui_bitmap.o \
        test_gui_colour.o \
        test_gui_ellipsization.o \
@@ -705,6 +706,9 @@ test_gui_size.o: $(srcdir)/geometry/size.cpp $(TEST_GUI_ODEP)
 test_gui_point.o: $(srcdir)/geometry/point.cpp $(TEST_GUI_ODEP)
        $(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/geometry/point.cpp
 
+test_gui_region.o: $(srcdir)/geometry/region.cpp $(TEST_GUI_ODEP)
+       $(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/geometry/region.cpp
+
 test_gui_bitmap.o: $(srcdir)/graphics/bitmap.cpp $(TEST_GUI_ODEP)
        $(CXXC) -c -o $@ $(TEST_GUI_CXXFLAGS) $(srcdir)/graphics/bitmap.cpp
 
diff --git a/tests/geometry/region.cpp b/tests/geometry/region.cpp
new file mode 100644 (file)
index 0000000..f6ae795
--- /dev/null
@@ -0,0 +1,114 @@
+///////////////////////////////////////////////////////////////////////////////
+// Name:        tests/geometry/region.cpp
+// Purpose:     wxRegion unit test
+// Author:      Vadim Zeitlin
+// Created:     2011-10-12
+// RCS-ID:      $Id$
+// Copyright:   (c) 2011 Vadim Zeitlin <vadim@wxwidgets.org>
+///////////////////////////////////////////////////////////////////////////////
+
+// ----------------------------------------------------------------------------
+// headers
+// ----------------------------------------------------------------------------
+
+#include "testprec.h"
+
+#ifdef __BORLANDC__
+    #pragma hdrstop
+#endif
+
+#ifndef WX_PRECOMP
+    #include "wx/region.h"
+#endif // WX_PRECOMP
+
+#include "wx/iosfwrap.h"
+
+// ----------------------------------------------------------------------------
+// helper functions
+// ----------------------------------------------------------------------------
+
+namespace
+{
+
+// This function could be easily added to wxRegionIterator itself, where it
+// could be implemented far more efficiently as all major platforms store the
+// number of rectangles anyhow, but as we only use it for debugging purposes,
+// just keep it here for now.
+unsigned GetRectsCount(const wxRegion& rgn)
+{
+    unsigned count = 0;
+    for ( wxRegionIterator iter(rgn); iter.HaveRects(); ++iter )
+        count++;
+    return count;
+}
+
+} // anonymous namespace
+
+// this operator is needed to use CPPUNIT_ASSERT_EQUAL with wxRegions
+std::ostream& operator<<(std::ostream& os, const wxRegion& rgn)
+{
+    wxRect r = rgn.GetBox();
+    os << "# rects = " << GetRectsCount(rgn)
+       << "; bounding box {"
+       << r.x << ", " << r.y << ", " << r.width << ", " << r.height
+       << "}";
+    return os;
+}
+
+// ----------------------------------------------------------------------------
+// test class
+// ----------------------------------------------------------------------------
+
+class RegionTestCase : public CppUnit::TestCase
+{
+public:
+    RegionTestCase() { }
+
+private:
+    CPPUNIT_TEST_SUITE( RegionTestCase );
+        CPPUNIT_TEST( Validity );
+    CPPUNIT_TEST_SUITE_END();
+
+    void Validity();
+
+    wxDECLARE_NO_COPY_CLASS(RegionTestCase);
+};
+
+// register in the unnamed registry so that these tests are run by default
+CPPUNIT_TEST_SUITE_REGISTRATION( RegionTestCase );
+
+// also include in its own registry so that these tests can be run alone
+CPPUNIT_TEST_SUITE_NAMED_REGISTRATION( RegionTestCase, "RegionTestCase" );
+
+void RegionTestCase::Validity()
+{
+    wxRegion r;
+
+    CPPUNIT_ASSERT_MESSAGE
+    (
+        "Default constructed region must be invalid",
+        !r.IsOk()
+    );
+
+    CPPUNIT_ASSERT_MESSAGE
+    (
+        "Invalid region should be empty",
+        r.IsEmpty()
+    );
+
+    // Offsetting an invalid region doesn't make sense.
+    WX_ASSERT_FAILS_WITH_ASSERT( r.Offset(1, 1) );
+
+    CPPUNIT_ASSERT_MESSAGE
+    (
+        "Combining with a valid region should create a valid region",
+        r.Union(0, 0, 10, 10)
+    );
+
+    CPPUNIT_ASSERT_EQUAL_MESSAGE
+    (
+        "Union() with invalid region should give the same region",
+        wxRegion(0, 0, 10, 10),
+        r
+    );
+}
index 081c70a0cdb4fc6e06490af4f160e28c2c4a1541..56bd7d3dd3c48ac19db2feef4a0066c718740bb7 100644 (file)
@@ -136,6 +136,7 @@ TEST_GUI_OBJECTS =  \
        $(OBJS)\test_gui_rect.obj \\r
        $(OBJS)\test_gui_size.obj \\r
        $(OBJS)\test_gui_point.obj \\r
+       $(OBJS)\test_gui_region.obj \
        $(OBJS)\test_gui_bitmap.obj \\r
        $(OBJS)\test_gui_colour.obj \\r
        $(OBJS)\test_gui_ellipsization.obj \\r
@@ -746,6 +747,9 @@ $(OBJS)\test_gui_size.obj: .\geometry\size.cpp
 $(OBJS)\test_gui_point.obj: .\geometry\point.cpp\r
        $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\geometry\point.cpp\r
 \r
+$(OBJS)\test_gui_region.obj: .\geometry\region.cpp
+       $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\geometry\region.cpp
+
 $(OBJS)\test_gui_bitmap.obj: .\graphics\bitmap.cpp\r
        $(CXX) -q -c -P -o$@ $(TEST_GUI_CXXFLAGS) .\graphics\bitmap.cpp\r
 \r
index 1498b6a970bab1b299c1e9e9f0f5fd11d30036e7..b7baaccd35b043ed878828d758330caa7d7091ac 100644 (file)
@@ -129,6 +129,7 @@ TEST_GUI_OBJECTS =  \
        $(OBJS)\test_gui_rect.o \\r
        $(OBJS)\test_gui_size.o \\r
        $(OBJS)\test_gui_point.o \\r
+       $(OBJS)\test_gui_region.o \
        $(OBJS)\test_gui_bitmap.o \\r
        $(OBJS)\test_gui_colour.o \\r
        $(OBJS)\test_gui_ellipsization.o \\r
@@ -727,6 +728,9 @@ $(OBJS)\test_gui_size.o: ./geometry/size.cpp
 $(OBJS)\test_gui_point.o: ./geometry/point.cpp\r
        $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
 \r
+$(OBJS)\test_gui_region.o: ./geometry/region.cpp
+       $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<
+
 $(OBJS)\test_gui_bitmap.o: ./graphics/bitmap.cpp\r
        $(CXX) -c -o $@ $(TEST_GUI_CXXFLAGS) $(CPPDEPS) $<\r
 \r
index 0442379cb7ec712bcca474a7d0e25bbba42de516..198f67cf3a84883417f72a7306b08c782646212c 100644 (file)
@@ -131,6 +131,7 @@ TEST_GUI_OBJECTS =  \
        $(OBJS)\test_gui_rect.obj \\r
        $(OBJS)\test_gui_size.obj \\r
        $(OBJS)\test_gui_point.obj \\r
+       $(OBJS)\test_gui_region.obj \
        $(OBJS)\test_gui_bitmap.obj \\r
        $(OBJS)\test_gui_colour.obj \\r
        $(OBJS)\test_gui_ellipsization.obj \\r
@@ -872,6 +873,9 @@ $(OBJS)\test_gui_size.obj: .\geometry\size.cpp
 $(OBJS)\test_gui_point.obj: .\geometry\point.cpp\r
        $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\geometry\point.cpp\r
 \r
+$(OBJS)\test_gui_region.obj: .\geometry\region.cpp
+       $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\geometry\region.cpp
+
 $(OBJS)\test_gui_bitmap.obj: .\graphics\bitmap.cpp\r
        $(CXX) /c /nologo /TP /Fo$@ $(TEST_GUI_CXXFLAGS) .\graphics\bitmap.cpp\r
 \r
index af16e37d8a743a6a17966dcd9ecfb41c051cbc49..f5cebac92a9404864c215be9153ddf6ca3493db5 100644 (file)
@@ -380,6 +380,7 @@ TEST_GUI_OBJECTS =  &
        $(OBJS)\test_gui_rect.obj &\r
        $(OBJS)\test_gui_size.obj &\r
        $(OBJS)\test_gui_point.obj &\r
+       $(OBJS)\test_gui_region.obj &
        $(OBJS)\test_gui_bitmap.obj &\r
        $(OBJS)\test_gui_colour.obj &\r
        $(OBJS)\test_gui_ellipsization.obj &\r
@@ -787,6 +788,9 @@ $(OBJS)\test_gui_size.obj :  .AUTODEPEND .\geometry\size.cpp
 $(OBJS)\test_gui_point.obj :  .AUTODEPEND .\geometry\point.cpp\r
        $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
 \r
+$(OBJS)\test_gui_region.obj :  .AUTODEPEND .\geometry\region.cpp
+       $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<
+
 $(OBJS)\test_gui_bitmap.obj :  .AUTODEPEND .\graphics\bitmap.cpp\r
        $(CXX) -bt=nt -zq -fo=$^@ $(TEST_GUI_CXXFLAGS) $<\r
 \r
index a35147fddd442d44e4e7d657c587a76446c1d1f7..bec842d7f1de4c11890848c8b610ec382d7c0a8e 100644 (file)
             geometry/rect.cpp
             geometry/size.cpp
             geometry/point.cpp
+            geometry/region.cpp
             graphics/bitmap.cpp
             graphics/colour.cpp
             graphics/ellipsization.cpp
index baeb86696cb6de0b976d713b867107a13e15abe7..e8729ec1cef41dc37c94c23022495ee8c1d5fec4 100644 (file)
@@ -461,6 +461,10 @@ SOURCE=.\geometry\rect.cpp
 # End Source File\r
 # Begin Source File\r
 \r
+SOURCE=.\geometry\region.cpp
+# End Source File
+# Begin Source File
+
 SOURCE=.\controls\richtextctrltest.cpp\r
 # End Source File\r
 # Begin Source File\r
index e237709d403dff2c1e30d84c1f6f66020046a445..2b7ea74c7921b7a5b41ebf8334c05534a1e1d294 100644 (file)
                                RelativePath=".\geometry\rect.cpp">\r
                        </File>\r
                        <File\r
+                               RelativePath=".\geometry\region.cpp">
+                       </File>
+                       <File
                                RelativePath=".\controls\richtextctrltest.cpp">\r
                        </File>\r
                        <File\r
index ad7842ee33fe46b2447fa4229f7d61e20b57afa6..6d3ccbcf032b5f3ff0d7b644fe870b07d60bedd2 100644 (file)
                                >\r
                        </File>\r
                        <File\r
+                               RelativePath=".\geometry\region.cpp"
+                               >
+                       </File>
+                       <File
                                RelativePath=".\controls\richtextctrltest.cpp"\r
                                >\r
                        </File>\r
index 2871094105f9568e9c9e0f7a4bda3bd6cce44645..caa1aa5c8eaf7d28a4be2d245e07431eb7aeae3a 100644 (file)
                                >\r
                        </File>\r
                        <File\r
+                               RelativePath=".\geometry\region.cpp"
+                               >
+                       </File>
+                       <File
                                RelativePath=".\controls\richtextctrltest.cpp"\r
                                >\r
                        </File>\r