Fixed infinite loop bug in C# code (550).

JimBobSquarePants · JimBobSquarePants · commit f57336ae056d · 2023-08-14T11:23:55.000+10:00
diff --git a/src/ImageSharp.Drawing/Shapes/PolygonClipper/PolygonClipper.cs b/src/ImageSharp.Drawing/Shapes/PolygonClipper/PolygonClipper.cs
@@ -333,13 +333,9 @@ private void ConvertHorzSegsToJoins()
                 for (int j = i + 1; j < k; j++)
                 {
                     HorzSegment hs2 = this.horzSegList[j];
-                    if (hs2.LeftOp.Point.X >= hs1.RightOp.Point.X)
-                    {
-                        break;
-                    }
-
-                    if (hs2.LeftToRight == hs1.LeftToRight ||
-                      (hs2.RightOp.Point.X <= hs1.LeftOp.Point.X))
+                    if ((hs2.LeftOp.Point.X >= hs1.RightOp.Point.X) ||
+                        (hs2.LeftToRight == hs1.LeftToRight) ||
+                        (hs2.RightOp.Point.X <= hs1.LeftOp.Point.X))
                     {
                         continue;
                     }
@@ -674,7 +670,7 @@ private void DoHorizontal(Active horz)
 
             if (IsHotEdge(horz))
             {
-                AddOutPt(horz, horz.Top);
+                this.AddToHorzSegList(AddOutPt(horz, horz.Top));
             }
 
             this.UpdateEdgeIntoAEL(horz); // this is the end of an intermediate horiz.

Original file line number	Diff line number	Diff line change
`@@ -333,13 +333,9 @@ private void ConvertHorzSegsToJoins()`
`333`	`333`	`for (int j = i + 1; j < k; j++)`
`334`	`334`	`{`
`335`	`335`	`HorzSegment hs2 = this.horzSegList[j];`
`336`		`- if (hs2.LeftOp.Point.X >= hs1.RightOp.Point.X)`
`337`		`- {`
`338`		`- break;`
`339`		`- }`
`340`		`-`
`341`		`- if (hs2.LeftToRight == hs1.LeftToRight \|\|`
`342`		`- (hs2.RightOp.Point.X <= hs1.LeftOp.Point.X))`
	`336`	`+ if ((hs2.LeftOp.Point.X >= hs1.RightOp.Point.X) \|\|`
	`337`	`+ (hs2.LeftToRight == hs1.LeftToRight) \|\|`
	`338`	`+ (hs2.RightOp.Point.X <= hs1.LeftOp.Point.X))`
`343`	`339`	`{`
`344`	`340`	`continue;`
`345`	`341`	`}`
`@@ -674,7 +670,7 @@ private void DoHorizontal(Active horz)`
`674`	`670`
`675`	`671`	`if (IsHotEdge(horz))`
`676`	`672`	`{`
`677`		`- AddOutPt(horz, horz.Top);`
	`673`	`+ this.AddToHorzSegList(AddOutPt(horz, horz.Top));`
`678`	`674`	`}`
`679`	`675`
`680`	`676`	`this.UpdateEdgeIntoAEL(horz); // this is the end of an intermediate horiz.`