sebastienros / jint

Javascript Interpreter for .NET
BSD 2-Clause "Simplified" License
4.11k stars 563 forks source link

Is the tail-call feature really needed? #1925

Closed ChrML closed 3 months ago

ChrML commented 3 months ago

I see that the README doc marks tail-calls red as one of the two missing features to comply with ES5. However it seems almost no JS engines today support it, and it was removed from V8. So is there really a need to add this complexity to Jint?

It seems status quoi is that as a JS developer, even tho it's been since ES5, you can't really rely your JS code on the engine having tail-calls. And there's lots of arguments about it not being so useful .. / narrow use-case / doesn't help performance.

Question is, will it be implemented, or should the readme just be changed to indicate that there is no intension to implement it?