flatstr

flatstr copied to clipboard

Added a type definition for flatstr

1

pan93412

Update benchmarks, possibly with node versions

1

I have found with node 12.22.1 that flatstr's performance is not that different from node now and in some cases is running slower. Is it getting time to retire flatstr...

mattbishop

Is the flattened representation itself not actually verified in tests?

2

You could check the result by using `string->isFlat()` in a native C++ (not N-API) addon used specifically for testing. It's undocumented, but exposed publicly. This would make it way less...

dead-claudia

Add benchmark

4

Add some tests to show how much the performance improves/degrees applying multiple time `flatstr`. The result is not so obvious!

allevo