refactor: apply suggestions from PR review

---
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: passed
  - task: lint_package_json
    status: passed
  - task: lint_repl_help
    status: passed
  - task: lint_javascript_src
    status: passed
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: passed
  - task: lint_javascript_tests
    status: passed
  - task: lint_javascript_benchmarks
    status: passed
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: passed
  - task: lint_c_examples
    status: passed
  - task: lint_c_benchmarks
    status: passed
  - 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
---

Author: anandkaranubc

lib/node_modules/@stdlib/math/base/special/wrapf/README.md

@@ -20,7 +20,7 @@ limitations under the License.
 
 # wrapf
 
-> Wrap a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+> Wrap a single-precision floating-point value to the half-open interval `[min,max)`.
 
 <!-- Section to include introductory text. Make sure to keep an empty line after the intro `section` element and another before the `/section` close. -->
 
@@ -42,7 +42,7 @@ var wrapf = require( '@stdlib/math/base/special/wrapf' );
 
 #### wrapf( v, min, max )
 
-Wrap a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+Wraps a single-precision floating-point value to the half-open interval `[min,max)`.
 
 ```javascript
 var v = wrapf( 3.14, 0.0, 5.0 );
@@ -106,20 +106,18 @@ var v = wrapf( 3.14, 3.0, 3.0 );
 <!-- eslint no-undef: "error" -->
 
 ```javascript
-var discreteUniform = require( '@stdlib/random/base/discrete-uniform' );
+var uniform = require( '@stdlib/random/array/uniform' );
+var logEachMap = require( '@stdlib/console/log-each-map' );
 var wrapf = require( '@stdlib/math/base/special/wrapf' );
 
-var min;
-var max;
-var v;
-var i;
+var opts = {
+    'dtype': 'float32'
+};
+var min = uniform( 100, 0.0, 10.0, opts );
+var max = uniform( 100, 5.0, 15.0, opts );
+var v = uniform( 100, -20.0, 20.0, opts );
 
-for ( i = 0; i < 100; i++ ) {
-    min = discreteUniform( 0.0, 10.0 );
-    max = discreteUniform( 5.0, 15.0 );
-    v = discreteUniform( -20.0, 20.0 );
-    console.log( 'wrapf(%d,%d,%d) => %d', v, min, max, wrapf( v, min, max ) );
-}
+logEachMap( 'wrapf(%0.4f,%0.4f,%0.4f) => %0.4f', v, min, max, wrapf );
 ```
 
 </section>
@@ -154,14 +152,14 @@ for ( i = 0; i < 100; i++ ) {
 
 #### stdlib_base_wrapf( v, min, max )
 
-Wraps a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+Wrap a single-precision floating-point value to the half-open interval `[min,max)`.
 
 ```c
-float v = stdlib_base_wrapf( 3.14, 0.0, 5.0 );
-// returns ~3.14
+float v = stdlib_base_wrapf( 3.14f, 0.0f, 5.0f );
+// returns ~3.14f
 
-v = stdlib_base_wrapf( -3.14, 0.0, 5.0 );
-// returns ~1.86
+v = stdlib_base_wrapf( -3.14f, 0.0f, 5.0f );
+// returns ~1.86f
 ```
 
 The function accepts the following arguments:

lib/node_modules/@stdlib/math/base/special/wrapf/benchmark/benchmark.js

@@ -21,7 +21,7 @@
 // MODULES //
 
 var bench = require( '@stdlib/bench' );
-var randu = require( '@stdlib/random/array/uniform' );
+var uniform = require( '@stdlib/random/array/uniform' );
 var isnanf = require( '@stdlib/math/base/assert/is-nanf' );
 var pkg = require( './../package.json' ).name;
 var wrapf = require( './../lib' );
@@ -34,7 +34,9 @@ bench( pkg, function benchmark( b ) {
 	var y;
 	var i;
 
-	x = randu( 100, -10.0, 10.0 );
+	x = uniform( 100, -10.0, 10.0, {
+		'dtype': 'float32'
+	});
 
 	b.tic();
 	for ( i = 0; i < b.iterations; i++ ) {

lib/node_modules/@stdlib/math/base/special/wrapf/benchmark/benchmark.native.js

@@ -22,7 +22,7 @@
 
 var resolve = require( 'path' ).resolve;
 var bench = require( '@stdlib/bench' );
-var randu = require( '@stdlib/random/array/uniform' );
+var uniform = require( '@stdlib/random/array/uniform' );
 var isnanf = require( '@stdlib/math/base/assert/is-nanf' );
 var tryRequire = require( '@stdlib/utils/try-require' );
 var pkg = require( './../package.json' ).name;
@@ -43,7 +43,9 @@ bench( pkg+'::native', opts, function benchmark( b ) {
 	var y;
 	var i;
 
-	x = randu( 100, -10.0, 10.0 );
+	x = uniform( 100, -10.0, 10.0, {
+		'dtype': 'float32'
+	});
 
 	b.tic();
 	for ( i = 0; i < b.iterations; i++ ) {

lib/node_modules/@stdlib/math/base/special/wrapf/benchmark/c/benchmark.c

@@ -89,7 +89,7 @@ static float rand_float( void ) {
 * @param v      value to wrap
 * @param min    minimum value
 * @param max    maximum value
-* @return wrapped value
+* @return       wrapped value
 */
 float wrapf( float v, float min, float max ) {
 	float delta;
@@ -99,9 +99,9 @@ float wrapf( float v, float min, float max ) {
 	// Perform range reduction...
 	delta = max - min;
 	if ( v < min ) {
-		v += delta * ( truncf( ( min - v ) / delta ) + 1.0 );
+		v += delta * ( truncf( (min-v)/delta ) + 1.0f );
 	}
-	return min + fmodf( ( v - min ), delta );
+	return min + fmodf( (v-min), delta );
 }
 
 /**
@@ -111,18 +111,18 @@ float wrapf( float v, float min, float max ) {
 */
 static double benchmark( void ) {
 	double elapsed;
-	double t;
 	float x[ 100 ];
+	double t;
 	float y;
 	int i;
 
 	for ( i = 0; i < 100; i++ ) {
-		x[ i ] = ( 20.0f * rand_float() ) - 10.0f;
+		x[ i ] = ( 20.0f*rand_float() ) - 10.0f;
 	}
 
 	t = tic();
 	for ( i = 0; i < ITERATIONS; i++ ) {
-		y = wrapf( x[ i%100 ], -5.0, 5.0 );
+		y = wrapf( x[ i%100 ], -5.0f, 5.0f );
 		if ( y != y ) {
 			printf( "should not return NaN\n" );
 			break;

lib/node_modules/@stdlib/math/base/special/wrapf/benchmark/c/native/benchmark.c

@@ -91,18 +91,18 @@ static float rand_float( void ) {
 */
 static double benchmark( void ) {
 	double elapsed;
-	double t;
 	float x[ 100 ];
+	double t;
 	float y;
 	int i;
 
 	for ( i = 0; i < 100; i++ ) {
-		x[ i ] = ( 20.0f * rand_float() ) - 10.0f;
+		x[ i ] = ( 20.0f*rand_float() ) - 10.0f;
 	}
 
 	t = tic();
 	for ( i = 0; i < ITERATIONS; i++ ) {
-		y = stdlib_base_wrapf( x[ i%100 ], -5.0, 5.0 );
+		y = stdlib_base_wrapf( x[ i%100 ], -5.0f, 5.0f );
 		if ( y != y ) {
 			printf( "should not return NaN\n" );
 			break;

lib/node_modules/@stdlib/math/base/special/wrapf/docs/repl.txt

@@ -1,7 +1,7 @@
 
 {{alias}}( v, min, max )
-    Wraps a value on the half-open interval `[min,max)` for single-precision
-    floating-point numbers.
+    Wraps a single-precision floating-point value to the half-open interval
+    `[min,max)`.
 
     The function does not distinguish between positive and negative zero. Where
     appropriate, the function returns positive zero.

lib/node_modules/@stdlib/math/base/special/wrapf/docs/types/index.d.ts

@@ -19,7 +19,7 @@
 // TypeScript Version: 4.1
 
 /**
-* Wraps a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+* Wraps a single-precision floating-point value to the half-open interval `[min,max)`.
 *
 * @param v - input value
 * @param min - minimum value

lib/node_modules/@stdlib/math/base/special/wrapf/examples/c/example.c

@@ -28,6 +28,6 @@ int main( void ) {
 	int i;
 	for ( i = 0; i < 5; i++ ) {
 		out = stdlib_base_wrapf( v[i], min[i], max[i] );
-		printf( "wrap(%f,%f,%f) => %f\n", v[i], min[i], max[i], out );
+		printf( "wrapf(%f,%f,%f) => %f\n", v[i], min[i], max[i], out );
 	}
 }

lib/node_modules/@stdlib/math/base/special/wrapf/examples/index.js

@@ -18,17 +18,15 @@
 
 'use strict';
 
-var discreteUniform = require( '@stdlib/random/base/discrete-uniform' );
+var uniform = require( '@stdlib/random/array/uniform' );
+var logEachMap = require( '@stdlib/console/log-each-map' );
 var wrapf = require( './../lib' );
 
-var min;
-var max;
-var v;
-var i;
+var opts = {
+	'dtype': 'float32'
+};
+var min = uniform( 100, 0.0, 10.0, opts );
+var max = uniform( 100, 5.0, 15.0, opts );
+var v = uniform( 100, -20.0, 20.0, opts );
 
-for ( i = 0; i < 100; i++ ) {
-	min = discreteUniform( 0.0, 10.0 );
-	max = discreteUniform( 5.0, 15.0 );
-	v = discreteUniform( -20.0, 20.0 );
-	console.log( 'wrapf(%d,%d,%d) => %d', v, min, max, wrapf( v, min, max ) );
-}
+logEachMap( 'wrapf(%0.4f,%0.4f,%0.4f) => %0.4f', v, min, max, wrapf );

lib/node_modules/@stdlib/math/base/special/wrapf/include/stdlib/math/base/special/wrapf.h

@@ -27,7 +27,7 @@ extern "C" {
 #endif
 
 /**
-* Wraps a value on the half-open interval [min,max) for single-precision floating-point numbers.
+* Wraps a single-precision floating-point value to the half-open interval [min,max).
 */
 float stdlib_base_wrapf( const float v, const float min, const float max );
 

lib/node_modules/@stdlib/math/base/special/wrapf/lib/index.js

@@ -19,7 +19,7 @@
 'use strict';
 
 /**
-* Wrap a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+* Wrap a single-precision floating-point value to the half-open interval `[min,max)`.
 *
 * @module @stdlib/math/base/special/wrapf
 *

lib/node_modules/@stdlib/math/base/special/wrapf/lib/main.js

@@ -22,13 +22,19 @@
 
 var isnanf = require( '@stdlib/math/base/assert/is-nanf' );
 var truncf = require( '@stdlib/math/base/special/truncf' );
-var float64ToFloat32 = require( '@stdlib/number/float64/base/to-float32' );
+var f32 = require( '@stdlib/number/float64/base/to-float32' );
+
+
+// VARIABLES //
+
+var ZERO = f32( 0.0 );
+var ONE = f32( 1.0 );
 
 
 // MAIN //
 
 /**
-* Wraps a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+* Wraps a single-precision floating-point value to the half-open interval `[min,max)`.
 *
 * @param {number} v - input value
 * @param {number} min - minimum value
@@ -65,6 +71,10 @@ var float64ToFloat32 = require( '@stdlib/number/float64/base/to-float32' );
 */
 function wrapf( v, min, max ) {
 	var delta;
+
+	v = f32( v );
+	min = f32( min );
+	max = f32( max );
 	if (
 		isnanf( v ) ||
 		isnanf( min ) ||
@@ -73,29 +83,26 @@ function wrapf( v, min, max ) {
 	) {
 		return NaN;
 	}
-
 	// Normalize +-0 to +0...
 	if ( v === 0.0 ) {
-		v = 0.0;
+		v = ZERO;
 	}
 	if ( min === 0.0 ) {
-		min = 0.0;
+		min = ZERO;
 	}
 	if ( max === 0.0 ) {
-		max = 0.0;
+		max = ZERO;
 	}
-
 	// Simple case where value is already within range...
 	if ( min <= v && v < max ) {
-		return float64ToFloat32( v );
+		return v;
 	}
-
 	// Perform range reduction...
-	delta = max - min;
+	delta = f32( max - min );
 	if ( v < min ) {
-		v += delta * ( truncf( ( min - v ) / delta ) + 1.0 );
+		v = f32( v + f32( delta * f32( truncf( f32(min-v) / f32(delta) ) + ONE ) ) );
 	}
-	return float64ToFloat32( float64ToFloat32( min ) + float64ToFloat32( float64ToFloat32( float64ToFloat32( v ) - float64ToFloat32( min ) ) % float64ToFloat32( delta ) ) );
+	return f32( min + f32( f32( v-min ) % delta ) );
 }
 
 

lib/node_modules/@stdlib/math/base/special/wrapf/lib/native.js

@@ -26,7 +26,7 @@ var addon = require( './../src/addon.node' );
 // MAIN //
 
 /**
-* Wraps a value on the half-open interval `[min,max)` for single-precision floating-point numbers.
+* Wraps a single-precision floating-point value to the half-open interval `[min,max)`.
 *
 * @private
 * @param {number} v - input value

lib/node_modules/@stdlib/math/base/special/wrapf/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@stdlib/math/base/special/wrapf",
   "version": "0.0.0",
-  "description": "Wrap a value on the half-open interval [min,max) for single-precision floating-point numbers",
+  "description": "Wrap a single-precision floating-point value to the half-open interval [min,max).",
   "license": "Apache-2.0",
   "author": {
     "name": "The Stdlib Authors",
@@ -57,6 +57,7 @@
     "mathematics",
     "math",
     "wrap",
+    "wrapf",
     "modulo",
     "mod",
     "range",

lib/node_modules/@stdlib/math/base/special/wrapf/src/addon.c

@@ -19,5 +19,4 @@
 #include "stdlib/math/base/special/wrapf.h"
 #include "stdlib/math/base/napi/ternary.h"
 
-// cppcheck-suppress shadowFunction
 STDLIB_MATH_BASE_NAPI_MODULE_FFF_F( stdlib_base_wrapf )

lib/node_modules/@stdlib/math/base/special/wrapf/src/main.c

@@ -21,10 +21,9 @@
 #include "stdlib/math/base/special/truncf.h"
 #include "stdlib/math/base/special/fmodf.h"
 #include "stdlib/math/base/assert/is_negative_zerof.h"
-#include <math.h>
 
 /**
-* Wraps a value on the half-open interval [min,max) for single-precision floating-point numbers.
+* Wraps a single-precision floating-point value to the half-open interval [min,max).
 *
 * @param v    input value
 * @param min  minimum value
@@ -33,7 +32,7 @@
 *
 * @example
 * float v = stdlib_base_wrapf( 3.14f, 0.0f, 5.0f );
-* // returns ~3.14
+* // returns ~3.14f
 */
 float stdlib_base_wrapf( const float v, const float min, const float max ) {
 	float delta;
@@ -58,12 +57,10 @@ float stdlib_base_wrapf( const float v, const float min, const float max ) {
 	if ( stdlib_base_is_negative_zerof( maxc ) ) {
 		maxc = 0.0f;
 	}
-
 	// Simple case where value is already within range...
 	if ( minc <= vc && vc < maxc ) {
 		return vc;
 	}
-
 	// Perform range reduction...
 	delta = maxc - minc;
 	if ( vc < minc ) {