Compiler produces error on variable named 'final'

[DartBot]

This issue was originally filed by [email protected]

---

What steps will reproduce the problem?

1. Try running following code:
   main() {
     var final = 'final';
     print('${final}');
   }
2. According to spec (v 0.01), section 14.1.1 'final' is not reserved word, so the code is valid according to spec.
   3.
   I would expect the change in spec according to current implementation (e.g. Dart.g assumes 'final' to be keyword).
   Product used http://www.dartlang.org/ 2011-10-12

[DartBot]

This comment was originally written by [email protected]

---

Gilad, is it correct that final should be a pseudokeyword and not a keyword?

---

Set owner to @gbracha.
Added Area-Compiler, Triaged labels.

[gbracha]

The rule is that something is a reserved word if it is a reserved word in
JS. Otherwise, it's a built-in identifier. Since JS doesn't have final, it
should be a built-in identifier (aka pseudo-keyword).

[DartBot]

This comment was originally written by [email protected]

---

Set owner to [email protected].
Added Accepted label.

[DartBot]

This comment was originally written by [email protected]

---

IMO making final as pseudokeyword allows further syntax abuse (similar to http://try.dartlang.org/s/H7AZ ) e.g. currently not allowed code becomes valid and very confusing:

class final {}
class abstract extends final {}
final class = new abstract();

[DartBot]

This comment was originally written by [email protected]

---

The VM doesn't support final either, and it isn't in the grammar's identifier production.

Are we sure that the spec is correct and that final should be acceptable as an identifier?

---

cc @gbracha.
cc @kasperl.

[gbracha]

We were just chatting about this yesterday. We will be promoting final, is
and a number of others to reserved word status in the next rev of the spec.

[DartBot]

This comment was originally written by [email protected]

---

Closing the bug if final will become a reserved word, as dartc already implements it that way.

---

Added WontFix label.

[DartBot]

This comment was originally written by [email protected]

---

Looking forward to see 'abstract', 'class', 'interface', 'extends', 'implements', 'static' becoming reserved keywords in compiler too