Skip to content

Commit 3b0e6e7

Browse files
committed
Remove split-long-paths optimizer pass and all related options and tests.
1 parent 9efe510 commit 3b0e6e7

3 files changed

Lines changed: 0 additions & 736 deletions

File tree

include/pagx/PAGXOptimizer.h

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -53,30 +53,6 @@ class PAGXOptimizer {
5353
bool canonicalizePaths = true;
5454
/** Replace alpha-rectangle masks with an equivalent scrollRect on the parent layer. */
5555
bool rectMaskToScrollRect = true;
56-
/**
57-
* Split Path elements whose PathData verb count exceeds `pathVerbThreshold` into multiple
58-
* sibling Paths at sub-path boundaries. Hole sub-paths (bounding-box-contained) are kept
59-
* together with their outer contour so fill-rule rendering is preserved exactly. Skipped
60-
* for reversed paths, paths with layout constraints or customData, and Paths whose
61-
* PathData is a shared/named resource.
62-
*
63-
* Off by default because the resulting Paths each carry their own PathData and slightly
64-
* inflate the document; enable when targeting a renderer that warns on high-verb paths.
65-
*/
66-
bool splitLongPaths = false;
67-
/** Verb count above which a Path becomes eligible for splitting. Matches the warning
68-
* threshold used by `pagx verify` (`> 1024`). */
69-
int pathVerbThreshold = 1024;
70-
/** Maximum verbs allowed in an emitted chunk; must be strictly less than the threshold. */
71-
int maxVerbsPerPath = 900;
72-
/**
73-
* Allow the splitter to chord-split a single closed sub-path that exceeds the threshold
74-
* by inserting an internal straight chord, producing two filled halves whose union
75-
* matches the original area. The chord runs inside the contour so it is invisible for
76-
* solid fills but may be perceptible on stroked or alpha-composited paths. Off by
77-
* default — opt in for assets where you have verified the affected paths are filled.
78-
*/
79-
bool enableChordSplit = false;
8056
/** Collapse PathData resources with identical contents into a single instance. */
8157
bool dedupPathData = true;
8258
/** Drop resources from `<Resources>` that no longer have any reference in the layer tree. */

0 commit comments

Comments
 (0)