[PATCH] d2d1: Silence SetSegmentFlags FIXME when setting default value

[Stefan Brüns]

Some software (Buhl Tax and variants) repeatedly calls SetSegmentFlags
with D2D1_PATH_SEGMENT_NONE, which is just the default value and has no
effect (unless the flags where already changed for the current segment,
which still reports a FIXME).

Signed-off-by: Stefan Brüns 
---
 dlls/d2d1/geometry.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/d2d1/geometry.c b/dlls/d2d1/geometry.c
index a7074899fda..ac5f1019f3f 100644
--- a/dlls/d2d1/geometry.c
+++ b/dlls/d2d1/geometry.c
@@ -2773,6 +2773,8 @@ static void STDMETHODCALLTYPE d2d_geometry_sink_SetFillMode(ID2D1GeometrySink *i

static void STDMETHODCALLTYPE d2d_geometry_sink_SetSegmentFlags(ID2D1GeometrySink *iface, D2D1_PATH_SEGMENT flags)
 {
+    if (flags == D2D1_PATH_SEGMENT_NONE)
+	return;
     FIXME("iface %p, flags %#x stub!\n", iface, flags);
 }

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=105141

Your paranoid android.

=== debian11 (64 bit WoW report) ===

d2d1:
d2d1.c:6688: Test failed: d2d1.c:9776: Test marked todo: Test 3: Got unexpected hr 0x88990029.