Skip to content

Commit

Permalink
feat: add LL_D in math/base/napi/binary
Browse files Browse the repository at this point in the history
PR-URL: #2726
Ref: #2206
Co-authored-by: Athan Reines <kgryte@gmail.com>
Reviewed-by: Athan Reines <kgryte@gmail.com> 
Signed-off-by: Athan Reines <kgryte@gmail.com>
  • Loading branch information
gunjjoshi and kgryte committed Aug 1, 2024
1 parent 43439d0 commit 46cda32
Show file tree
Hide file tree
Showing 3 changed files with 172 additions and 0 deletions.
63 changes: 63 additions & 0 deletions lib/node_modules/@stdlib/math/base/napi/binary/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -555,6 +555,46 @@ The function accepts the following arguments:
void stdlib_math_base_napi_ci_c( napi_env env, napi_callback_info info, stdlib_complex64_t (*fcn)( stdlib_complex64_t, int32_t ) );
```

#### stdlib_math_base_napi_ll_d( env, info, fcn )

Invokes a binary function accepting signed 64-bit integers and returning a double-precision floating-point number.

```c
#include <node_api.h>
#include <stdint.h>

// ...

static double fcn( const int64_t x, const int64_t y ) {
// ...
}

// ...

/**
* Receives JavaScript callback invocation data.
*
* @param env environment under which the function is invoked
* @param info callback data
* @return Node-API value
*/
napi_value addon( napi_env env, napi_callback_info info ) {
return stdlib_math_base_napi_ll_d( env, info, fcn );
}

// ...
```
The function accepts the following arguments:
- **env**: `[in] napi_env` environment under which the function is invoked.
- **info**: `[in] napi_callback_info` callback data.
- **fcn**: `[in] double (*fcn)( int64_t, int64_t )` binary function.
```c
void stdlib_math_base_napi_ll_d( napi_env env, napi_callback_info info, double (*fcn)( int64_t, int64_t ) );
```

#### STDLIB_MATH_BASE_NAPI_MODULE_DD_D( fcn )

Macro for registering a Node-API module exporting an interface for invoking a binary function accepting and returning double-precision floating-point numbers.
Expand Down Expand Up @@ -897,6 +937,29 @@ The macro expects the following arguments:

When used, this macro should be used **instead of** `NAPI_MODULE`. The macro includes `NAPI_MODULE`, thus ensuring Node-API module registration.

#### STDLIB_MATH_BASE_NAPI_MODULE_LL_D( fcn )

Macro for registering a Node-API module exporting an interface for invoking a binary function accepting signed 64-bit integers and returning a double-precision floating-point number.

```c
#include <stdint.h>

static double fcn( const int64_t x, const int64_t y ) {
// ...
}

// ...

// Register a Node-API module:
STDLIB_MATH_BASE_NAPI_MODULE_II_D( fcn );
```
The macro expects the following arguments:
- **fcn**: `double (*fcn)( int64_t, int64_t )` binary function.
When used, this macro should be used **instead of** `NAPI_MODULE`. The macro includes `NAPI_MODULE`, thus ensuring Node-API module registration.
</section>
<!-- /.usage -->
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -570,6 +570,48 @@
}; \
NAPI_MODULE( NODE_GYP_MODULE_NAME, stdlib_math_base_napi_cf_c_init )

/**
* Macro for registering a Node-API module exporting an interface invoking a binary function accepting signed 64-bit integers and returning a double-precision floating-point number.
*
* @param fcn binary function
*
* @example
* #include <stdint.h>
*
* static double fcn( const int_64 x, const int_64 y ) {
* // ...
* }
*
* // ...
*
* // Register a Node-API module:
* STDLIB_MATH_BASE_NAPI_MODULE_LL_D( fcn );
*/
#define STDLIB_MATH_BASE_NAPI_MODULE_LL_D( fcn ) \
static napi_value stdlib_math_base_napi_ll_d_wrapper( \
napi_env env, \
napi_callback_info info \
) { \
return stdlib_math_base_napi_ll_d( env, info, fcn ); \
}; \
static napi_value stdlib_math_base_napi_ll_d_init( \
napi_env env, \
napi_value exports \
) { \
napi_value fcn; \
napi_status status = napi_create_function( \
env, \
"exports", \
NAPI_AUTO_LENGTH, \
stdlib_math_base_napi_ll_d_wrapper, \
NULL, \
&fcn \
); \
assert( status == napi_ok ); \
return fcn; \
}; \
NAPI_MODULE( NODE_GYP_MODULE_NAME, stdlib_math_base_napi_ll_d_init )

/*
* If C++, prevent name mangling so that the compiler emits a binary file having undecorated names, thus mirroring the behavior of a C compiler.
*/
Expand Down Expand Up @@ -637,6 +679,11 @@ napi_value stdlib_math_base_napi_zd_z( napi_env env, napi_callback_info info, st
*/
napi_value stdlib_math_base_napi_cf_c( napi_env env, napi_callback_info info, stdlib_complex64_t (*fcn)( stdlib_complex64_t, float ) );

/**
* Invokes a binary function accepting signed 64-bit integers and returning a double-precision floating-point number.
*/
napi_value stdlib_math_base_napi_ll_d( napi_env env, napi_callback_info info, double (*fcn)( int64_t, int64_t ) );

#ifdef __cplusplus
}
#endif
Expand Down
62 changes: 62 additions & 0 deletions lib/node_modules/@stdlib/math/base/napi/binary/src/main.c
Original file line number Diff line number Diff line change
Expand Up @@ -1185,3 +1185,65 @@ napi_value stdlib_math_base_napi_cf_c( napi_env env, napi_callback_info info, st

return obj;
}

/**
* Invokes a binary function accepting signed 64-bit integers and returning a double-precision floating-point number.
*
* ## Notes
*
* - This function expects that the callback `info` argument provides access to the following JavaScript arguments:
*
* - `x`: input value.
* - `y`: input value.
*
* @param env environment under which the function is invoked
* @param info callback data
* @param fcn binary function
* @return function return value as a Node-API double-precision floating-point number
*/
napi_value stdlib_math_base_napi_ll_d( napi_env env, napi_callback_info info, double (*fcn)( int64_t, int64_t ) ) {
napi_status status;

size_t argc = 2;
napi_value argv[ 2 ];
status = napi_get_cb_info( env, info, &argc, argv, NULL, NULL );
assert( status == napi_ok );

if ( argc < 2 ) {
status = napi_throw_error( env, NULL, "invalid invocation. Must provide two numbers." );
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 ( vtype1 != napi_number ) {
status = napi_throw_type_error( env, NULL, "invalid argument. Second argument must be a number." );
assert( status == napi_ok );
return NULL;
}

int64_t x;
status = napi_get_value_int64( env, argv[ 0 ], &x );
assert( status == napi_ok );

int64_t y;
status = napi_get_value_int64( env, argv[ 1 ], &y );
assert( status == napi_ok );

napi_value v;
status = napi_create_double( env, fcn( x, y ), &v );
assert( status == napi_ok );

return v;
}

1 comment on commit 46cda32

@stdlib-bot
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage Report

Package Statements Branches Functions Lines
math/base/napi/binary $\color{green}154/154$
$\color{green}+100.00\%$
$\color{green}5/5$
$\color{green}+100.00\%$
$\color{green}1/1$
$\color{green}+100.00\%$
$\color{green}154/154$
$\color{green}+100.00\%$

The above coverage report was generated for the changes in this push.

Please sign in to comment.