-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Function.prototype.toString
revision
#664
Comments
Let's leave this open until the proposal has been merged into the main spec, so that people looking at web reality can get a better view. |
I don't think @michaelficarra 's proposal reflects current web reality, as it is not what any particular browser currently ships, as far as I know. |
My understanding is that there is a significant percentage that intersects web reality, and for the rest, there isn't cross-browser agreement. @michaelficarra could you post/point to the data from your presentation about that? |
Yeah, I don't think that the Function.prototype.toString proposal differs from web reality. It just makes even stronger guarantees than both what was specified and what exists in implementations. |
By the way, this is why we have stage 3. We're waiting on implementation feedback to confirm that this is both technically feasible and web compatible before moving it to stage 4. |
fwiw, I believe the proposal disagrees from web reality in its treatment of source text occurring prior to the formal parameters in normal function expressions and declarations. |
@bakkot That's right. I should have said that I believe it is web compatible. So technically different, but not in a way that will break a meaningful number of web pages. But we'll have to wait for implementation feedback (hint, nudge, wink, etc.) before we know this to be true. |
Sure. Just wanted to clarify that "for the rest, there isn't cross-browser agreement" wasn't quite right. |
https://github.com/tc39/Function-prototype-toString-revision by @michaelficarra
Please tag this issue with the “web reality” label. Thanks!
The text was updated successfully, but these errors were encountered: