summaryrefslogtreecommitdiff
Side-by-side diff
Diffstat (more/less context) (ignore whitespace changes)
-rw-r--r--noncore/graphics/drawpad/filldrawmode.cpp68
-rw-r--r--noncore/graphics/drawpad/filldrawmode.h5
2 files changed, 25 insertions, 48 deletions
diff --git a/noncore/graphics/drawpad/filldrawmode.cpp b/noncore/graphics/drawpad/filldrawmode.cpp
index 1f81cd5..6ae0e58 100644
--- a/noncore/graphics/drawpad/filldrawmode.cpp
+++ b/noncore/graphics/drawpad/filldrawmode.cpp
@@ -39,9 +39,5 @@ void FillDrawMode::mousePressEvent(QMouseEvent* e)
if (m_oldRgb != m_fillRgb) {
- m_image.setPixel(x, y, m_fillRgb);
- fillEast(x + 1, y);
- fillSouth(x, y + 1);
- fillWest(x - 1, y);
- fillNorth(x, y - 1);
+ fillLine(x, y);
m_pDrawPadCanvas->currentPage()->convertFromImage(m_image);
@@ -60,48 +56,32 @@ void FillDrawMode::mouseMoveEvent(QMouseEvent* e)
}
-void FillDrawMode::fillEast(int x, int y)
+void FillDrawMode::fillLine(int x, int y)
{
- if (x < m_image.width()) {
+ if ((x >= 0) && (x < m_image.width()) && (y >= 0) && (y < m_image.height())) {
if (m_image.pixel(x, y) == m_oldRgb) {
- m_image.setPixel(x, y, m_fillRgb);
- fillEast(x + 1, y);
- fillSouth(x, y + 1);
- fillNorth(x, y - 1);
- }
- }
-}
+ int x1, x2;
-void FillDrawMode::fillSouth(int x, int y)
-{
- if (y < m_image.height()) {
- if (m_image.pixel(x, y) == m_oldRgb) {
- m_image.setPixel(x, y, m_fillRgb);
- fillEast(x + 1, y);
- fillSouth(x, y + 1);
- fillWest(x - 1, y);
- }
- }
-}
+ x1 = x - 1;
+ x2 = x + 1;
-void FillDrawMode::fillWest(int x, int y)
-{
- if (x >= 0) {
- if (m_image.pixel(x, y) == m_oldRgb) {
- m_image.setPixel(x, y, m_fillRgb);
- fillSouth(x, y + 1);
- fillWest(x - 1, y);
- fillNorth(x, y - 1);
- }
- }
-}
+ while ((x1 >= 0) && (m_image.pixel(x1, y) == m_oldRgb)) {
+ x1--;
+ }
-void FillDrawMode::fillNorth(int x, int y)
-{
- if (y >= 0) {
- if (m_image.pixel(x, y) == m_oldRgb) {
- m_image.setPixel(x, y, m_fillRgb);
- fillEast(x + 1, y);
- fillWest(x - 1, y);
- fillNorth(x, y - 1);
+ while ((x2 < m_image.width()) && (m_image.pixel(x2, y) == m_oldRgb)) {
+ x2++;
+ }
+
+ for (int i = x1 + 1; i < x2; i++) {
+ m_image.setPixel(i, y, m_fillRgb);
+ }
+
+ for (int i = x1 + 1; i < x2; i++) {
+ fillLine(i, y - 1);
+ }
+
+ for (int i = x1 + 1; i < x2; i++) {
+ fillLine(i, y + 1);
+ }
}
}
diff --git a/noncore/graphics/drawpad/filldrawmode.h b/noncore/graphics/drawpad/filldrawmode.h
index 1b7e7b5..604f9c7 100644
--- a/noncore/graphics/drawpad/filldrawmode.h
+++ b/noncore/graphics/drawpad/filldrawmode.h
@@ -30,8 +30,5 @@ public:
private:
- void fillEast(int x, int y);
- void fillSouth(int x, int y);
- void fillWest(int x, int y);
- void fillNorth(int x, int y);
+ void fillLine(int x, int y);
QImage m_image;