diff options
author | 2009-09-14 02:36:35 (JST) | |
---|---|---|
committer | 2009-09-17 03:17:56 (JST) | |
commit | 40e174d5364910750413d94b5417e57d108190ef (patch) | |
tree | 21d71be66e1b058cecc45b861523c071fb4ee126 | |
parent | 27479ac54c99584612f7246c9456119e88621eaa (diff) | |
download | cgit-40e174d5364910750413d94b5417e57d108190ef.zip cgit-40e174d5364910750413d94b5417e57d108190ef.tar.gz |
First version of side-by-side diff.
This constitutes the first prototype of a side-by-side diff. It is not
possible to switch between unidiff and side-by-side diff at all at this
stage.
Signed-off-by: Ragnar Ouchterlony <ragnar@lysator.liu.se>
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | Makefile | 1 | ||||
-rw-r--r-- | cgit.css | 35 | ||||
-rw-r--r-- | ui-diff.c | 15 | ||||
-rw-r--r-- | ui-ssdiff.c | 264 | ||||
-rw-r--r-- | ui-ssdiff.h | 12 |
5 files changed, 325 insertions, 2 deletions
@@ -90,6 +90,7 @@ OBJECTS += ui-refs.o | |||
90 | OBJECTS += ui-repolist.o | 90 | OBJECTS += ui-repolist.o |
91 | OBJECTS += ui-shared.o | 91 | OBJECTS += ui-shared.o |
92 | OBJECTS += ui-snapshot.o | 92 | OBJECTS += ui-snapshot.o |
93 | OBJECTS += ui-ssdiff.o | ||
93 | OBJECTS += ui-stats.o | 94 | OBJECTS += ui-stats.o |
94 | OBJECTS += ui-summary.o | 95 | OBJECTS += ui-summary.o |
95 | OBJECTS += ui-tag.o | 96 | OBJECTS += ui-tag.o |
@@ -601,3 +601,38 @@ table.hgraph div.bar { | |||
601 | background-color: #eee; | 601 | background-color: #eee; |
602 | height: 1em; | 602 | height: 1em; |
603 | } | 603 | } |
604 | |||
605 | table.ssdiff td.add { | ||
606 | color: black; | ||
607 | background: #afa; | ||
608 | } | ||
609 | |||
610 | table.ssdiff td.add_dark { | ||
611 | color: black; | ||
612 | background: #9c9; | ||
613 | } | ||
614 | |||
615 | table.ssdiff td.del { | ||
616 | color: black; | ||
617 | background: #faa; | ||
618 | } | ||
619 | |||
620 | table.ssdiff td.del_dark { | ||
621 | color: black; | ||
622 | background: #c99; | ||
623 | } | ||
624 | |||
625 | table.ssdiff td.changed { | ||
626 | color: black; | ||
627 | background: #ffa; | ||
628 | } | ||
629 | |||
630 | table.ssdiff td.changed_dark { | ||
631 | color: black; | ||
632 | background: #cc9; | ||
633 | } | ||
634 | |||
635 | table.ssdiff td.hunk { | ||
636 | color: #black; | ||
637 | background: #ccf; | ||
638 | } | ||
@@ -9,6 +9,7 @@ | |||
9 | #include "cgit.h" | 9 | #include "cgit.h" |
10 | #include "html.h" | 10 | #include "html.h" |
11 | #include "ui-shared.h" | 11 | #include "ui-shared.h" |
12 | #include "ui-ssdiff.h" | ||
12 | 13 | ||
13 | unsigned char old_rev_sha1[20]; | 14 | unsigned char old_rev_sha1[20]; |
14 | unsigned char new_rev_sha1[20]; | 15 | unsigned char new_rev_sha1[20]; |
@@ -32,6 +33,7 @@ static struct fileinfo { | |||
32 | int binary:1; | 33 | int binary:1; |
33 | } *items; | 34 | } *items; |
34 | 35 | ||
36 | static int use_ssdiff = 0; | ||
35 | 37 | ||
36 | static void print_fileinfo(struct fileinfo *info) | 38 | static void print_fileinfo(struct fileinfo *info) |
37 | { | 39 | { |
@@ -244,6 +246,8 @@ static void header(unsigned char *sha1, char *path1, int mode1, | |||
244 | html_txt(path2); | 246 | html_txt(path2); |
245 | } | 247 | } |
246 | html("</div>"); | 248 | html("</div>"); |
249 | if (use_ssdiff) | ||
250 | cgit_ssdiff_header(); | ||
247 | } | 251 | } |
248 | 252 | ||
249 | static void filepair_cb(struct diff_filepair *pair) | 253 | static void filepair_cb(struct diff_filepair *pair) |
@@ -251,9 +255,14 @@ static void filepair_cb(struct diff_filepair *pair) | |||
251 | unsigned long old_size = 0; | 255 | unsigned long old_size = 0; |
252 | unsigned long new_size = 0; | 256 | unsigned long new_size = 0; |
253 | int binary = 0; | 257 | int binary = 0; |
258 | linediff_fn print_line_fn = print_line; | ||
254 | 259 | ||
255 | header(pair->one->sha1, pair->one->path, pair->one->mode, | 260 | header(pair->one->sha1, pair->one->path, pair->one->mode, |
256 | pair->two->sha1, pair->two->path, pair->two->mode); | 261 | pair->two->sha1, pair->two->path, pair->two->mode); |
262 | if (use_ssdiff) { | ||
263 | cgit_ssdiff_header(); | ||
264 | print_line_fn = cgit_ssdiff_line_cb; | ||
265 | } | ||
257 | if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) { | 266 | if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) { |
258 | if (S_ISGITLINK(pair->one->mode)) | 267 | if (S_ISGITLINK(pair->one->mode)) |
259 | print_line(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); | 268 | print_line(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); |
@@ -261,11 +270,13 @@ static void filepair_cb(struct diff_filepair *pair) | |||
261 | print_line(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); | 270 | print_line(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); |
262 | return; | 271 | return; |
263 | } | 272 | } |
264 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, | 273 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, |
265 | &new_size, &binary, print_line)) | 274 | &new_size, &binary, print_line_fn)) |
266 | cgit_print_error("Error running diff"); | 275 | cgit_print_error("Error running diff"); |
267 | if (binary) | 276 | if (binary) |
268 | html("Binary files differ"); | 277 | html("Binary files differ"); |
278 | if (use_ssdiff) | ||
279 | cgit_ssdiff_footer(); | ||
269 | } | 280 | } |
270 | 281 | ||
271 | void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) | 282 | void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) |
diff --git a/ui-ssdiff.c b/ui-ssdiff.c new file mode 100644 index 0000000..3591ab4 --- /dev/null +++ b/ui-ssdiff.c | |||
@@ -0,0 +1,264 @@ | |||
1 | #include "cgit.h" | ||
2 | #include "html.h" | ||
3 | #include "ui-shared.h" | ||
4 | |||
5 | extern int use_ssdiff; | ||
6 | |||
7 | static int current_old_line, current_new_line; | ||
8 | |||
9 | struct deferred_lines { | ||
10 | int line_no; | ||
11 | char *line; | ||
12 | struct deferred_lines *next; | ||
13 | }; | ||
14 | |||
15 | static struct deferred_lines *deferred_old, *deferred_old_last; | ||
16 | static struct deferred_lines *deferred_new, *deferred_new_last; | ||
17 | |||
18 | static int line_from_hunk(char *line, char type) | ||
19 | { | ||
20 | char *buf1, *buf2; | ||
21 | int len; | ||
22 | |||
23 | buf1 = strchr(line, type); | ||
24 | if (buf1 == NULL) | ||
25 | return 0; | ||
26 | buf1 += 1; | ||
27 | buf2 = strchr(buf1, ','); | ||
28 | if (buf2 == NULL) | ||
29 | return 0; | ||
30 | len = buf2 - buf1; | ||
31 | buf2 = xmalloc(len + 1); | ||
32 | strncpy(buf2, buf1, len); | ||
33 | buf2[len] = '\0'; | ||
34 | int res = atoi(buf2); | ||
35 | free(buf2); | ||
36 | return res; | ||
37 | } | ||
38 | |||
39 | static char *replace_tabs(char *line) | ||
40 | { | ||
41 | char *prev_buf = line; | ||
42 | char *cur_buf; | ||
43 | int linelen = strlen(line); | ||
44 | int n_tabs = 0; | ||
45 | int i; | ||
46 | char *result; | ||
47 | char *spaces = " "; | ||
48 | |||
49 | if (linelen == 0) { | ||
50 | result = xmalloc(1); | ||
51 | result[0] = '\0'; | ||
52 | return result; | ||
53 | } | ||
54 | |||
55 | for (i = 0; i < linelen; i++) | ||
56 | if (line[i] == '\t') | ||
57 | n_tabs += 1; | ||
58 | result = xmalloc(linelen + n_tabs * 8 + 1); | ||
59 | result[0] = '\0'; | ||
60 | |||
61 | while (1) { | ||
62 | cur_buf = strchr(prev_buf, '\t'); | ||
63 | if (!cur_buf) { | ||
64 | strcat(result, prev_buf); | ||
65 | break; | ||
66 | } else { | ||
67 | strcat(result, " "); | ||
68 | strncat(result, spaces, 8 - (strlen(result) % 8)); | ||
69 | strncat(result, prev_buf, cur_buf - prev_buf); | ||
70 | } | ||
71 | prev_buf = cur_buf + 1; | ||
72 | } | ||
73 | return result; | ||
74 | } | ||
75 | |||
76 | static void deferred_old_add(char *line, int line_no) | ||
77 | { | ||