Title: [156228] trunk/Source/WebCore
Revision
156228
Author
[email protected]
Date
2013-09-21 06:45:10 -0700 (Sat, 21 Sep 2013)

Log Message

Handle windows lineendings in makeprop.pl after r155511
https://bugs.webkit.org/show_bug.cgi?id=121740

Reviewed by Ryosuke Niwa.

Check for an optinal \r at the end of a line in regular expressions
to make it work again on native windows machines which use CRLF.

* css/makeprop.pl:

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (156227 => 156228)


--- trunk/Source/WebCore/ChangeLog	2013-09-21 13:05:56 UTC (rev 156227)
+++ trunk/Source/WebCore/ChangeLog	2013-09-21 13:45:10 UTC (rev 156228)
@@ -1,3 +1,15 @@
+2013-09-21  Patrick Gansterer  <[email protected]>
+
+        Handle windows lineendings in makeprop.pl after r155511
+        https://bugs.webkit.org/show_bug.cgi?id=121740
+
+        Reviewed by Ryosuke Niwa.
+
+        Check for an optinal \r at the end of a line in regular expressions
+        to make it work again on native windows machines which use CRLF.
+
+        * css/makeprop.pl:
+
 2013-09-21  Andreas Kling  <[email protected]>
 
         Re-inline some hot ElementData functions.

Modified: trunk/Source/WebCore/css/makeprop.pl (156227 => 156228)


--- trunk/Source/WebCore/css/makeprop.pl	2013-09-21 13:05:56 UTC (rev 156227)
+++ trunk/Source/WebCore/css/makeprop.pl	2013-09-21 13:45:10 UTC (rev 156228)
@@ -47,7 +47,7 @@
   next if (/^#/);
 
   # Input may use a different EOL sequence than $/, so avoid chomp.
-  $_ =~ s/\s*\[(.+?)\]$//;
+  $_ =~ s/\s*\[(.+?)\]\r?$//;
   my @options = ();
   if ($1) {
     @options = split(/\s*,\s*/, $1);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to