ThisExpression as a property

[eliekhoury]

Shouldn't this in test.this be considered as an Identifier instead of a ThisExpression?

[LeaVerou]

@EricSmekens Actually, what's the point of ThisExpression altogether? this can be parsed just fine as an identifier. It seems a bit arbitrary to parse this specially when we don't support other related parts of JS, like e.g. the new operator.

[EricSmekens]

I agree, we should have a look into this.

[LeaVerou]

I agree, we should have a look into this.

I guess we need to start by asking @soney how he feels about us removing it, since he added it pretty early on.

[gms1]

pros for keeping ThisExpression:

1. evaluating an identifier node in the AST can be done by returning "context[node.name]". which would not work if 'this' is threated as identifier ( except you want to force AST consumers to do silly things like e.g. "context['this'] = context",... )

2. it would be nice to keep jsep's AST output similar to esprima's