2011-03-10 Caio Marcelo de Oliveira Filho <caio.oliveira@openbossa.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Mar 2011 00:56:37 +0000 (00:56 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 11 Mar 2011 00:56:37 +0000 (00:56 +0000)
        Reviewed by Adam Roben.

        Formatted Diff for attachment 23920 is mangled
        https://bugs.webkit.org/show_bug.cgi?id=21222

        The mangled diff files mentioned in the bug were using Mac line
        ending and this was causing problems for String#each_line. Now we
        normalize the line endings in patch data before processing it.

        * PrettyPatch/PrettyPatch.rb:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@80795 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Websites/bugs.webkit.org/ChangeLog
Websites/bugs.webkit.org/PrettyPatch/PrettyPatch.rb

index e11beaadc944b4ac5e77c002441bc8e63766a030..cec9e8b8459e95006a1400e9335a481b203653c8 100644 (file)
@@ -1,3 +1,16 @@
+2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
+
+        Reviewed by Adam Roben.
+
+        Formatted Diff for attachment 23920 is mangled
+        https://bugs.webkit.org/show_bug.cgi?id=21222
+
+        The mangled diff files mentioned in the bug were using Mac line
+        ending and this was causing problems for String#each_line. Now we
+        normalize the line endings in patch data before processing it.
+
+        * PrettyPatch/PrettyPatch.rb:
+
 2011-03-10  Caio Marcelo de Oliveira Filho  <caio.oliveira@openbossa.org>
 
         Reviewed by Adam Roben.
index 2eaf3c14c78a2981a06686b86d7d563a77b41b16..5ab3676d2f26305227fa2cf69fb950ac03de9a65 100644 (file)
@@ -12,6 +12,7 @@ public
     GIT_PATH = "git"
 
     def self.prettify(string)
+        string = normalize_line_ending(string)
         fileDiffs = FileDiff.parse(string)
 
         str = HEADER + "\n"
@@ -88,6 +89,10 @@ private
         Websites
     ]
 
+    def self.normalize_line_ending(s)
+        s.gsub /\r\n?/, "\n"
+    end
+
     def self.find_url_and_path(file_path)
         # Search file_path from the bottom up, at each level checking whether
         # we've found a directory we know exists in the source tree.