Title: [131798] trunk/Tools
Revision
131798
Author
[email protected]
Date
2012-10-18 13:58:24 -0700 (Thu, 18 Oct 2012)

Log Message

Teach prepare-ChangeLog how to handle non-function properties in _javascript_ prototypes.

https://bugs.webkit.org/show_bug.cgi?id=99747

Reviewed by Joseph Pecoraro.

* Scripts/prepare-ChangeLog:
(get_function_line_ranges_for_javascript): Clear currentIdentifiers when comma is encountered
so it does not tack those identifiers onto the next changed function.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (131797 => 131798)


--- trunk/Tools/ChangeLog	2012-10-18 20:24:39 UTC (rev 131797)
+++ trunk/Tools/ChangeLog	2012-10-18 20:58:24 UTC (rev 131798)
@@ -1,3 +1,15 @@
+2012-10-18  Timothy Hatcher  <[email protected]>
+
+        Teach prepare-ChangeLog how to handle non-function properties in _javascript_ prototypes.
+
+        https://bugs.webkit.org/show_bug.cgi?id=99747
+
+        Reviewed by Joseph Pecoraro.
+
+        * Scripts/prepare-ChangeLog:
+        (get_function_line_ranges_for_javascript): Clear currentIdentifiers when comma is encountered
+        so it does not tack those identifiers onto the next changed function.
+
 2012-10-18  James Simonsen  <[email protected]>
 
         Add a script for exporting http/tests/w3c/webperf/submission to W3C

Modified: trunk/Tools/Scripts/prepare-ChangeLog (131797 => 131798)


--- trunk/Tools/Scripts/prepare-ChangeLog	2012-10-18 20:24:39 UTC (rev 131797)
+++ trunk/Tools/Scripts/prepare-ChangeLog	2012-10-18 20:58:24 UTC (rev 131798)
@@ -1218,7 +1218,7 @@
         }
 
         # Find function names.
-        while (m-(\w+|[(){}=:;])-g) {
+        while (m-(\w+|[(){}=:;,])-g) {
             # Open parenthesis.
             if ($1 eq '(') {
                 $parenthesesDepth++;
@@ -1259,8 +1259,8 @@
                 next;
             }
 
-            # Semicolon.
-            if ($1 eq ';') {
+            # Semicolon or comma.
+            if ($1 eq ';' or $1 eq ',') {
                 @currentIdentifiers = ();
                 next;
             }
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to