Improve mergePaths plugin performance on large files

svg · Sep 27, 2023 · d9b4580 · d9b4580
1 parent ad3e604
commit d9b4580
Showing 1 changed file with 32 additions and 8 deletions.
diff --git a/plugins/mergePaths.js b/plugins/mergePaths.js
@@ -1,6 +1,5 @@
 'use strict';
 
-const { detachNodeFromParent } = require('../lib/xast.js');
 const { collectStylesheet, computeStyle } = require('../lib/style.js');
 const { path2js, js2path, intersects } = require('./_path.js');
 
@@ -26,7 +25,15 @@ exports.fn = (root, params) => {
     element: {
       enter: (node) => {
         let prevChild = null;
+        let prevPathJS = null;
 
+        const saveCurrent = () => {
+          js2path(prevChild, prevPathJS, {
+            floatPrecision,
+            noSpaceAfterFlags,
+          });
+          prevPathJS = null;
+        };
         for (const child of node.children) {
           // skip if previous element is not path or contains animation elements
           if (
@@ -36,6 +43,9 @@ exports.fn = (root, params) => {
             prevChild.children.length !== 0 ||
             prevChild.attributes.d == null
           ) {
+            if (prevPathJS) {
+              saveCurrent();
+            }
             prevChild = child;
             continue;
           }
@@ -47,6 +57,9 @@ exports.fn = (root, params) => {
             child.children.length !== 0 ||
             child.attributes.d == null
           ) {
+            if (prevPathJS) {
+              saveCurrent();
+            }
             prevChild = child;
             continue;
           }
@@ -58,10 +71,12 @@ exports.fn = (root, params) => {
             computedStyle['marker-mid'] ||
             computedStyle['marker-end']
           ) {
+            if (prevPathJS) {
+              saveCurrent();
+            }
             prevChild = child;
             continue;
           }
-
           const prevChildAttrs = Object.keys(prevChild.attributes);
           const childAttrs = Object.keys(child.attributes);
           let attributesAreEqual = prevChildAttrs.length === childAttrs.length;
@@ -75,23 +90,32 @@ exports.fn = (root, params) => {
               }
             }
           }
-          const prevPathJS = path2js(prevChild);
+          const hasPrevPath = !!prevPathJS;
+          if (!hasPrevPath) prevPathJS = path2js(prevChild);
           const curPathJS = path2js(child);
 
           if (
             attributesAreEqual &&
             (force || !intersects(prevPathJS, curPathJS))
           ) {
-            js2path(prevChild, prevPathJS.concat(curPathJS), {
-              floatPrecision,
-              noSpaceAfterFlags,
-            });
-            detachNodeFromParent(child, node);
+            prevPathJS.push(...curPathJS);
+            child._deleteAfterwards = 1;
+
             continue;
           }
 
+          if (hasPrevPath) {
+            saveCurrent();
+          }
+          prevPathJS = null;
           prevChild = child;
         }
+        if (prevPathJS) {
+          saveCurrent();
+        }
+        node.children = node.children.filter(
+          (child) => !child._deleteAfterwards
+        );
       },
     },
   };