Merge pull request #19972 from Snuffleupagus/issue-19971

Snuffleupagus · web-flow · commit f11d8b0f6772 · 2025-05-23T18:34:07.000+02:00
Ignore empty paths when optimizing `constructPath` operations (issue 19971)
diff --git a/src/core/operator_list.js b/src/core/operator_list.js
@@ -519,18 +519,20 @@ addState(
     const transform = argsArray[iFirstTransform];
     const [, [buffer], minMax] = args;
 
-    Util.scaleMinMax(transform, minMax);
-    for (let k = 0, kk = buffer.length; k < kk; ) {
-      switch (buffer[k++]) {
-        case DrawOPS.moveTo:
-        case DrawOPS.lineTo:
-          Util.applyTransform(buffer, transform, k);
-          k += 2;
-          break;
-        case DrawOPS.curveTo:
-          Util.applyTransformToBezier(buffer, transform, k);
-          k += 6;
-          break;
+    if (minMax) {
+      Util.scaleMinMax(transform, minMax);
+      for (let k = 0, kk = buffer.length; k < kk; ) {
+        switch (buffer[k++]) {
+          case DrawOPS.moveTo:
+          case DrawOPS.lineTo:
+            Util.applyTransform(buffer, transform, k);
+            k += 2;
+            break;
+          case DrawOPS.curveTo:
+            Util.applyTransformToBezier(buffer, transform, k);
+            k += 6;
+            break;
+        }
       }
     }
     // Replace queue items.
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
@@ -622,6 +622,7 @@
 !bug1815476.pdf
 !issue16021.pdf
 !bug1770750.pdf
+!issue19971.pdf
 !issue16063.pdf
 !issue19389.pdf
 !issue16067.pdf
diff --git a/test/pdfs/issue19971.pdf b/test/pdfs/issue19971.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -4142,6 +4142,14 @@
     "lastPage": 1,
     "type": "eq"
   },
+  {
+    "id": "issue19971",
+    "file": "pdfs/issue19971.pdf",
+    "md5": "2eb897b3ef3342bfc5ad31886e7cd1fe",
+    "rounds": 1,
+    "lastPage": 1,
+    "type": "eq"
+  },
   {
     "id": "issue8702-text",
     "file": "pdfs/issue8702.pdf",
