5 ms·
Waste of memory by creating new function objects on each render.
by bacro 5y ago
Waste of memory by creating new function objects on each render.
- symlinkk 5y agoYeah probably best to move this to a function outside the render loop with useCallback
- beaconstudios 5y agoMicro-optimisations aren't useful. If you get to the point where the performance cost of inline functions is a bottleneck, you probably shouldn't be using React at all.
- bacro 5y agoMaybe, but I just prefer to have a named function defined with a useCallback and just call it.
- beaconstudios 5y agoThat's fine, but that's more of a stylistic preference than a performance concern.
- erikpukinskis 5y agoWon’t V8 just optimize this anyway? I doubt there’s any real difference in memory usage.
- bacro 5y agoI am not sure, it has a closure for the 'error' local variable.