From 0d576ac87425d5329ac402f98f12e8c4183686f0 Mon Sep 17 00:00:00 2001 From: Bhargav Dabhade Date: Fri, 9 Jan 2026 18:19:55 +0000 Subject: [PATCH 1/3] bench: refactor to use dynamic memory allocation in stats/strided/dnanmeanpn --- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: missing_dependencies - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: na - task: lint_license_headers status: passed --- --- .../strided/dnanmeanpn/benchmark/c/benchmark.length.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c index ce81ec85d19f..ce40d0490125 100644 --- a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c +++ b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c @@ -96,11 +96,11 @@ static double rand_double( void ) { */ static double benchmark1( int iterations, int len ) { double elapsed; - double x[ len ]; + double *x; double v; double t; int i; - + x = (double *) malloc( len * sizeof( double ) ); for ( i = 0; i < len; i++ ) { if ( rand_double() < 0.2 ) { x[ i ] = 0.0 / 0.0; // NaN @@ -122,6 +122,7 @@ static double benchmark1( int iterations, int len ) { if ( v != v ) { printf( "should not return NaN\n" ); } + free( x ); return elapsed; } @@ -134,11 +135,11 @@ static double benchmark1( int iterations, int len ) { */ static double benchmark2( int iterations, int len ) { double elapsed; - double x[ len ]; + double *x; double v; double t; int i; - + x = (double *) malloc( len * sizeof( double ) ); for ( i = 0; i < len; i++ ) { if ( rand_double() < 0.2 ) { x[ i ] = 0.0 / 0.0; // NaN @@ -160,6 +161,7 @@ static double benchmark2( int iterations, int len ) { if ( v != v ) { printf( "should not return NaN\n" ); } + free( x ); return elapsed; } From 27ac7945e9fc061a6b4594b2f4bdbba65eb0b405 Mon Sep 17 00:00:00 2001 From: Bhargav Dabhade Date: Fri, 9 Jan 2026 18:24:47 +0000 Subject: [PATCH 2/3] bench: refactor to use dynamic memory allocation in stats/strided/dnanmeanpn --- type: pre_commit_static_analysis_report description: Results of running static analysis checks when committing changes. report: - task: lint_filenames status: passed - task: lint_editorconfig status: passed - task: lint_markdown status: na - task: lint_package_json status: na - task: lint_repl_help status: na - task: lint_javascript_src status: na - task: lint_javascript_cli status: na - task: lint_javascript_examples status: na - task: lint_javascript_tests status: na - task: lint_javascript_benchmarks status: na - task: lint_python status: na - task: lint_r status: na - task: lint_c_src status: na - task: lint_c_examples status: na - task: lint_c_benchmarks status: missing_dependencies - task: lint_c_tests_fixtures status: na - task: lint_shell status: na - task: lint_typescript_declarations status: passed - task: lint_typescript_tests status: na - task: lint_license_headers status: passed --- --- .../stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c index ce40d0490125..7a39e76bb599 100644 --- a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c +++ b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c @@ -139,6 +139,7 @@ static double benchmark2( int iterations, int len ) { double v; double t; int i; + x = (double *) malloc( len * sizeof( double ) ); for ( i = 0; i < len; i++ ) { if ( rand_double() < 0.2 ) { From 2adde92be3ad51651c100a22b5faaea1e2f642f1 Mon Sep 17 00:00:00 2001 From: Athan Date: Fri, 9 Jan 2026 15:52:21 -0800 Subject: [PATCH 3/3] Apply suggestions from code review Signed-off-by: Athan --- .../stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c | 1 + 1 file changed, 1 insertion(+) diff --git a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c index 7a39e76bb599..bf1106dd941a 100644 --- a/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c +++ b/lib/node_modules/@stdlib/stats/strided/dnanmeanpn/benchmark/c/benchmark.length.c @@ -100,6 +100,7 @@ static double benchmark1( int iterations, int len ) { double v; double t; int i; + x = (double *) malloc( len * sizeof( double ) ); for ( i = 0; i < len; i++ ) { if ( rand_double() < 0.2 ) {