Update the grammar to prevent an extension from having the name type (

#3431)
dart-lang · Oct 31, 2023 · 8a4003d · 8a4003d
1 parent c214fd9
commit 8a4003d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 5 deletions.
diff --git a/specification/dart.sty b/specification/dart.sty
@@ -62,6 +62,7 @@
 \def\THROW{\keyword{throw}}
 \def\TRUE{\keyword{true}}
 \def\TRY{\keyword{try}}
+\def\TYPE{\keyword{type}}
 \def\VAR{\keyword{var}}
 \def\VOID{\keyword{void}}
 \def\WHILE{\keyword{while}}

diff --git a/specification/dartLangSpec.tex b/specification/dartLangSpec.tex
@@ -41,6 +41,11 @@
 % version of the language which will actually be specified by the next stable
 % release of this document.
 %
+% Oct 2023
+% - Introduce the rule that an `extension` declaration cannot have the name
+%   `type`. This is needed in order to disambiguate an `extension type`
+%   named `on`.
+%
 % Aug 2023
 % - Correct text about built-in identifier error and turn it into commentary
 %   (the normative text is in grammar rules using `typeIdentifier`), and
@@ -6083,8 +6088,8 @@ \section{Extensions}
 and whether the invocation satisfies several other requirements.
 
 \begin{grammar}
-<extensionDeclaration> ::=
-  \gnewline{} \EXTENSION{} <typeIdentifier>? <typeParameters>? \ON{} <type>
+<extensionDeclaration> ::= \gnewline{}
+  \EXTENSION{} <typeIdentifierNotType>? <typeParameters>? \ON{} <type>
   \gnewline{} `\{' (<metadata> <classMemberDeclaration>)* `\}'
 \end{grammar}
 
@@ -17154,10 +17159,13 @@ \subsection{Identifier Reference}
   \alt <BUILT\_IN\_IDENTIFIER>
   \alt <OTHER\_IDENTIFIER>
 
-<typeIdentifier> ::= <IDENTIFIER>
-  \alt <OTHER\_IDENTIFIER>
+<typeIdentifierNotType> ::= <IDENTIFIER>
+  \alt <OTHER\_IDENTIFIER\_NOT\_TYPE>
   \alt \DYNAMIC{}
 
+<typeIdentifier> ::= <typeIdentifierNotType>
+  \alt \TYPE{}
+
 <qualifiedName> ::= <typeIdentifier> `.' <identifier>
   \alt <typeIdentifier> `.' <typeIdentifier> `.' <identifier>
 
@@ -17168,10 +17176,13 @@ \subsection{Identifier Reference}
     \LIBRARY{} | \MIXIN{} | \OPERATOR{}
   \alt\hspace{-3mm} \PART{} | \REQUIRED{} | \SET{} | \STATIC{} | \TYPEDEF{}
 
-<OTHER\_IDENTIFIER> ::= \gnewline{}
+<OTHER\_IDENTIFIER\_NOT\_TYPE> ::= \gnewline{}
   \ASYNC{} | \HIDE{} | \OF{} | \ON{} | \SHOW{} | \SYNC{} |
   \AWAIT{} | \YIELD{}
 
+<OTHER\_IDENTIFIER> ::= \gnewline{}
+  <OTHER\_IDENTIFIER\_NOT\_TYPE> | \TYPE{}
+
 <IDENTIFIER\_NO\_DOLLAR> ::= <IDENTIFIER\_START\_NO\_DOLLAR>
   \gnewline{} <IDENTIFIER\_PART\_NO\_DOLLAR>*