Stefan Brüns : d2d1: Use the TYPE_END vertex type when last/first vertices coincide.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: b92b6c29298805abd18a3c7ea0f7905099af572d
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=b92b6c29298805abd18a3c7ea...

Author: Stefan Brüns <stefan.bruens(a)rwth-aachen.de>
Date:   Tue Jan 11 16:28:16 2022 +0100

d2d1: Use the TYPE_END vertex type when last/first vertices coincide.

When the last vertex is coincident with the first vertex, the last
segment should be suppressed for both END_OPEN and END_CLOSED.
Only when last and first vertex are not coincident the additional
line segment may be added - always for intersection tests and
similar, and for stroking operations when the figure is CLOSED.

Trying to use an zero-length segment in d2d_geometry_intersect_self()
will create invalid segments, causing infinite loops later.

Instead of reducing the vertex_count for coincident first/last
vertices add a dedicated type. This is required as some operations
need the last segment, others do not.

This also allows to remove some replicated code in
StrokeContains()/GetBounds()/Simplify(), as a last Bézier segment
is always processed in the regular loop.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=51139
Signed-off-by: Stefan Brüns <stefan.bruens(a)rwth-aachen.de>
Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/d2d1/geometry.c   | 105 +++++++++++++++----------------------------------
 dlls/d2d1/tests/d2d1.c |  16 +++++++-
 2 files changed, 46 insertions(+), 75 deletions(-)

Diff:   https://source.winehq.org/git/wine.git/?a=commitdiff;h=b92b6c29298805abd18a3...