-
Notifications
You must be signed in to change notification settings - Fork 6
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
Potential hazard #10
Comments
This doesn't seem ambiguous to me. The first ternary is As to your suggestion, remember that const ? 1.:get() |
I think we could parse .: only when <value>::<symbol> pattern is encountered and your example functions as it is now, and 1 ? 1..:1 is a syntax error |
Is |
@MadProbe This issue already be listed in the docs (https://github.com/tc39/proposal-extensions/blob/master/docs/syntax.md#syntax-ambiguity-of-abc) though I'm sorry it not listed in the repo readme page. There are many possible solutions,
@justinmchase Yes, it's valid. It just use |
(To avoid this hazard i think we can change ':' ext name operator to '.:', which seems to have no hazard at all)
The text was updated successfully, but these errors were encountered: