Title: [129769] trunk/Source/WebCore
Revision
129769
Author
a...@chromium.org
Date
2012-09-27 09:06:48 -0700 (Thu, 27 Sep 2012)

Log Message

Remove unused regular expressions from IDLStructure.pm
https://bugs.webkit.org/show_bug.cgi?id=97790

Reviewed by Kentaro Hara.

After http://trac.webkit.org/changeset/129723 these regular expressions are no longer used.

No new tests, run-binding-tests generates the same output.

* bindings/scripts/IDLStructure.pm:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (129768 => 129769)


--- trunk/Source/WebCore/ChangeLog	2012-09-27 15:54:06 UTC (rev 129768)
+++ trunk/Source/WebCore/ChangeLog	2012-09-27 16:06:48 UTC (rev 129769)
@@ -1,3 +1,16 @@
+2012-09-27  Erik Arvidsson  <a...@chromium.org>
+
+        Remove unused regular expressions from IDLStructure.pm
+        https://bugs.webkit.org/show_bug.cgi?id=97790
+
+        Reviewed by Kentaro Hara.
+
+        After http://trac.webkit.org/changeset/129723 these regular expressions are no longer used.
+
+        No new tests, run-binding-tests generates the same output.
+
+        * bindings/scripts/IDLStructure.pm:
+
 2012-09-27  Tommy Widenflycht  <tom...@google.com>
 
         MediaStream API: Enhance MediaConstraints to make it easier to get the constraint data

Modified: trunk/Source/WebCore/bindings/scripts/IDLStructure.pm (129768 => 129769)


--- trunk/Source/WebCore/bindings/scripts/IDLStructure.pm	2012-09-27 15:54:06 UTC (rev 129768)
+++ trunk/Source/WebCore/bindings/scripts/IDLStructure.pm	2012-09-27 16:06:48 UTC (rev 129769)
@@ -78,39 +78,4 @@
     extendedAttributes => '$', # Extended attributes
 });
 
-# Helpers
-our $idlId = '[a-zA-Z0-9]';        # Generic identifier
-our $idlIdNs = '[a-zA-Z0-9:]';      # Generic identifier including namespace
-our $idlIdNsList = '[a-zA-Z0-9:,\ ]';  # List of Generic identifiers including namespace
-
-our $idlType = '[a-zA-Z0-9_]';      # Generic type/"value string" identifier
-# Match a string value, a hexadecimal number, or an integral number.
-# Note: some of the characters that are allowed in the string value may not be allowed by
-# interfaceSelector.
-our $constValue = '("[^"\r\n]*")|(0[xX][a-fA-F0-9]+)|(-?[0-9]*)';
-our $idlDataType = '[a-zA-Z0-9\ ]';   # Generic data type identifier
-
-# Magic IDL parsing regular expressions
-my $supportedTypes = "((?:(?:unsigned )?(?:int|short|(?:long )?long)|(?:$idlIdNs*))(?:\\[\\]|<(?:$idlIdNsList*)>)?)";
-my $supportedTypeSuffix = "(\\?)?";
-
-# Special IDL notations. This regular _expression_ extracts the string between the first [ and its corresponding ].
-our $extendedAttributeSyntax = qr/\[[^\[\]]*(?:(??{$IDLStructure::extendedAttributeSyntax})[^\[\]]*)*\]/x; # Used for extended attributes
-
-# Regular _expression_ based IDL 'syntactical tokenizer' used in the IDLParser
-our $moduleSelector = 'module\s*(' . $idlId . '*)\s*{';
-our $moduleNSSelector = 'module\s*(' . $idlId . '*)\s*\[ns\s*(' . $idlIdNs . '*)\s*(' . $idlIdNs . '*)\]\s*;';
-our $constantSelector = '(' . $extendedAttributeSyntax . ' )?const\s+' . $supportedTypes . '\s*(' . $idlType . '*)\s*=\s*(' . $constValue . ')';
-our $raisesSelector = 'raises\s*\((' . $idlIdNsList . '*)\s*\)';
-our $getterRaisesSelector = '\bgetter\s+raises\s*\((' . $idlIdNsList . '*)\s*\)';
-our $setterRaisesSelector = '\bsetter\s+raises\s*\((' . $idlIdNsList . '*)\s*\)';
-
-our $typeNamespaceSelector = '((?:' . $idlId . '*::)*)\s*(' . $idlDataType . '*)';
-
-our $interfaceSelector = '(interface|exception)\s*((?:' . $extendedAttributeSyntax . ' )?)(' . $idlIdNs . '*)\s*(?::(\s*[^{]*))?{([-a-zA-Z0-9_"=\s(),;:\[\]<>&\|?]*)';
-our $interfaceMethodSelector = '\s*((?:' . $extendedAttributeSyntax . ' )?)(static\s+)?' . $supportedTypes . '\s*(' . $idlIdNs . '*)\s*\(\s*([a-zA-Z0-9:\s,=\[\]<>?]*)';
-our $interfaceParameterSelector = '(in|out)\s*((?:' . $extendedAttributeSyntax . ' )?)' . $supportedTypes . $supportedTypeSuffix . '\s*(' . $idlIdNs . '*)';
-
-our $interfaceAttributeSelector = '\s*(static\s+)?(readonly attribute|attribute)\s*(' . $extendedAttributeSyntax . ' )?' . $supportedTypes . '\s*(' . $idlType . '*)';
-
 1;
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to