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 e11beaa..cec9e8b 100644 (file)
@@ -2,6 +2,19 @@
 
         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.
+
         PrettyPatch displays last two lines of a git-format-patch diff strangely
         https://bugs.webkit.org/show_bug.cgi?id=29317
 
index 2eaf3c1..5ab3676 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.