search

jonthegeek / stbl

Stabilize Function Arguments
https://jonthegeek.github.io/stbl/
Other
14 stars 0 forks source link

Spike: Can I unite to_* and stabilize_*? #46

Closed jonthegeek closed 1 year ago

jonthegeek commented 1 year ago

If I can make stabilize* fast for defaults, I don't need multiple functions. In any case, I think I want everything to be named stabilize. Maybe if I need more args I use stabilize__more?

jonthegeek commented 1 year ago

The speed hit appears to be negligible, and it's actually considerably faster to check NULLs, which might prove useful. Fastest is with a single shared function that does NOT use S3. Might be possible to make it even faster by moving the S3 stuff inside the length check; putting the length check up front appears to be on of our biggest gains.