archive-gh-me/mathjs
1
0
Fork 0
mirror of https://github.com/josdejong/mathjs.git synced 2026-01-25 15:07:57 +00:00
Code Issues Projects Releases Wiki Activity

Add comments and simplify the code with regular expressions.

Browse Source
This commit is contained in:
David Contreras 2025-03-07 19:47:23 -06:00
parent 3700e9b057
commit 3c48eebde9
1 changed files with 22 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
src/utils/optimizeCallback.js
View File

@ -55,18 +55,30 @@ function _findSingleSignatureWithArity (callback, arity) {
}
}
/**
* Determines if a given callback function is unary (i.e., takes exactly one argument).
*
* This function checks the following conditions to determine if the callback is unary:
* 1. The callback function should have exactly one parameter.
* 2. The callback function should not use the `arguments` object.
* 3. The callback function should not use rest parameters (`...`).
* If in doubt, this function shall return `false` to be safe
*
* @param {Function} callback - The callback function to be checked.
* @returns {boolean} - Returns `true` if the callback is unary, otherwise `false`.
*/
function _findIfCallbackIsUnary (callback) {
const callbackStr = callback.toString()
const firstParenthesisIndex = callbackStr.indexOf('(')
const firstClosingParenthesisIndex = callbackStr.indexOf(')', firstParenthesisIndex)
const paramsStr = callbackStr.slice(firstParenthesisIndex + 1, firstClosingParenthesisIndex)
if (callback.length !== 1) return false
if (callback.length === 1) {
if (/arguments/.test(callbackStr)) return false
if (/\.\.\.\w*/.test(paramsStr)) return false
return true
}
return false
const callbackStr = callback.toString()
// Check if the callback function uses `arguments`
if (/arguments/.test(callbackStr)) return false
// Extract the parameters of the callback function
const paramsStr = callbackStr.match(/\(.*?\)/)
// Check if the callback function uses rest parameters
if (/\.\.\./.test(paramsStr)) return false
return true
}
function _findNumberOfArgumentsTyped (callback, value, index, array) {