RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-25 06:17:35 +00:00
Code Issues Activity Actions

LibGfx: Make Painter::fill_path() take Path by const reference

Browse source 
Taking a mutable reference here made the API look very strange.
This commit is contained in:
Andreas Kling 2021-09-17 13:41:03 +02:00
parent 8c863ad959
commit 32b9d80ee5
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
Userland/Libraries/LibGfx/Painter.cpp
View file

@ -2005,11 +2005,11 @@ void Painter::stroke_path(const Path& path, Color color, int thickness)
from.set_x(previous_to.value().x()); from.set_x(previous_to.value().x());
} }
void Painter::fill_path(Path& path, Color color, WindingRule winding_rule) void Painter::fill_path(Path const& path, Color color, WindingRule winding_rule)
{ {
VERIFY(scale() == 1); // FIXME: Add scaling support. VERIFY(scale() == 1); // FIXME: Add scaling support.
const auto& segments = path.split_lines(); auto const& segments = path.split_lines();
if (segments.size() == 0) if (segments.size() == 0)
return; return;

2
Userland/Libraries/LibGfx/Painter.h
View file

@ -97,7 +97,7 @@ public:
Nonzero, Nonzero,
EvenOdd, EvenOdd,
}; };
void fill_path(Path&, Color, WindingRule rule = WindingRule::Nonzero); void fill_path(Path const&, Color, WindingRule rule = WindingRule::Nonzero);
const Font& font() const { return *state().font; } const Font& font() const { return *state().font; }
void set_font(const Font& font) { state().font = &font; } void set_font(const Font& font) { state().font = &font; }