Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upNote on usage of the name NewTarget #177
Comments
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
rwaldron
Nov 15, 2015
Contributor
I think you're looking at an outdated version of the specification? The section you're referring to is 15.2.1.1. NewTarget (italics) in the context of static semantics, can only ever refer to the grammar:
NewTarget :
new . target
|
I think you're looking at an outdated version of the specification? The section you're referring to is 15.2.1.1. NewTarget (italics) in the context of static semantics, can only ever refer to the grammar: NewTarget : |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
yasushikuno
commented
Nov 15, 2015
|
[JP22] Thank you for reply. I will look for current version. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
bterlson
Nov 19, 2015
Member
Italics (emu-nt in the source text) refers to a particular non-terminal of the grammar.
(By the way, I think you can just prefix your title with JP22 rather than each comment :))
|
Italics (emu-nt in the source text) refers to a particular non-terminal of the grammar. (By the way, I think you can just prefix your title with JP22 rather than each comment :)) |
yasushikuno commentedNov 15, 2015
[JP22] 15.2.2.1 Bullet 6 says "It is a Syntax Error if MouduleItemList Contains NewTarget" and I was first puzzled on what this "NewTarget" means. Isn't it more kind to just use "new . target" (in typeface)? The problem stems from the fact that identifier "NewTarget" is used in multiple ways throughout the document, and it might be difficult to resolve. However, I guess the above change will resolve the problem here. (I also noticed that a period is missing at the end of the sentence.)