Title: [147934] trunk/Source/_javascript_Core
Revision
147934
Author
[email protected]
Date
2013-04-08 10:51:01 -0700 (Mon, 08 Apr 2013)

Log Message

Allow KeywordLookupGenerator.py to work on Windows with Windows style line endings
https://bugs.webkit.org/show_bug.cgi?id=63234

Patch by Justin Haygood <[email protected]> on 2013-04-08
Reviewed by Oliver Hunt.

* KeywordLookupGenerator.py:
(parseKeywords):

Modified Paths

Diff

Modified: trunk/Source/_javascript_Core/ChangeLog (147933 => 147934)


--- trunk/Source/_javascript_Core/ChangeLog	2013-04-08 17:48:28 UTC (rev 147933)
+++ trunk/Source/_javascript_Core/ChangeLog	2013-04-08 17:51:01 UTC (rev 147934)
@@ -1,3 +1,13 @@
+2013-04-08  Justin Haygood  <[email protected]>
+
+        Allow KeywordLookupGenerator.py to work on Windows with Windows style line endings
+        https://bugs.webkit.org/show_bug.cgi?id=63234
+
+        Reviewed by Oliver Hunt.
+
+        * KeywordLookupGenerator.py:
+        (parseKeywords):
+
 2013-04-08  Filip Pizlo  <[email protected]>
 
         REGRESSION(r146669): Assertion hit in JSC::DFG::SpeculativeJIT::fillSpeculateCell() running webgl tests

Modified: trunk/Source/_javascript_Core/KeywordLookupGenerator.py (147933 => 147934)


--- trunk/Source/_javascript_Core/KeywordLookupGenerator.py	2013-04-08 17:48:28 UTC (rev 147933)
+++ trunk/Source/_javascript_Core/KeywordLookupGenerator.py	2013-04-08 17:51:01 UTC (rev 147934)
@@ -68,11 +68,16 @@
 
 
 def parseKeywords(keywordsText):
+
+    if sys.platform == "cygwin":
+        keywordsText = keywordsText.replace("\r\n", "\n")
+
     lines = keywordsText.split("\n")
     lines = [line.split("#")[0] for line in lines]
     lines = [line for line in lines if (not allWhitespace(line))]
     name = lines[0].split()
     terminator = lines[-1]
+
     if not name[0] == "@begin":
         raise Exception("expected description beginning with @begin")
     if not terminator == "@end":
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to