RGBCube/serenity
1
Fork 0
mirror of https://github.com/RGBCube/serenity synced 2025-07-26 10:27:34 +00:00
Code Issues Activity Actions

LibGfx: Ensure last subpath is closed by Path::close_all_subpaths()

Browse source
This commit is contained in:
MacDue 2023-06-24 16:51:17 +01:00 • committed by Andreas Kling
parent 6b40271b95
commit 3d755a57b6
1 changed files with 16 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

25
Userland/Libraries/LibGfx/Path.cpp
View file

@ -144,20 +144,24 @@ void Path::close_all_subpaths()
Optional<FloatPoint> cursor, start_of_subpath; Optional<FloatPoint> cursor, start_of_subpath;
bool is_first_point_in_subpath { false }; bool is_first_point_in_subpath { false };
auto close_previous_subpath = [&] {
if (cursor.has_value() && !is_first_point_in_subpath) {
// This is a move from a subpath to another
// connect the two ends of this subpath before
// moving on to the next one
VERIFY(start_of_subpath.has_value());
append_segment<MoveSegment>(cursor.value());
append_segment<LineSegment>(start_of_subpath.value());
}
};
auto segment_count = m_segments.size(); auto segment_count = m_segments.size();
for (size_t i = 0; i < segment_count; i++) { for (size_t i = 0; i < segment_count; i++) {
// Note: We need to use m_segments[i] as append_segment() may invalidate any references. // Note: We need to use m_segments[i] as append_segment() may invalidate any references.
switch (m_segments[i]->type()) { switch (m_segments[i]->type()) {
case Segment::Type::MoveTo: { case Segment::Type::MoveTo: {
if (cursor.has_value() && !is_first_point_in_subpath) { close_previous_subpath();
// This is a move from a subpath to another
// connect the two ends of this subpath before
// moving on to the next one
VERIFY(start_of_subpath.has_value());
append_segment<MoveSegment>(cursor.value());
append_segment<LineSegment>(start_of_subpath.value());
}
is_first_point_in_subpath = true; is_first_point_in_subpath = true;
cursor = m_segments[i]->point(); cursor = m_segments[i]->point();
break; break;
@ -177,6 +181,9 @@ void Path::close_all_subpaths()
break; break;
} }
} }
if (m_segments.last()->type() != Segment::Type::MoveTo)
close_previous_subpath();
} }
DeprecatedString Path::to_deprecated_string() const DeprecatedString Path::to_deprecated_string() const