fix speed regression in mdiff caused by line split bugfix. 0.9
authorVadim Gelfer <vadim.gelfer@gmail.com>
Wed, 10 May 2006 13:39:12 -0700
changeset 2251 35fb62a3a673
parent 2250 45aef5ddcdbe
child 2252 ba7afc7dd901
fix speed regression in mdiff caused by line split bugfix.
mercurial/mdiff.py
--- a/mercurial/mdiff.py	Wed May 10 11:10:18 2006 -0700
+++ b/mercurial/mdiff.py	Wed May 10 13:39:12 2006 -0700
@@ -9,20 +9,15 @@
 import struct, bdiff, util, mpatch
 demandload(globals(), "re")
 
-
-def splitnewlines(text, keepends=False):
+def splitnewlines(text):
     '''like str.splitlines, but only split on newlines.'''
-    i = 0
-    lines = []
-    while True:
-        n = text.find('\n', i)
-        if n == -1:
-            last = text[i:]
-            if last:
-                lines.append(last)
-            return lines
-        lines.append(text[i:keepends and n+1 or n])
-        i = n + 1
+    lines = [l + '\n' for l in text.split('\n')]
+    if lines:
+        if lines[-1] == '\n':
+            lines.pop()
+        else:
+            lines[-1] = lines[-1][:-1]
+    return lines
 
 def unidiff(a, ad, b, bd, fn, r=None, text=False,
             showfunc=False, ignorews=False):
@@ -33,7 +28,7 @@
     if not text and (util.binary(a) or util.binary(b)):
         l = ['Binary file %s has changed\n' % fn]
     elif not a:
-        b = splitnewlines(b, keepends=True)
+        b = splitnewlines(b)
         if a is None:
             l1 = "--- %s\t%s\n" % ("/dev/null", epoch)
         else:
@@ -42,7 +37,7 @@
         l3 = "@@ -0,0 +1,%d @@\n" % len(b)
         l = [l1, l2, l3] + ["+" + e for e in b]
     elif not b:
-        a = splitnewlines(a, keepends=True)
+        a = splitnewlines(a)
         l1 = "--- %s\t%s\n" % ("a/" + fn, ad)
         if b is None:
             l2 = "+++ %s\t%s\n" % ("/dev/null", epoch)
@@ -51,8 +46,8 @@
         l3 = "@@ -1,%d +0,0 @@\n" % len(a)
         l = [l1, l2, l3] + ["-" + e for e in a]
     else:
-        al = splitnewlines(a, keepends=True)
-        bl = splitnewlines(b, keepends=True)
+        al = splitnewlines(a)
+        bl = splitnewlines(b)
         l = list(bunidiff(a, b, al, bl, "a/" + fn, "b/" + fn,
                           showfunc=showfunc, ignorews=ignorews))
         if not l: return ""