HTML Diff

From: Olivier Matz Date: Fri, 15 Jun 2012 13:04:42 +0000 (+0200) Subject: cosmetic: fix whitespaces X-Git-Tag: v1.0~13 X-Git-Url: http://git.droids-corp.org/?p=diff2html.git;a=commitdiff_plain;h=b8d3cbfcf53806cc2045b818109feb6d009ca515;hp=d5d43cb0e52cacb457e5c96329e18bf54a2e0ea5 cosmetic: fix whitespaces enhance coding style. Signed-off-by: Sergey Satskiy Signed-off-by: Olivier Matz --- diff --git a/diff2html.py b/diff2html.py index 14094de..d616a5e 100644 --- a/diff2html.py +++ b/diff2html.py @@ -45,7 +45,7 @@ show_CR = False show_hunk_infos = False -html_hdr=""" +html_hdr = """ HTML Diff @@ -67,56 +67,56 @@ html_hdr=""" """ -html_footer=""" +html_footer = """ """ -table_hdr=""" +table_hdr = """ """ -table_footer=""" +table_footer = """ """ -DIFFON="\x01" -DIFFOFF="\x02" +DIFFON = "\x01" +DIFFOFF = "\x02" -buf=[] -add_cpt, del_cpt = 0,0 -line1, line2 = 0,0 -hunk_off1, hunk_size1, hunk_off2, hunk_size2 = 0,0,0,0 +buf = [] +add_cpt, del_cpt = 0, 0 +line1, line2 = 0, 0 +hunk_off1, hunk_size1, hunk_off2, hunk_size2 = 0, 0, 0, 0 # Characters we're willing to word wrap on -WORDBREAK=" \t;.,/):" +WORDBREAK = " \t;.,/):" def sane(x): - r="" + r = "" for i in x: j = ord(i) if i not in ['\t', '\n'] and ((j < 32) or (j >= 127)): - r=r+"." + r = r + "." else: - r=r+i + r = r + i return r def linediff(s, t): if len(s): - s=str(reduce(lambda x,y:x+y, [ sane(c) for c in s ])) + s = str(reduce(lambda x, y:x+y, [ sane(c) for c in s ])) if len(t): - t=str(reduce(lambda x,y:x+y, [ sane(c) for c in t ])) - - m,n = len(s), len(t) - d=[[(0,0) for i in range(n+1)] for i in range(m+1)] - x=[[(0,0) for i in range(n+1)] for i in range(m+1)] - - - d[0][0] = (0, (0,0)) + t = str(reduce(lambda x, y:x+y, [ sane(c) for c in t ])) + + m, n = len(s), len(t) + d = [[(0, 0) for i in range(n+1)] for i in range(m+1)] + x = [[(0, 0) for i in range(n+1)] for i in range(m+1)] + + + d[0][0] = (0, (0, 0)) for i in range(m+1)[1:]: - d[i][0] = (i,(i-1,0)) + d[i][0] = (i,(i-1, 0)) for j in range(n+1)[1:]: - d[0][j] = (j,(0,j-1)) + d[0][j] = (j,(0, j-1)) for i in range(m+1)[1:]: for j in range(n+1)[1:]: @@ -124,34 +124,34 @@ def linediff(s, t): cost = 0 else: cost = 1 - d[i][j] = min((d[i-1][j][0] + 1, (i-1,j)), - (d[i][j-1][0] + 1, (i,j-1)), - (d[i-1][j-1][0] + cost, (i-1,j-1))) - - l=[] - coord = (m,n) - while coord != (0,0): + d[i][j] = min((d[i-1][j][0] + 1, (i-1, j)), + (d[i][j-1][0] + 1, (i, j-1)), + (d[i-1][j-1][0] + cost, (i-1, j-1))) + + l = [] + coord = (m, n) + while coord != (0, 0): l.insert(0, coord) - x,y = coord + x, y = coord coord = d[x][y][1] l1 = [] l2 = [] for coord in l: - cx,cy = coord + cx, cy = coord child_val = d[cx][cy][0] - + father_coord = d[cx][cy][1] - fx,fy = father_coord + fx, fy = father_coord father_val = d[fx][fy][0] diff = (cx-fx, cy-fy) - if diff == (0,1): + if diff == (0, 1): l1.append("") l2.append(DIFFON + t[fy] + DIFFOFF) - elif diff == (1,0): + elif diff == (1, 0): l1.append(DIFFON + s[fx] + DIFFOFF) l2.append("") elif child_val-father_val == 1: @@ -161,48 +161,48 @@ def linediff(s, t): l1.append(s[fx]) l2.append(t[fy]) - r1,r2 = (reduce(lambda x,y:x+y, l1), reduce(lambda x,y:x+y, l2)) - return r1,r2 + r1, r2 = (reduce(lambda x, y:x+y, l1), reduce(lambda x, y:x+y, l2)) + return r1, r2 def convert(s, linesize=0, ponct=0): - i=0 - t="" + i = 0 + t = "" l=[] for c in s: # used by diffs - if c==DIFFON: + if c == DIFFON: t += '' - elif c==DIFFOFF: + elif c == DIFFOFF: t += "" # special html chars elif htmlentitydefs.codepoint2name.has_key(ord(c)): - t += "&%s;"%(htmlentitydefs.codepoint2name[ord(c)]) + t += "&%s;" % (htmlentitydefs.codepoint2name[ord(c)]) i += 1 # special highlighted chars - elif c=="\t" and ponct==1: + elif c == "\t" and ponct == 1: n = tabsize-(i%tabsize) - if n==0: - n=tabsize + if n == 0: + n = tabsize t += ('»'+' '*(n-1)) - elif c==" " and ponct==1: + elif c == " " and ponct == 1: t += '·' - elif c=="\n" and ponct==1: + elif c == "\n" and ponct == 1: if show_CR: t += '\' else: t += c i += 1 - if linesize and (WORDBREAK.count(c)==1): + if linesize and (WORDBREAK.count(c) == 1): t += '' - i=0 - if linesize and i>linesize: - i=0 + i = 0 + if linesize and i > linesize: + i = 0 t += "" - + return t @@ -221,49 +221,49 @@ def add_hunk(): outputfile.write('Offset %d, %d lines modified\n'%(hunk_off2, hunk_size2)) else: # ⋮ - vertical ellipsis - outputfile.write('⋮⋮'); + outputfile.write('⋮⋮') def add_line(s1, s2): global line1 global line2 - if s1==None and s2==None: - type_name="unmodified" - elif s1==None or s1=="": - type_name="added" - elif s2==None or s1=="": - type_name="deleted" - elif s1==s2: - type_name="unmodified" + if s1 == None and s2 == None: + type_name = "unmodified" + elif s1 == None or s1 == "": + type_name = "added" + elif s2 == None or s1 == "": + type_name = "deleted" + elif s1 == s2: + type_name = "unmodified" else: - type_name="changed" - s1,s2 = linediff(s1, s2) + type_name = "changed" + s1, s2 = linediff(s1, s2) - outputfile.write(''%type_name) - if s1!=None and s1!="": - outputfile.write('%d '%line1) + outputfile.write('' % type_name) + if s1 != None and s1 != "": + outputfile.write('%d ' % line1) outputfile.write('') outputfile.write(convert(s1, linesize=linesize, ponct=1)) outputfile.write('') else: - s1="" + s1 = "" outputfile.write(' ') - - if s2!=None and s2!="": + + if s2 != None and s2 != "": outputfile.write('%d '%line2) outputfile.write('') outputfile.write(convert(s2, linesize=linesize, ponct=1)) outputfile.write('') else: - s2="" + s2 = "" outputfile.write('') outputfile.write('\n') - if s1!="": + if s1 != "": line1 += 1 - if s2!="": + if s2 != "": line2 += 1 @@ -286,13 +286,13 @@ def empty_buffer(): max_len = (len(l0) > len(l1)) and len(l0) or len(l1) for i in range(max_len): s0, s1 = "", "" - if i