Implement FillStrokePath() operator and filling rule.

src/UglyToad.PdfPig.Tests/Graphics/TestOperationContext.cs

@@ -69,8 +69,13 @@
         public void StrokePath(bool close)
         {
         }
-        public void FillPath(bool close)
+
+        public void FillPath(FillingRule fillingRule, bool close)
         {
+        }
+        public void FillStrokePath(FillingRule fillingRule, bool close)
+        {
+
         }
 
         public void ClosePath()

src/UglyToad.PdfPig/Graphics/ContentStreamProcessor.cs

@@ -17,6 +17,7 @@
     using Tokenization.Scanner;
     using Tokens;
     using XObjects;
+    using static UglyToad.PdfPig.Core.PdfSubpath;
 
     internal class ContentStreamProcessor : IOperationContext
     {
@@ -94,7 +95,14 @@
             this.pageContentParser = pageContentParser ?? throw new ArgumentNullException(nameof(pageContentParser));
             this.filterProvider = filterProvider ?? throw new ArgumentNullException(nameof(filterProvider));
             this.log = log;
-            graphicsStack.Push(new CurrentGraphicsState());
+
+            // initiate CurrentClippingPath to cropBox
+            var clippingSubpath = new PdfSubpath();
+            clippingSubpath.Rectangle(cropBox.BottomLeft.X, cropBox.BottomLeft.Y, cropBox.Width, cropBox.Height);
+            var clippingPath = new PdfPath() { clippingSubpath };
+            clippingPath.SetClipping(FillingRule.NonZeroWinding);
+
+            graphicsStack.Push(new CurrentGraphicsState() { CurrentClippingPath = clippingPath });
             ColorSpaceContext = new ColorSpaceContext(GetCurrentState, resourceStore);
         }
 
@@ -399,7 +407,41 @@
 
         public void BeginSubpath()
         {
-  
+            if (CurrentPath == null)
+            {
+                CurrentPath = new PdfPath();
+            }
+
+            AddSubpath();
+            CurrentSubpath = new PdfSubpath();
+        }
+
+        public PdfPoint CloseSubpath()
+        {
+            PdfPoint point;
+            if (CurrentSubpath.Commands[0] is Move move)
+            {
+                point = move.Location;
+            }
+            else
+            {
+                throw new ArgumentException("CloseSubpath(): first command not Move.");
+            }
+
+            CurrentSubpath.ClosePath();
+            AddSubpath();
+            return point;
+        }
+
+        public void AddSubpath()
+        {
+            if (CurrentSubpath == null)
+            {
+                return;
+            }
+
+            CurrentPath.Add(CurrentSubpath);
+            CurrentSubpath = null;
         }
 
         public void StrokePath(bool close)
@@ -407,11 +449,16 @@
 
         }
 
-        public void FillPath(bool close)
+        public void FillPath(FillingRule fillingRule, bool close)
         {
 
         }
 
+        public void FillStrokePath(FillingRule fillingRule, bool close)
+        {
+            
+        }
+        
         public void ClosePath()
         {
 

src/UglyToad.PdfPig/Graphics/IOperationContext.cs

@@ -95,8 +95,16 @@
         /// <summary>
         /// Fill the current path.
         /// </summary>
+        /// <param name="fillingRule">The filling rule to use.</param>
         /// <param name="close">Whether to also close the path.</param>
-        void FillPath(bool close);
+        void FillPath(FillingRule fillingRule, bool close);
+
+        /// <summary>
+        /// Fill and stroke the current path.
+        /// </summary>
+        /// <param name="fillingRule">The filling rule to use.</param>
+        /// <param name="close">Whether to also close the path.</param>
+        void FillStrokePath(FillingRule fillingRule, bool close);
 
         /// <summary>
         /// Close the current path.

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/CloseFillPathEvenOddRuleAndStroke.cs

@@ -28,7 +28,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(true);
+            operationContext.FillStrokePath(PdfPig.Core.FillingRule.EvenOdd, true);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/CloseFillPathNonZeroWindingAndStroke.cs

@@ -28,7 +28,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(true);
+            operationContext.FillStrokePath(PdfPig.Core.FillingRule.NonZeroWinding, true);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/FillPathEvenOddRule.cs

@@ -28,7 +28,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(false);
+            operationContext.FillPath(PdfPig.Core.FillingRule.EvenOdd, false);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/FillPathEvenOddRuleAndStroke.cs

@@ -28,7 +28,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(false);
+            operationContext.FillStrokePath(PdfPig.Core.FillingRule.EvenOdd, false);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/FillPathNonZeroWinding.cs

@@ -29,7 +29,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(false);
+            operationContext.FillPath(PdfPig.Core.FillingRule.NonZeroWinding, false);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/FillPathNonZeroWindingAndStroke.cs

@@ -28,7 +28,7 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(false);
+            operationContext.FillStrokePath(PdfPig.Core.FillingRule.NonZeroWinding, false);
         }
 
         /// <inheritdoc />

src/UglyToad.PdfPig/Graphics/Operations/PathPainting/FillPathNonZeroWindingCompatibility.cs

@@ -15,7 +15,7 @@
         public const string Symbol = "F";
 
         /// <summary>
-        /// The instance of the <see cref="FillPathEvenOddRuleAndStroke"/> operation.
+        /// The instance of the <see cref="FillPathNonZeroWindingCompatibility"/> operation.
         /// </summary>
         public static readonly FillPathNonZeroWindingCompatibility Value = new FillPathNonZeroWindingCompatibility();
 
@@ -29,13 +29,14 @@
         /// <inheritdoc />
         public void Run(IOperationContext operationContext)
         {
-            operationContext.FillPath(false);
+            operationContext.FillPath(PdfPig.Core.FillingRule.NonZeroWinding, false);
         }
 
         /// <inheritdoc />
         public void Write(Stream stream)
         {
-            stream.WriteText(Symbol);
+            // Although PDF reader applications shall be able to accept this operator, PDF writer applications should use f instead.
+            stream.WriteText(FillPathNonZeroWinding.Symbol); 
             stream.WriteNewLine();
         }