<div dir="ltr"><div>From dd1b2a40e37a4569ec6baf8785e0f04ebaf05d83 Mon Sep 17 00:00:00 2001</div><div>From: Elton Chung <<a href="mailto:elton%2Bwine@elton.tk">elton+wine@elton.tk</a>></div><div>Date: Sat, 6 Jul 2013 18:25:18 +0800</div>

<div>Subject: [PATCH] gdiplus: fixing identical sub-expressions</div><div><br></div><div>There are identical sub-expressions '!(types[i + 1] &</div><div> PathPointTypeBezier)' to the left and to the right of the '||' operator.</div>

<div><br></div><div>Credits to PVS-Studio:</div><div><a href="http://www.viva64.com/en/b/0192/#ID0EYFAG">http://www.viva64.com/en/b/0192/#ID0EYFAG</a></div><div><br></div><div>Solution given by Vincent Povirk.</div><div>
---</div>
<div> dlls/gdiplus/graphics.c | 2 +-</div><div> 1 file changed, 1 insertion(+), 1 deletion(-)</div><div><br></div><div>diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c</div><div>index dcdc29e..89bb4f5 100644</div>

<div>--- a/dlls/gdiplus/graphics.c</div><div>+++ b/dlls/gdiplus/graphics.c</div><div>@@ -1748,7 +1748,7 @@ static GpStatus draw_poly(GpGraphics *graphics, GpPen *pen, GDIPCONST GpPointF *</div><div>     for(i = 1; i < count; i++){</div>

<div>         if((types[i] & PathPointTypePathTypeMask) == PathPointTypeBezier){</div><div>             if((i + 2 >= count) || !(types[i + 1] & PathPointTypeBezier)</div><div>-                || !(types[i + 1] & PathPointTypeBezier)){</div>

<div>+                || !(types[i + 2] & PathPointTypeBezier)){</div><div>                 ERR("Bad bezier points\n");</div><div>                 goto end;</div><div>             }</div><div>-- </div><div>
1.8.1.msysgit.1</div>
<div><br></div></div>