re-frame-10x icon indicating copy to clipboard operation
re-frame-10x copied to clipboard

Improve performance of string handling

Open danielcompton opened this issue 8 years ago • 1 comments

The string.split and string/includes? calls get very expensive when there are a large number of them. We should look into a cheaper way of doing this, perhaps by creating another Reagent function which returns a vector of the component hierarchy, rather than a string.

danielcompton avatar Jun 29 '17 06:06 danielcompton

We have a memoize over the component-path which amortises the cost, but it would probably be better to operate over data instead of splitting strings if possible.

danielcompton avatar Apr 09 '18 22:04 danielcompton