tc39 / Function-prototype-toString-revision

:fishing_pole_and_fish: ECMA-262 proposal to update Function.prototype.toString

Home Page:https://tc39.github.io/Function-prototype-toString-revision

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

CreateDynamicFunction lacks line break after opening brace

bterlson opened this issue · comments

All implementations today add a line break after the opening bracket, but this specification says we should not include it. Was this deliberate? I think the line break aids in readability but I care only a tiny amount.

It was deliberate because there is no need for it. Also, I don't think adding it matters for compatibility since engines differ significantly here.

Closing until I receive further feedback.

I agree that it aids readability, and the fact that all engines today include it is a good indication it should stay.

I've noticed a more peculiar issue: a spurious line break is currently inserted before the closing parenthesis at the end of the arguments list. I've considered that as just a typo, that line break should appear after the opening brace. See #13 (that I opened before noticing the present issue).