Support parsing of constructs like T[]?.

lib/dom/scripts/idlparser.py

 #!/usr/bin/python
 # Copyright (c) 2011, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 import re
 import subprocess
 import tempfile
 
 from pegparser import *
@@ skipping 269 matching lines @@
     def ReturnType():
       return OR(VoidType, _Type)
 
     def InterfaceType():
       return ScopedName
 
     def ArrayModifiers():
       return re.compile(r'(\[\])+')
 
     def _Type():
-      return [OR(AnyType, ObjectType, _NullableType), MAYBE(ArrayModifiers)]
-
-    def _NullableType():
-      return [OR(_IntegerType, BooleanType, OctetType, FloatType,
-             DoubleType, SequenceType, ScopedName),
-          MAYBE(Nullable)]
+      return OR(
+          [AnyType, MAYBE([ArrayModifiers, MAYBE(Nullable)])],

[podivilov, 2012/07/06 15:15:28]

Why special case for AnyType?

[Anton Muhin, 2012/07/06 15:16:44]

Per WebIDL spec any is not nullable.

[podivilov, 2012/07/06 15:24:48]

This structure is too big to parse. Please keep _NullableType function, just
remove MAYBE(Nullable) part from it.

+          [
+            OR(
+              _IntegerType, BooleanType, OctetType, FloatType, DoubleType,
+              ObjectType, SequenceType, ScopedName

[podivilov, 2012/07/06 15:15:28]

Why ObjectType became nullable?

[Anton Muhin, 2012/07/06 15:16:44]

I found no references in the spec that it shouldn't be.  If you'd like to
separate those two issues, I'll move Object to any.

+            ),
+            MAYBE(ArrayModifiers),
+            MAYBE(Nullable),
+          ])
 
     def Nullable():
       return '?'
 
     def SequenceType():
       return ['sequence', '<', Type, '>']
 
     def AnyType():
       return 'any'
 
@@ skipping 123 matching lines @@
     Args:
       content -- text to parse.
       defines -- an array of pre-processor defines.
       includePaths -- an array of path strings used by the
         gcc pre-processor.
     """
     if self._syntax == WEBKIT_SYNTAX:
       content = self._pre_process(content, defines, includePaths)
 
     return self._pegparser.parse(content)