Pre-cache PLY yacc parse table and use optimized mode (45% build time improvement)

Source/bindings/scripts/idl_reader.py

 # Copyright (C) 2013 Google Inc. All rights reserved.
 #
 # Redistribution and use in source and binary forms, with or without
 # modification, are permitted provided that the following conditions are
 # met:
 #
 #     * Redistributions of source code must retain the above copyright
 # notice, this list of conditions and the following disclaimer.
 #     * Redistributions in binary form must reproduce the above
 # copyright notice, this list of conditions and the following disclaimer
@@ skipping 13 matching lines @@
 # DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 # THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 # (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 # OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 """Read an IDL file or complete IDL interface, producing an IdlDefinitions object."""
 
 import os
 
 import blink_idl_parser
+from blink_idl_parser import BlinkIDLParser

[Nils Barth (inactive), 2014/03/05 06:11:51]

These are just some formatting fixes for consistency and legibility:
we import classes |from| modules (these are obvious as CamelCase).

 from idl_definitions import IdlDefinitions
-import idl_validator
-import interface_dependency_resolver
+from idl_validator import IDLInvalidExtendedAttributeError, IDLExtendedAttributeValidator
+from interface_dependency_resolver import InterfaceDependencyResolver
 
 
 class IdlReader(object):
     def __init__(self, interfaces_info=None, idl_attributes_filename=None, outputdir=''):
         if idl_attributes_filename:
-            self.extended_attribute_validator = idl_validator.IDLExtendedAttributeValidator(idl_attributes_filename)
+            self.extended_attribute_validator = IDLExtendedAttributeValidator(idl_attributes_filename)
         else:
             self.extended_attribute_validator = None
 
         if interfaces_info:
-            self.interface_dependency_resolver = interface_dependency_resolver.InterfaceDependencyResolver(interfaces_info, self)
+            self.interface_dependency_resolver = InterfaceDependencyResolver(interfaces_info, self)
         else:
             self.interface_dependency_resolver = None
 
-        self.parser = blink_idl_parser.BlinkIDLParser(outputdir=outputdir)
+        self.parser = BlinkIDLParser(outputdir=outputdir)
 
     def read_idl_definitions(self, idl_filename):
         """Returns an IdlDefinitions object for an IDL file, including all dependencies."""
         definitions = self.read_idl_file(idl_filename)
         if not self.interface_dependency_resolver:
             return definitions
 
         interface_name, _ = os.path.splitext(os.path.basename(idl_filename))
         self.interface_dependency_resolver.resolve_dependencies(
             definitions, interface_name)
         return definitions
 
     def read_idl_file(self, idl_filename):
         """Returns an IdlDefinitions object for an IDL file, without any dependencies."""
         ast = blink_idl_parser.parse_file(self.parser, idl_filename)
         if not ast:
             raise Exception('Failed to parse %s' % idl_filename)
         definitions = IdlDefinitions(ast)
         if not self.extended_attribute_validator:
             return definitions
 
         try:
             self.extended_attribute_validator.validate_extended_attributes(definitions)
-        except idl_validator.IDLInvalidExtendedAttributeError as error:
-            raise idl_validator.IDLInvalidExtendedAttributeError("""IDL ATTRIBUTE ERROR in file %s:
+        except IDLInvalidExtendedAttributeError as error:
+            raise IDLInvalidExtendedAttributeError(
+    """IDL ATTRIBUTE ERROR in file %s:
     %s
 If you want to add a new IDL extended attribute, please add it to
     bindings/IDLExtendedAttributes.txt
 and add an explanation to the Blink IDL documentation at:
     http://www.chromium.org/blink/webidl/blink-idl-extended-attributes
     """ % (idl_filename, str(error)))
 
         return definitions