LibGfx: Fix clipping in fill_ellipse

fill_ellipse used to clip the bounding box, so instead of drawing a
part of an ellipse it drew a smaller ellipse. This commit fixes this
behaviour.

1 files changed, 4 insertions, 8 deletions

diff --git a/Userland/Libraries/LibGfx/Painter.cpp b/Userland/Libraries/LibGfx/Painter.cpp
index ff4e31f603..5ec78c6e01 100644
--- a/Userland/Libraries/LibGfx/Painter.cpp
+++ b/Userland/Libraries/LibGfx/Painter.cpp
@@ -255,14 +255,10 @@ void Painter::fill_ellipse(const IntRect& a_rect, Color color)
 
     VERIFY(m_target->rect().contains(rect));
 
-    RGBA32* dst = m_target->scanline(rect.top()) + rect.left() + rect.width() / 2;
-    const size_t dst_skip = m_target->pitch() / sizeof(RGBA32);
-
-    for (int i = 0; i < rect.height(); i++) {
-        double y = rect.height() * 0.5 - i;
-        double x = rect.width() * sqrt(0.25 - y * y / rect.height() / rect.height());
-        fast_u32_fill(dst - (int)x, color.value(), 2 * (int)x);
-        dst += dst_skip;
+    for (int i = 1; i < a_rect.height(); i++) {
+        double y = a_rect.height() * 0.5 - i;
+        double x = a_rect.width() * sqrt(0.25 - y * y / a_rect.height() / a_rect.height());
+        draw_line({ a_rect.x() + a_rect.width() / 2 - (int)x, a_rect.y() + i }, { a_rect.x() + a_rect.width() / 2 + (int)x - 1, a_rect.y() + i }, color);
     }
 }