memoized function has no length

[ChristopherChudzicki]

Memoizing a function sets its length property to zero:

import memoizeOne from 'memoize-one'
f = (a, b, c) => a*b*c
memoF = memoizeOne(f)
console.log(f.length) // 3
console.log(memoF.length) // 0

This is not really a big deal: length is a configurable property, so if you care about f.length you can just set it manually afterwards using Object.definiteProperty.

Should memoizeOne automatically configure the memoized functions length property?

[alexreardon]

At one point we tried adding this, but it broke ie11 #18

We could wrap the assignment in a try / catch...

I am not sure what the best path forward is. /cc @theKashey

[theKashey]

We shall just redo #18

[alexreardon]

Maybe if we wrap it in a try catch? My issue is that people might expect the function to have a length in ie11 - but it wont

[theKashey]

There is 2 ways to define length

• as we did in #18
• using util-arity, which would use eval to create function with expected number of arguments.