You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
const selector = defaultMemoize(function () {
const params = []
const length = dependencies.length
for (let i = 0; i < length; i++) {
// apply arguments instead of spreading and mutate a local list of params for performance.
params.push(dependencies[i].apply(null, arguments))
}
// apply arguments instead of spreading for performance.
return memoizedResultFunc.apply(null, params)
})
Why should this not use the provided memoize? All tests pass with the change, but any other option to allow setting this will be very helpful. another helpful addition is passing memoizeOptions here as well, but this does break a test case which will have to be updated, although I do believe this is the expected behavior as an end user.
from
index.js:68
:Why should this not use the provided
memoize
? All tests pass with the change, but any other option to allow setting this will be very helpful. another helpful addition is passingmemoizeOptions
here as well, but this does break a test case which will have to be updated, although I do believe this is the expected behavior as an end user.#297
The text was updated successfully, but these errors were encountered: