Update based on function namespace packages

kgryte · kgryte · commit b865993e82c9 · 2022-08-17T22:36:03.000-07:00
diff --git a/lib/node_modules/@stdlib/assert/is-arrow-function/lib/main.js b/lib/node_modules/@stdlib/assert/is-arrow-function/lib/main.js
@@ -21,15 +21,13 @@
 // MODULES //
 
 var isFunction = require( '@stdlib/assert/is-function' );
+var function2string = require( '@stdlib/function/to-string' );
 
 
 // VARIABLES //
 
 var RE_ARROW_FUNCTION = /^(?:\([^)]*\) *|[^=]*)=>/;
 
-// Using `Function#toString` bypasses a value's own `toString` method to provide an extra, but not security proof, precaution to prevent a provided function from impersonating an arrow function:
-var fcn2str = Function.prototype.toString;
-
 
 // MAIN //
 
@@ -57,7 +55,7 @@ function isArrowFunction( value ) {
 	if ( !isFunction( value ) ) {
 		return false;
 	}
-	str = fcn2str.call( value );
+	str = function2string( value );
 	return RE_ARROW_FUNCTION.test( str );
 }
 
diff --git a/lib/node_modules/@stdlib/assert/is-browser/lib/global_scope.js b/lib/node_modules/@stdlib/assert/is-browser/lib/global_scope.js
@@ -16,8 +16,6 @@
 * limitations under the License.
 */
 
-/* eslint-disable no-new-func */
-
 'use strict';
 
 // VARIABLES //
@@ -40,7 +38,7 @@ function globalScope() {
 	fcn += '} catch ( err ) {';
 	fcn += 'return false;';
 	fcn += '}';
-	return (new Function( fcn ))();
+	return (new Function( fcn ))(); // eslint-disable-line stdlib/require-globals, no-new-func
 }
 
 
diff --git a/lib/node_modules/@stdlib/assert/is-class/lib/main.js b/lib/node_modules/@stdlib/assert/is-class/lib/main.js
@@ -21,15 +21,13 @@
 // MODULES //
 
 var isFunction = require( '@stdlib/assert/is-function' );
+var function2string = require( '@stdlib/function/to-string' );
 
 
 // VARIABLES //
 
 var RE_CLASS = /^\s*class\s/;
 
-// Using `Function#toString` bypasses a value's own `toString` method to provide an extra, but not security proof, precaution to prevent a provided function from impersonating a class:
-var fcn2str = Function.prototype.toString;
-
 
 // MAIN //
 
@@ -56,7 +54,7 @@ function isClass( value ) {
 	if ( !isFunction( value ) ) {
 		return false;
 	}
-	str = fcn2str.call( value );
+	str = function2string( value );
 	return RE_CLASS.test( str );
 }
 
diff --git a/lib/node_modules/@stdlib/assert/is-native-function/lib/main.js b/lib/node_modules/@stdlib/assert/is-native-function/lib/main.js
@@ -22,12 +22,7 @@
 
 var isFunction = require( '@stdlib/assert/is-function' );
 var reNativeFunction = require( '@stdlib/regexp/native-function' );
-
-
-// VARIABLES //
-
-// Using `Function#toString` bypasses a value's own `toString` method to provide an extra, but not security proof, precaution to prevent a provided function from impersonating a native function:
-var fcn2str = Function.prototype.toString;
+var function2string = require( '@stdlib/function/to-string' );
 
 
 // MAIN //
@@ -52,7 +47,7 @@ var fcn2str = Function.prototype.toString;
 function isNativeFunction( value ) {
 	return (
 		isFunction( value ) &&
-		reNativeFunction.REGEXP.test( fcn2str.call( value ) )
+		reNativeFunction.REGEXP.test( function2string( value ) )
 	);
 }
 
diff --git a/lib/node_modules/@stdlib/assert/is-node/lib/global_scope.js b/lib/node_modules/@stdlib/assert/is-node/lib/global_scope.js
@@ -38,7 +38,7 @@ function globalScope() {
 	fcn += '} catch ( err ) {';
 	fcn += 'return false;';
 	fcn += '}';
-	return (new Function( fcn ))(); // eslint-disable-line no-new-func
+	return (new Function( fcn ))(); // eslint-disable-line no-new-func, stdlib/require-globals
 }
 
 
diff --git a/lib/node_modules/@stdlib/assert/is-plain-object/test/test.js b/lib/node_modules/@stdlib/assert/is-plain-object/test/test.js
@@ -23,6 +23,7 @@
 var tape = require( 'tape' );
 var proxyquire = require( 'proxyquire' );
 var defineProperty = require( '@stdlib/utils/define-property' );
+var Function = require( '@stdlib/function/ctor' );
 var isPlainObject = require( './../lib' );
 
 
diff --git a/lib/node_modules/@stdlib/assert/is-web-worker/lib/global_scope.js b/lib/node_modules/@stdlib/assert/is-web-worker/lib/global_scope.js
@@ -16,8 +16,6 @@
 * limitations under the License.
 */
 
-/* eslint-disable no-new-func */
-
 'use strict';
 
 // VARIABLES //
@@ -40,7 +38,7 @@ function globalScope() {
 	fcn += '} catch ( err ) {';
 	fcn += 'return false;';
 	fcn += '}';
-	return (new Function( fcn ))();
+	return (new Function( fcn ))(); // eslint-disable-line stdlib/require-globals, no-new-func
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ function globalScope() {`
`38`	`38`	`fcn += '} catch ( err ) {';`
`39`	`39`	`fcn += 'return false;';`
`40`	`40`	`fcn += '}';`
`41`		`- return (new Function( fcn ))(); // eslint-disable-line no-new-func`
	`41`	`+ return (new Function( fcn ))(); // eslint-disable-line no-new-func, stdlib/require-globals`
`42`	`42`	`}`
`43`	`43`
`44`	`44`