]> git.saurik.com Git - wxWidgets.git/commitdiff
cleaned up the mess caused by FloodFill patch
authorVáclav Slavík <vslavik@fastmail.fm>
Sun, 7 Apr 2002 23:07:17 +0000 (23:07 +0000)
committerVáclav Slavík <vslavik@fastmail.fm>
Sun, 7 Apr 2002 23:07:17 +0000 (23:07 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@15011 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/common/imagfill.cpp
src/mac/carbon/dc.cpp
src/mac/dc.cpp
src/mgl/dc.cpp
src/motif/dcclient.cpp
src/x11/dcclient.cpp

index d337301e7bb1b38fbadc7e2aa6e6d1b655cbb345..81f49d8471d532a2058e163fb2530f4e2599b7fb 100644 (file)
@@ -7,8 +7,6 @@
 // Licence:     wxWindows licence
 /////////////////////////////////////////////////////////////////////////////
 
-#ifdef __GNUG__
-#endif
 
 // For compilers that support precompilation, includes "wx.h".
 #include "wx/wxprec.h"
     #pragma hdrstop
 #endif
 
-#if wxUSE_IMAGE
+#include "wx/defs.h"
+
+#if wxUSE_IMAGE && !defined(__WXMSW__)
+// we have no use for this code in wxMSW...
 
 #include "wx/image.h"
 
 #ifndef WX_PRECOMP
     #include "wx/brush.h"
+    #include "wx/dc.h"
+    #include "wx/dcmemory.h"
 #endif
 
 // DoFloodFill
 // a color different from the start pixel is reached (wxFLOOD_SURFACE)
 // or fill color is reached (wxFLOOD_BORDER)
 
-bool wxImage::MatchPixel(int x, int y, int w, int h, const wxColour & c)
+static LINKAGEMODE bool MatchPixel(wxImage *img, int x, int y, int w, int h, const wxColour& c)
 {
     if ((x<0)||(x>=w)||(y<0)||(y>=h)) return FALSE;
 
-    unsigned char r = GetRed(x,y);
-    unsigned char g = GetGreen(x,y);
-    unsigned char b = GetBlue(x,y);
+    unsigned char r = img->GetRed(x,y);
+    unsigned char g = img->GetGreen(x,y);
+    unsigned char b = img->GetBlue(x,y);
     return c.Red() == r && c.Green() == g && c.Blue() == b ;
 }
 
-bool wxImage::MatchBoundaryPixel(int x, int y, int w, int h, const wxColour & fill, const wxColour & bound)
+static LINKAGEMODE bool MatchBoundaryPixel(wxImage *img, int x, int y, int w, int h, const wxColour & fill, const wxColour& bound)
 {
     if ((x<0)||(x>=w)||(y<0)||(y>=h)) return TRUE;
 
-    unsigned char r = GetRed(x,y);
-    unsigned char g = GetGreen(x,y);
-    unsigned char b = GetBlue(x,y);
-    if ( fill.Red() == r && fill.Green() == g && fill.Blue() == b ) return TRUE;
-    if ( bound.Red() == r && bound.Green() == g && bound.Blue() == b ) return TRUE;
-    return FALSE ;
+    unsigned char r = img->GetRed(x,y);
+    unsigned char g = img->GetGreen(x,y);
+    unsigned char b = img->GetBlue(x,y);
+    if ( fill.Red() == r && fill.Green() == g && fill.Blue() == b ) 
+        return TRUE;
+    if ( bound.Red() == r && bound.Green() == g && bound.Blue() == b ) 
+        return TRUE;
+    return FALSE;
 }
 
 
-void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
-        const wxColour& testColour, int style /*=wxFLOOD_SURFACE */,
-        int LogicalFunction /*= wxCOPY, currently unused */)
+static LINKAGEMODE
+void wxImageFloodFill(wxImage *image,
+                      wxCoord x, wxCoord y, const wxBrush & fillBrush,
+                      const wxColour& testColour, int style,
+                      int LogicalFunction)
 {
     /* A diamond flood-fill using a circular queue system.
     Each pixel surrounding the current pixel is added to
     the queue if it meets the criteria, then is retrieved in
     its turn.  Code originally based on http://www.drawit.co.nz/Developers.htm */
 
-    int width = GetWidth();
-    int height = GetHeight();
+    int width = image->GetWidth();
+    int height = image->GetHeight();
 
     //Draw using a pen made from the current brush colour
     //Potentially allows us to use patterned flood fills in future code
@@ -76,9 +83,9 @@ void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
     if (style == wxFLOOD_SURFACE)
     {
        //if wxFLOOD_SURFACE, if fill colour is same as required, we don't do anything
-       if (     GetRed(x,y)   != r
-             || GetGreen(x,y) != g
-             || GetBlue (x,y) != b   )
+       if (     image->GetRed(x,y)   != r
+             || image->GetGreen(x,y) != g
+             || image->GetBlue (x,y) != b   )
         {
         //prepare memory for queue
         //queue save, start, read
@@ -101,55 +108,55 @@ void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
         *qs=yt=y;
         qs++;
 
-        SetRGB(xt,yt,r,g,b);
+        image->SetRGB(xt,yt,r,g,b);
 
         //Main queue loop
         while(qr!=qs)
         {
             //Add new members to queue
             //Above current pixel
-            if(MatchPixel(xt,yt-1,width,height,testColour))
+            if(MatchPixel(image,xt,yt-1,width,height,testColour))
             {
                 *qs=xt;
                 qs++;
                 *qs=yt-1;
                 qs++;
-                SetRGB(xt,yt-1,r,g,b);
+                image->SetRGB(xt,yt-1,r,g,b);
 
                 //Loop back to beginning of queue
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Below current pixel
-            if(MatchPixel(xt,yt+1,width,height,testColour))
+            if(MatchPixel(image,xt,yt+1,width,height,testColour))
             {
                 *qs=xt;
                 qs++;
                 *qs=yt+1;
                 qs++;
-                SetRGB(xt,yt+1,r,g,b);
+                image->SetRGB(xt,yt+1,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Left of current pixel
-            if(MatchPixel(xt-1,yt,width,height,testColour))
+            if(MatchPixel(image,xt-1,yt,width,height,testColour))
             {
                 *qs=xt-1;
                 qs++;
                 *qs=yt;
                 qs++;
-                SetRGB(xt-1,yt,r,g,b);
+                image->SetRGB(xt-1,yt,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Right of current pixel
-            if(MatchPixel(xt+1,yt,width,height,testColour))
+            if(MatchPixel(image,xt+1,yt,width,height,testColour))
             {
                 *qs=xt+1;
                 qs++;
                 *qs=yt;
                 qs++;
-                SetRGB(xt+1,yt,r,g,b);
+                image->SetRGB(xt+1,yt,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
@@ -164,17 +171,17 @@ void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
         //Go Back to beginning of loop
         }
 
-        delete [] qst ;
+        delete[] qst;
         }
     }
     else
     {
     //style is wxFLOOD_BORDER
     // fill up to testColor border - if already testColour don't do anything
-    if (  GetRed(x,y)   != testColour.Red()
-       || GetGreen(x,y) != testColour.Green()
-       || GetBlue(x,y)  != testColour.Blue()   )
-        {
+    if (  image->GetRed(x,y)   != testColour.Red()
+          || image->GetGreen(x,y) != testColour.Green()
+          || image->GetBlue(x,y)  != testColour.Blue() )
+    {
         //prepare memory for queue
         //queue save, start, read
         size_t *qs, *qst, *qr;
@@ -196,55 +203,55 @@ void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
         *qs=yt=y;
         qs++;
 
-        SetRGB(xt,yt,r,g,b);
+        image->SetRGB(xt,yt,r,g,b);
 
         //Main queue loop
-        while(qr!=qs)
+        while (qr!=qs)
         {
             //Add new members to queue
             //Above current pixel
-            if(!MatchBoundaryPixel(xt,yt-1,width,height,fillColour,testColour))
+            if(!MatchBoundaryPixel(image,xt,yt-1,width,height,fillColour,testColour))
             {
                 *qs=xt;
                 qs++;
                 *qs=yt-1;
                 qs++;
-                SetRGB(xt,yt-1,r,g,b);
+                image->SetRGB(xt,yt-1,r,g,b);
 
                 //Loop back to beginning of queue
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Below current pixel
-            if(!MatchBoundaryPixel(xt,yt+1,width,height,fillColour,testColour))
+            if(!MatchBoundaryPixel(image,xt,yt+1,width,height,fillColour,testColour))
             {
                 *qs=xt;
                 qs++;
                 *qs=yt+1;
                 qs++;
-                SetRGB(xt,yt+1,r,g,b);
+                image->SetRGB(xt,yt+1,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Left of current pixel
-            if(!MatchBoundaryPixel(xt-1,yt,width,height,fillColour,testColour))
+            if(!MatchBoundaryPixel(image,xt-1,yt,width,height,fillColour,testColour))
             {
                 *qs=xt-1;
                 qs++;
                 *qs=yt;
                 qs++;
-                SetRGB(xt-1,yt,r,g,b);
+                image->SetRGB(xt-1,yt,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
             //Right of current pixel
-            if(!MatchBoundaryPixel(xt+1,yt,width,height,fillColour,testColour))
+            if(!MatchBoundaryPixel(image,xt+1,yt,width,height,fillColour,testColour))
             {
                 *qs=xt+1;
                 qs++;
                 *qs=yt;
                 qs++;
-                SetRGB(xt+1,yt,r,g,b);
+                image->SetRGB(xt+1,yt,r,g,b);
                 if(qs>=(qst+qSz)) qs=qst;
             }
 
@@ -259,11 +266,41 @@ void wxImage::DoFloodFill (wxCoord x, wxCoord y, const wxBrush & fillBrush,
         //Go Back to beginning of loop
         }
 
-        delete [] qst ;
+        delete[] qst;
         }
     }
     //all done,
 }
 
+
+void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                   const wxColour& col, int style)
+{
+    if (dc->GetBrush().GetStyle() == wxTRANSPARENT)
+        return;
+
+    int height = 0;
+    int width  = 0;
+    dc->GetSize(&width, &height);
+
+    //it would be nice to fail if we don't get a sensible size...
+    wxCHECK_RET(width >= 1 && height >= 1, wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
+
+    //this is much faster than doing the individual pixels
+    wxMemoryDC memdc;
+    wxBitmap bitmap(width, height);
+    memdc.SelectObject(bitmap);
+    memdc.Blit(0, 0, width, height, dc, 0, 0);
+    memdc.SelectObject(wxNullBitmap);
+
+    wxImage image = bitmap.ConvertToImage();
+    wxImageFloodFill(&image, x,y, dc->GetBrush(), col, style, 
+                     dc->GetLogicalFunction());
+    bitmap = wxBitmap(image);
+    memdc.SelectObject(bitmap);
+    dc->Blit(0, 0, width, height, &memdc, 0, 0);
+    memdc.SelectObject(wxNullBitmap);
+}
+
 #endif // wxUSE_IMAGE
 
index 4a044f288767aa003ab74be04885dff6cf38fb2c..5778b90b072740d901e9e6591795d84405ec4b8d 100644 (file)
@@ -649,40 +649,15 @@ void  wxDC::SetLogicalFunction( int function )
        m_macPenInstalled = false ;
 }
 
-void  wxDC::DoFloodFill( wxCoord x, wxCoord y, const wxColour& col,
-                             int style )
-{
-    if (GetBrush().GetStyle() == wxTRANSPARENT)
-    {
-        wxLogDebug(wxT("In FloodFill, Current Brush is transparent, no filling done"));
-        return ;
-    }
-    int height = 0;
-    int width  = 0;
-    this->GetSize(&width, &height);
-    //it would be nice to fail if we don't get a sensible size...
-    if (width < 1 || height < 1)
-    {
-        wxLogError(wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
-        return ;
-    }
-
-    //this is much faster than doing the individual pixels
-    wxMemoryDC memdc;
-    wxBitmap bitmap(width, height);
-    memdc.SelectObject(bitmap);
-    memdc.Blit(0, 0, width, height, (wxDC*) this, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+extern void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                          const wxColour & col, int style);
 
-    wxImage image = bitmap.ConvertToImage() ;
-    image.DoFloodFill (x,y, GetBrush(), col, style, GetLogicalFunction());
-    bitmap = wxBitmap(image);
-    memdc.SelectObject(bitmap);
-    this->Blit(0, 0, width, height, &memdc, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+void wxDC::DoFloodFill(wxCoord x, wxCoord y,
+                       const wxColour& col, int style)
+{
+    wxDoFloodFill(this, x, y, col, style);
 }
 
-
 bool  wxDC::DoGetPixel( wxCoord x, wxCoord y, wxColour *col ) const 
 {
     wxCHECK_MSG( Ok(), false, wxT("wxDC::DoGetPixel  Invalid DC") );
index 4a044f288767aa003ab74be04885dff6cf38fb2c..5778b90b072740d901e9e6591795d84405ec4b8d 100644 (file)
@@ -649,40 +649,15 @@ void  wxDC::SetLogicalFunction( int function )
        m_macPenInstalled = false ;
 }
 
-void  wxDC::DoFloodFill( wxCoord x, wxCoord y, const wxColour& col,
-                             int style )
-{
-    if (GetBrush().GetStyle() == wxTRANSPARENT)
-    {
-        wxLogDebug(wxT("In FloodFill, Current Brush is transparent, no filling done"));
-        return ;
-    }
-    int height = 0;
-    int width  = 0;
-    this->GetSize(&width, &height);
-    //it would be nice to fail if we don't get a sensible size...
-    if (width < 1 || height < 1)
-    {
-        wxLogError(wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
-        return ;
-    }
-
-    //this is much faster than doing the individual pixels
-    wxMemoryDC memdc;
-    wxBitmap bitmap(width, height);
-    memdc.SelectObject(bitmap);
-    memdc.Blit(0, 0, width, height, (wxDC*) this, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+extern void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                          const wxColour & col, int style);
 
-    wxImage image = bitmap.ConvertToImage() ;
-    image.DoFloodFill (x,y, GetBrush(), col, style, GetLogicalFunction());
-    bitmap = wxBitmap(image);
-    memdc.SelectObject(bitmap);
-    this->Blit(0, 0, width, height, &memdc, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+void wxDC::DoFloodFill(wxCoord x, wxCoord y,
+                       const wxColour& col, int style)
+{
+    wxDoFloodFill(this, x, y, col, style);
 }
 
-
 bool  wxDC::DoGetPixel( wxCoord x, wxCoord y, wxColour *col ) const 
 {
     wxCHECK_MSG( Ok(), false, wxT("wxDC::DoGetPixel  Invalid DC") );
index 4a2a4909ec10484ae491edc7eda9827bf55cd51d..2409fcaaa211ab4911c91b8e98bbc8bc64e84b61 100644 (file)
@@ -332,36 +332,13 @@ void wxDC::Clear()
     }
 }
 
-void wxDC::DoFloodFill(wxCoord x, wxCoord y, const wxColour& col, int style)
-{
-    if (GetBrush().GetStyle() == wxTRANSPARENT)
-    {
-        wxLogDebug(wxT("In FloodFill, Current Brush is transparent, no filling done"));
-        return ;
-    }
-    int height = 0;
-    int width  = 0;
-    this->GetSize(&width, &height);
-    //it would be nice to fail if we don't get a sensible size...
-    if (width < 1 || height < 1)
-    {
-        wxLogError(wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
-        return ;
-    }
+extern void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                          const wxColour & col, int style);
 
-    //this is much faster than doing the individual pixels
-    wxMemoryDC memdc;
-    wxBitmap bitmap(width, height);
-    memdc.SelectObject(bitmap);
-    memdc.Blit(0, 0, width, height, (wxDC*) this, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
-
-    wxImage image(bitmap);
-    image.DoFloodFill (x,y, GetBrush(), col, style, GetLogicalFunction());
-    bitmap = wxBitmap(image);
-    memdc.SelectObject(bitmap);
-    this->Blit(0, 0, width, height, &memdc, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+void wxDC::DoFloodFill(wxCoord x, wxCoord y,
+                       const wxColour& col, int style)
+{
+    wxDoFloodFill(this, x, y, col, style);
 }
 
 bool wxDC::DoGetPixel(wxCoord x, wxCoord y, wxColour *col) const
index 7b126fc446fa34a9555da5e04354446bbf8e1287..a5e57af9d1c9acd096899e55473bbba50c8f8bf8 100644 (file)
@@ -231,37 +231,13 @@ wxWindowDC::~wxWindowDC()
     m_userRegion = (WXRegion) 0;
 }
 
-void wxWindowDC::DoFloodFill( wxCoord x1, wxCoord y1,
-                             const wxColour& col, int style )
-{
-    if (GetBrush().GetStyle() == wxTRANSPARENT)
-    {
-        wxLogDebug(wxT("In FloodFill, current brush is transparent, no filling done"));
-        return ;
-    }
-    int height = 0;
-    int width  = 0;
-    this->GetSize(&width, &height);
-    //it would be nice to fail if we don't get a sensible size...
-    if (width < 1 || height < 1)
-    {
-        wxLogError(wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
-        return ;
-    }
+extern void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                          const wxColour & col, int style);
 
-    //this is much faster than doing the individual pixels
-    wxMemoryDC memdc;
-    wxBitmap bitmap(width, height);
-    memdc.SelectObject(bitmap);
-    memdc.Blit(0, 0, width, height, (wxDC*) this, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
-
-    wxImage image(bitmap);
-    image.DoFloodFill (x1,y1, GetBrush(), col, style, GetLogicalFunction());
-    bitmap = wxBitmap(image);
-    memdc.SelectObject(bitmap);
-    this->Blit(0, 0, width, height, &memdc, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+void wxWindowDC::DoFloodFill(wxCoord x, wxCoord y,
+                             const wxColour& col, int style)
+{
+    wxDoFloodFill(this, x, y, col, style);
 }
   
 bool wxWindowDC::DoGetPixel( wxCoord x1, wxCoord y1, wxColour *col ) const
index 13cfe8ad9ab98006b61d618f9acf9211ff468775..5ea0c08e591c5951106e5a6a90a622e826f0ceae 100644 (file)
@@ -313,37 +313,13 @@ void wxWindowDC::DoGetSize( int* width, int* height ) const
     m_owner->GetSize(width, height);
 }
 
-void wxWindowDC::DoFloodFill( wxCoord x, wxCoord y,
-                           const wxColour & col, int style )
-{
-    if (GetBrush().GetStyle() == wxTRANSPARENT)
-    {
-        wxLogDebug(wxT("In FloodFill, current brush is transparent, no filling done"));
-        return ;
-    }
-    int height = 0;
-    int width  = 0;
-    this->GetSize(&width, &height);
-    //it would be nice to fail if we don't get a sensible size...
-    if (width < 1 || height < 1)
-    {
-        wxLogError(wxT("In FloodFill, dc.GetSize routine failed, method not supported by this DC"));
-        return ;
-    }
+extern void wxDoFloodFill(wxDC *dc, wxCoord x, wxCoord y, 
+                          const wxColour & col, int style);
 
-    //this is much faster than doing the individual pixels
-    wxMemoryDC memdc;
-    wxBitmap bitmap(width, height);
-    memdc.SelectObject(bitmap);
-    memdc.Blit(0, 0, width, height, (wxDC*) this, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
-
-    wxImage image(bitmap);
-    image.DoFloodFill (x,y, GetBrush(), col, style, GetLogicalFunction());
-    bitmap = wxBitmap(image);
-    memdc.SelectObject(bitmap);
-    this->Blit(0, 0, width, height, &memdc, 0, 0);
-    memdc.SelectObject(wxNullBitmap);
+void wxWindowDC::DoFloodFill(wxCoord x, wxCoord y,
+                             const wxColour& col, int style)
+{
+    wxDoFloodFill(this, x, y, col, style);
 }
 
 bool wxWindowDC::DoGetPixel( wxCoord x1, wxCoord y1, wxColour *col ) const