chore(plot): replace array pre-allocation with push

anxhukumar · web-flow · commit fea4b5b38cc1 · 2025-03-07T17:36:26.000-05:00
PR-URL: #5852 Closes: #5791 Reviewed-by: Philipp Burckhardt <pburckhardt@outlook.com> Reviewed-by: Gururaj Gurram <gururajgurram1512@gmail.com>
diff --git a/lib/node_modules/@stdlib/plot/components/svg/symbols/lib/render/closed_circles.js b/lib/node_modules/@stdlib/plot/components/svg/symbols/lib/render/closed_circles.js
@@ -67,7 +67,7 @@ function render( state ) {
 	x = state.x;
 	y = state.y;
 
-	out = new Array( x.length );
+	out = [];
 	for ( i = 0; i < x.length; i++ ) {
 		xi = x[ i ];
 		yi = y[ i ];
@@ -91,7 +91,7 @@ function render( state ) {
 			}
 		};
 		debug( 'Generating a virtual DOM tree (%s) with properties: %s.', ELEMENT, JSON.stringify( props ) );
-		out[ i ] = h( ELEMENT, props, [] );
+		out.push( h( ELEMENT, props, [] ) );
 	}
 	return out;
 }

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ function render( state ) {`
`67`	`67`	`x = state.x;`
`68`	`68`	`y = state.y;`
`69`	`69`
`70`		`- out = new Array( x.length );`
	`70`	`+ out = [];`
`71`	`71`	`for ( i = 0; i < x.length; i++ ) {`
`72`	`72`	`xi = x[ i ];`
`73`	`73`	`yi = y[ i ];`
`@@ -91,7 +91,7 @@ function render( state ) {`
`91`	`91`	`}`
`92`	`92`	`};`
`93`	`93`	`debug( 'Generating a virtual DOM tree (%s) with properties: %s.', ELEMENT, JSON.stringify( props ) );`
`94`		`- out[ i ] = h( ELEMENT, props, [] );`
	`94`	`+ out.push( h( ELEMENT, props, [] ) );`
`95`	`95`	`}`
`96`	`96`	`return out;`
`97`	`97`	`}`