Revert "refactor: use stdlib fmod and DDD_D napi function"

This reverts commit b345bfe.

Author: gunjjoshi

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

@@ -22,7 +22,6 @@
 
 var isnan = require( '@stdlib/math/base/assert/is-nan' );
 var trunc = require( '@stdlib/math/base/special/trunc' );
-var fmod = require( '@stdlib/math/base/special/fmod' );
 
 
 // MAIN //
@@ -92,7 +91,7 @@ function wrap( v, min, max ) {
 	if ( v < min ) {
 		v += delta * ( trunc( (min-v)/delta ) + 1.0 );
 	}
-	return min + ( fmod( ( v - min ), delta ) );
+	return min + ( (v-min) % delta );
 }
 
 

lib/node_modules/@stdlib/math/base/special/wrap/manifest.json

@@ -1,7 +1,5 @@
 {
-  "options": {
-    "task": "build"
-  },
+  "options": {},
   "fields": [
     {
       "field": "src",
@@ -26,7 +24,6 @@
   ],
   "confs": [
     {
-      "task": "build",
       "src": [
         "./src/main.c"
       ],
@@ -36,45 +33,8 @@
       "libraries": [],
       "libpath": [],
       "dependencies": [
-        "@stdlib/math/base/napi/ternary",
         "@stdlib/math/base/assert/is-nan",
-        "@stdlib/math/base/special/trunc",
-        "@stdlib/math/base/special/fmod",
-        "@stdlib/math/base/special/abs"
-      ]
-    },
-    {
-      "task": "benchmark",
-      "src": [
-        "./src/main.c"
-      ],
-      "include": [
-        "./include"
-      ],
-      "libraries": [],
-      "libpath": [],
-      "dependencies": [
-        "@stdlib/math/base/assert/is-nan",
-        "@stdlib/math/base/special/trunc",
-        "@stdlib/math/base/special/fmod",
-        "@stdlib/math/base/special/abs"
-      ]
-    },
-    {
-      "task": "examples",
-      "src": [
-        "./src/main.c"
-      ],
-      "include": [
-        "./include"
-      ],
-      "libraries": [],
-      "libpath": [],
-      "dependencies": [
-        "@stdlib/math/base/assert/is-nan",
-        "@stdlib/math/base/special/trunc",
-        "@stdlib/math/base/special/fmod",
-        "@stdlib/math/base/special/abs"
+        "@stdlib/math/base/special/trunc"
       ]
     }
   ]

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

@@ -17,7 +17,97 @@
 */
 
 #include "stdlib/math/base/special/wrap.h"
-#include "stdlib/math/base/napi/ternary.h"
+#include <node_api.h>
+#include <assert.h>
 
-// cppcheck-suppress shadowFunction
-STDLIB_MATH_BASE_NAPI_MODULE_DDD_D( stdlib_base_wrap )
+/**
+* Receives JavaScript callback invocation data.
+*
+* @param env    environment under which the function is invoked
+* @param info   callback data
+* @return       Node-API value
+*/
+static napi_value addon( napi_env env, napi_callback_info info ) {
+	napi_status status;
+
+	// Get callback arguments:
+	size_t argc = 3;
+	napi_value argv[ 3 ];
+	status = napi_get_cb_info( env, info, &argc, argv, NULL, NULL );
+	assert( status == napi_ok );
+
+	// Check whether we were provided the correct number of arguments:
+	if ( argc < 3 ) {
+		status = napi_throw_error( env, NULL, "invalid invocation. Insufficient arguments." );
+		assert( status == napi_ok );
+		return NULL;
+	}
+	if ( argc > 3 ) {
+		status = napi_throw_error( env, NULL, "invalid invocation. Too many arguments." );
+		assert( status == napi_ok );
+		return NULL;
+	}
+
+	napi_valuetype vtype0;
+	status = napi_typeof( env, argv[ 0 ], &vtype0 );
+	assert( status == napi_ok );
+	if ( vtype0 != napi_number ) {
+		status = napi_throw_type_error( env, NULL, "invalid argument. First argument must be a number." );
+		assert( status == napi_ok );
+		return NULL;
+	}
+
+	napi_valuetype vtype1;
+	status = napi_typeof( env, argv[ 1 ], &vtype1 );
+	assert( status == napi_ok );
+	if ( vtype0 != napi_number ) {
+		status = napi_throw_type_error( env, NULL, "invalid argument. Second argument must be a number." );
+		assert( status == napi_ok );
+		return NULL;
+	}
+
+	napi_valuetype vtype2;
+	status = napi_typeof( env, argv[ 2 ], &vtype2 );
+	assert( status == napi_ok );
+	if ( vtype0 != napi_number ) {
+		status = napi_throw_type_error( env, NULL, "invalid argument. Third argument must be a number." );
+		assert( status == napi_ok );
+		return NULL;
+	}
+
+	double v;
+	status = napi_get_value_double( env, argv[ 0 ], &v );
+	assert( status == napi_ok );
+
+	double min;
+	status = napi_get_value_double( env, argv[ 1 ], &min );
+	assert( status == napi_ok );
+
+	double max;
+	status = napi_get_value_double( env, argv[ 2 ], &max );
+	assert( status == napi_ok );
+
+	double out = stdlib_base_wrap( v, min, max );
+
+	napi_value w;
+	status = napi_create_double( env, out, &w );
+	assert( status == napi_ok );
+
+	return w;
+}
+
+/**
+* Initializes a Node-API module.
+*
+* @param env      environment under which the function is invoked
+* @param exports  exports object
+* @return         main export
+*/
+static napi_value init( napi_env env, napi_value exports ) {
+	napi_value fcn;
+	napi_status status = napi_create_function( env, "exports", NAPI_AUTO_LENGTH, addon, NULL, &fcn );
+	assert( status == napi_ok );
+	return fcn;
+}
+
+NAPI_MODULE( NODE_GYP_MODULE_NAME, init )

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

@@ -19,8 +19,9 @@
 #include "stdlib/math/base/special/wrap.h"
 #include "stdlib/math/base/assert/is_nan.h"
 #include "stdlib/math/base/special/trunc.h"
-#include "stdlib/math/base/special/fmod.h"
-#include "stdlib/math/base/special/abs.h"
+#include <math.h>
+
+// TODO: remove <math.h> header and fmod once we have stdlib equivalent
 
 /**
 * Wraps a value on the half-open interval [min,max).
@@ -48,13 +49,13 @@ double stdlib_base_wrap( const double v, const double min, const double max ) {
 	vc = v;
 
 	// Normalize +-0 to +0...
-	if ( stdlib_base_abs( vc ) == 0.0 ) {
+	if ( vc == 0.0 ) {
 		vc = 0.0;
 	}
-	if ( stdlib_base_abs( minc ) == 0.0 ) {
+	if ( minc == 0.0 ) {
 		minc = 0.0;
 	}
-	if ( stdlib_base_abs( maxc ) == 0.0 ) {
+	if ( maxc == 0.0 ) {
 		maxc = 0.0;
 	}
 	// Simple case where value is already within range...
@@ -66,5 +67,5 @@ double stdlib_base_wrap( const double v, const double min, const double max ) {
 	if ( vc < minc ) {
 		vc += delta * ( stdlib_base_trunc( ( minc - vc ) / delta ) + 1.0 );
 	}
-	return minc + ( stdlib_base_fmod( vc - minc, delta ) );
+	return minc + ( fmod( vc - minc, delta ) );
 }