Repository navigation
Should we allow super [ Expression ] in PatternMatchingMemberExpression? #331
Description
Here is the current PatternMatchingMemberExpression production:
PatternMatchingMemberExpression[Yield, Await] :
this
MetaProperty
IdentifierReference[?Yield, ?Await]
super . IdentifierName
PatternMatchingMemberExpression[?Yield, ?Await] . IdentifierName
PatternMatchingMemberExpression[?Yield, ?Await] . PrivateIdentifier
PatternMatchingMemberExpression[?Yield, ?Await] [ Expression[+In, ?Yield, ?Await] ]
As we can see, the following examples are valid PatternMatchingMemberExpression:
this[expr]super.prop[expr]import.meta[expr]
However, this production does not cover super [ Expression[+In, ?Yield, ?Await] ], should we also include it?
If yes we can replace super . IdentifierName by the SuperProperty production, which already covers super[expr].
Activity
ljharb commentedon Jul 6, 2024
My expectation is that we’d match decorators here.
Jack-Works commentedon Jul 7, 2024
looks like I forgot that production
Jack-Works commentedon Jul 7, 2024
DecoratorMemberExpression is too limited. It doesn't include
thisMetaPropertyorsuper.*, it also includes(Expression)we don't want.