Skip to content

Commit be14857

Browse files
committed
chore: pre-check path length
1 parent 440d1b8 commit be14857

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

intersect.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -109,7 +109,7 @@ function parsePathString(pathString) {
109109
*/
110110
function isPathAbsolute(pathComponents) {
111111

112-
for (var i = 0; i < pathComponents.length; i++) {
112+
for (var i = 0, ii = pathComponents.length; i < ii; i++) {
113113
var command = pathComponents[i][0];
114114
if (typeof command === 'string' && command !== command.toUpperCase()) {
115115
return false;
@@ -128,7 +128,8 @@ function isPathAbsolute(pathComponents) {
128128
* @return {boolean}
129129
*/
130130
function isPathCurve(pathComponents) {
131-
for (var i = 0; i < pathComponents.length; i++) {
131+
132+
for (var i = 0, ii = pathComponents.length; i < ii; i++) {
132133
var command = pathComponents[i][0];
133134
if (command !== 'M' && command !== 'C') {
134135
return false;
@@ -170,11 +171,11 @@ function pathClone(pathComponents) {
170171

171172
var pathComponentsClone = new Array(pathComponents.length);
172173

173-
for (var i = 0; i < pathComponents.length; i++) {
174+
for (var i = 0, ii = pathComponents.length; i < ii; i++) {
174175
var component = pathComponents[i];
175176
var componentClone = pathComponentsClone[i] = new Array(component.length);
176177

177-
for (var j = 0; j < component.length; j++) {
178+
for (var j = 0, jj = component.length; j < jj; j++) {
178179
componentClone[j] = component[j];
179180
}
180181
}

0 commit comments

Comments
 (0)