feat: may be an approach to fix #1105

src/core/PathProxy.ts

@@ -365,6 +365,9 @@ export default class PathProxy {
     }
 
     setData(data: Float32Array | number[]) {
+        if (!this._saveData) {
+            return;
+        }
 
         const len = data.length;
 
@@ -380,6 +383,9 @@ export default class PathProxy {
     }
 
     appendPath(path: PathProxy | PathProxy[]) {
+        if (!this._saveData) {
+            return;
+        }
         if (!(path instanceof Array)) {
             path = [path];
         }
@@ -389,8 +395,14 @@ export default class PathProxy {
         for (let i = 0; i < len; i++) {
             appendSize += path[i].len();
         }
-        if (hasTypedArray && (this.data instanceof Float32Array)) {
+        const oldData = this.data;
+        if (hasTypedArray && (oldData instanceof Float32Array || !oldData)) {
             this.data = new Float32Array(offset + appendSize);
+            if (offset > 0 && oldData) {
+                for (let k = 0; k < offset; k++) {
+                    this.data[k] = oldData[k];
+                }
+            }
         }
         for (let i = 0; i < len; i++) {
             const appendPathData = path[i].data;
@@ -970,6 +982,10 @@ export default class PathProxy {
         return newProxy;
     }
 
+    canSave(): boolean {
+        return !!this._saveData;
+    }
+
     private static initDefaultProps = (function () {
         const proto = PathProxy.prototype;
         proto._saveData = true;

src/tool/path.ts

@@ -383,22 +383,27 @@ class SVGPath extends Path {
 function isPathProxy(path: PathProxy | CanvasRenderingContext2D): path is PathProxy {
     return (path as PathProxy).setData != null;
 }
+
 // TODO Optimize double memory cost problem
 function createPathOptions(str: string, opts: SVGPathOption): InnerSVGPathOption {
     const pathProxy = createPathProxyFromString(str);
     const innerOpts: InnerSVGPathOption = extend({}, opts);
     innerOpts.buildPath = function (path: PathProxy | CanvasRenderingContext2D) {
-        if (isPathProxy(path)) {
-            path.setData(pathProxy.data);
+        const beProxy = isPathProxy(path);
+        if (beProxy && path.canSave()) {
+            // path.setData(pathProxy.data);
+            path.appendPath(pathProxy);
             // Svg and vml renderer don't have context
             const ctx = path.getContext();
             if (ctx) {
                 path.rebuildPath(ctx, 1);
             }
         }
         else {
-            const ctx = path;
-            pathProxy.rebuildPath(ctx, 1);
+            const ctx = beProxy ? path.getContext() : path;
+            if (ctx) {
+                pathProxy.rebuildPath(ctx, 1);
+            }
         }
     };