[bacro]

Waste of memory by creating new function objects on each render.

[beaconstudios]

Micro-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]

Maybe, but I just prefer to have a named function defined with a useCallback and just call it.

[beaconstudios]

That's fine, but that's more of a stylistic preference than a performance concern.

[erikpukinskis]

Won’t V8 just optimize this anyway? I doubt there’s any real difference in memory usage.

[bacro]

I am not sure, it has a closure for the 'error' local variable.

[symlinkk]

Yeah probably best to move this to a function outside the render loop with useCallback