summaryrefslogtreecommitdiffabout
path: root/ui-diff.c
Unidiff
Diffstat (limited to 'ui-diff.c') (more/less context) (ignore whitespace changes)
-rw-r--r--ui-diff.c14
1 files changed, 11 insertions, 3 deletions
diff --git a/ui-diff.c b/ui-diff.c
index fb836df..e0a72f7 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -1,355 +1,363 @@
1/* ui-diff.c: show diff between two blobs 1/* ui-diff.c: show diff between two blobs
2 * 2 *
3 * Copyright (C) 2006 Lars Hjemli 3 * Copyright (C) 2006 Lars Hjemli
4 * 4 *
5 * Licensed under GNU General Public License v2 5 * Licensed under GNU General Public License v2
6 * (see COPYING for full license text) 6 * (see COPYING for full license text)
7 */ 7 */
8 8
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#include "ui-ssdiff.h"
13 13
14unsigned char old_rev_sha1[20]; 14unsigned char old_rev_sha1[20];
15unsigned char new_rev_sha1[20]; 15unsigned char new_rev_sha1[20];
16 16
17static int files, slots; 17static int files, slots;
18static int total_adds, total_rems, max_changes; 18static int total_adds, total_rems, max_changes;
19static int lines_added, lines_removed; 19static int lines_added, lines_removed;
20 20
21static struct fileinfo { 21static struct fileinfo {
22 char status; 22 char status;
23 unsigned char old_sha1[20]; 23 unsigned char old_sha1[20];
24 unsigned char new_sha1[20]; 24 unsigned char new_sha1[20];
25 unsigned short old_mode; 25 unsigned short old_mode;
26 unsigned short new_mode; 26 unsigned short new_mode;
27 char *old_path; 27 char *old_path;
28 char *new_path; 28 char *new_path;
29 unsigned int added; 29 unsigned int added;
30 unsigned int removed; 30 unsigned int removed;
31 unsigned long old_size; 31 unsigned long old_size;
32 unsigned long new_size; 32 unsigned long new_size;
33 int binary:1; 33 int binary:1;
34} *items; 34} *items;
35 35
36static int use_ssdiff = 0; 36static int use_ssdiff = 0;
37 37
38static void print_fileinfo(struct fileinfo *info) 38static void print_fileinfo(struct fileinfo *info)
39{ 39{
40 char *class; 40 char *class;
41 41
42 switch (info->status) { 42 switch (info->status) {
43 case DIFF_STATUS_ADDED: 43 case DIFF_STATUS_ADDED:
44 class = "add"; 44 class = "add";
45 break; 45 break;
46 case DIFF_STATUS_COPIED: 46 case DIFF_STATUS_COPIED:
47 class = "cpy"; 47 class = "cpy";
48 break; 48 break;
49 case DIFF_STATUS_DELETED: 49 case DIFF_STATUS_DELETED:
50 class = "del"; 50 class = "del";
51 break; 51 break;
52 case DIFF_STATUS_MODIFIED: 52 case DIFF_STATUS_MODIFIED:
53 class = "upd"; 53 class = "upd";
54 break; 54 break;
55 case DIFF_STATUS_RENAMED: 55 case DIFF_STATUS_RENAMED:
56 class = "mov"; 56 class = "mov";
57 break; 57 break;
58 case DIFF_STATUS_TYPE_CHANGED: 58 case DIFF_STATUS_TYPE_CHANGED:
59 class = "typ"; 59 class = "typ";
60 break; 60 break;
61 case DIFF_STATUS_UNKNOWN: 61 case DIFF_STATUS_UNKNOWN:
62 class = "unk"; 62 class = "unk";
63 break; 63 break;
64 case DIFF_STATUS_UNMERGED: 64 case DIFF_STATUS_UNMERGED:
65 class = "stg"; 65 class = "stg";
66 break; 66 break;
67 default: 67 default:
68 die("bug: unhandled diff status %c", info->status); 68 die("bug: unhandled diff status %c", info->status);
69 } 69 }
70 70
71 html("<tr>"); 71 html("<tr>");
72 htmlf("<td class='mode'>"); 72 htmlf("<td class='mode'>");
73 if (is_null_sha1(info->new_sha1)) { 73 if (is_null_sha1(info->new_sha1)) {
74 cgit_print_filemode(info->old_mode); 74 cgit_print_filemode(info->old_mode);
75 } else { 75 } else {
76 cgit_print_filemode(info->new_mode); 76 cgit_print_filemode(info->new_mode);
77 } 77 }
78 78
79 if (info->old_mode != info->new_mode && 79 if (info->old_mode != info->new_mode &&
80 !is_null_sha1(info->old_sha1) && 80 !is_null_sha1(info->old_sha1) &&
81 !is_null_sha1(info->new_sha1)) { 81 !is_null_sha1(info->new_sha1)) {
82 html("<span class='modechange'>["); 82 html("<span class='modechange'>[");
83 cgit_print_filemode(info->old_mode); 83 cgit_print_filemode(info->old_mode);
84 html("]</span>"); 84 html("]</span>");
85 } 85 }
86 htmlf("</td><td class='%s'>", class); 86 htmlf("</td><td class='%s'>", class);
87 cgit_diff_link(info->new_path, NULL, NULL, ctx.qry.head, ctx.qry.sha1, 87 cgit_diff_link(info->new_path, NULL, NULL, ctx.qry.head, ctx.qry.sha1,
88 ctx.qry.sha2, info->new_path, 0); 88 ctx.qry.sha2, info->new_path, 0);
89 if (info->status == DIFF_STATUS_COPIED || info->status == DIFF_STATUS_RENAMED) 89 if (info->status == DIFF_STATUS_COPIED || info->status == DIFF_STATUS_RENAMED)
90 htmlf(" (%s from %s)", 90 htmlf(" (%s from %s)",
91 info->status == DIFF_STATUS_COPIED ? "copied" : "renamed", 91 info->status == DIFF_STATUS_COPIED ? "copied" : "renamed",
92 info->old_path); 92 info->old_path);
93 html("</td><td class='right'>"); 93 html("</td><td class='right'>");
94 if (info->binary) { 94 if (info->binary) {
95 htmlf("bin</td><td class='graph'>%d -> %d bytes", 95 htmlf("bin</td><td class='graph'>%d -> %d bytes",
96 info->old_size, info->new_size); 96 info->old_size, info->new_size);
97 return; 97 return;
98 } 98 }
99 htmlf("%d", info->added + info->removed); 99 htmlf("%d", info->added + info->removed);
100 html("</td><td class='graph'>"); 100 html("</td><td class='graph'>");
101 htmlf("<table summary='file diffstat' width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes)); 101 htmlf("<table summary='file diffstat' width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes));
102 htmlf("<td class='add' style='width: %.1f%%;'/>", 102 htmlf("<td class='add' style='width: %.1f%%;'/>",
103 info->added * 100.0 / max_changes); 103 info->added * 100.0 / max_changes);
104 htmlf("<td class='rem' style='width: %.1f%%;'/>", 104 htmlf("<td class='rem' style='width: %.1f%%;'/>",
105 info->removed * 100.0 / max_changes); 105 info->removed * 100.0 / max_changes);
106 htmlf("<td class='none' style='width: %.1f%%;'/>", 106 htmlf("<td class='none' style='width: %.1f%%;'/>",
107 (max_changes - info->removed - info->added) * 100.0 / max_changes); 107 (max_changes - info->removed - info->added) * 100.0 / max_changes);
108 html("</tr></table></td></tr>\n"); 108 html("</tr></table></td></tr>\n");
109} 109}
110 110
111static void count_diff_lines(char *line, int len) 111static void count_diff_lines(char *line, int len)
112{ 112{
113 if (line && (len > 0)) { 113 if (line && (len > 0)) {
114 if (line[0] == '+') 114 if (line[0] == '+')
115 lines_added++; 115 lines_added++;
116 else if (line[0] == '-') 116 else if (line[0] == '-')
117 lines_removed++; 117 lines_removed++;
118 } 118 }
119} 119}
120 120
121static void inspect_filepair(struct diff_filepair *pair) 121static void inspect_filepair(struct diff_filepair *pair)
122{ 122{
123 int binary = 0; 123 int binary = 0;
124 unsigned long old_size = 0; 124 unsigned long old_size = 0;
125 unsigned long new_size = 0; 125 unsigned long new_size = 0;
126 files++; 126 files++;
127 lines_added = 0; 127 lines_added = 0;
128 lines_removed = 0; 128 lines_removed = 0;
129 cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, &new_size, 129 cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, &new_size,
130 &binary, count_diff_lines); 130 &binary, 0, count_diff_lines);
131 if (files >= slots) { 131 if (files >= slots) {
132 if (slots == 0) 132 if (slots == 0)
133 slots = 4; 133 slots = 4;
134 else 134 else
135 slots = slots * 2; 135 slots = slots * 2;
136 items = xrealloc(items, slots * sizeof(struct fileinfo)); 136 items = xrealloc(items, slots * sizeof(struct fileinfo));
137 } 137 }
138 items[files-1].status = pair->status; 138 items[files-1].status = pair->status;
139 hashcpy(items[files-1].old_sha1, pair->one->sha1); 139 hashcpy(items[files-1].old_sha1, pair->one->sha1);
140 hashcpy(items[files-1].new_sha1, pair->two->sha1); 140 hashcpy(items[files-1].new_sha1, pair->two->sha1);
141 items[files-1].old_mode = pair->one->mode; 141 items[files-1].old_mode = pair->one->mode;
142 items[files-1].new_mode = pair->two->mode; 142 items[files-1].new_mode = pair->two->mode;
143 items[files-1].old_path = xstrdup(pair->one->path); 143 items[files-1].old_path = xstrdup(pair->one->path);
144 items[files-1].new_path = xstrdup(pair->two->path); 144 items[files-1].new_path = xstrdup(pair->two->path);
145 items[files-1].added = lines_added; 145 items[files-1].added = lines_added;
146 items[files-1].removed = lines_removed; 146 items[files-1].removed = lines_removed;
147 items[files-1].old_size = old_size; 147 items[files-1].old_size = old_size;
148 items[files-1].new_size = new_size; 148 items[files-1].new_size = new_size;
149 items[files-1].binary = binary; 149 items[files-1].binary = binary;
150 if (lines_added + lines_removed > max_changes) 150 if (lines_added + lines_removed > max_changes)
151 max_changes = lines_added + lines_removed; 151 max_changes = lines_added + lines_removed;
152 total_adds += lines_added; 152 total_adds += lines_added;
153 total_rems += lines_removed; 153 total_rems += lines_removed;
154} 154}
155 155
156void cgit_print_diffstat(const unsigned char *old_sha1, 156void cgit_print_diffstat(const unsigned char *old_sha1,
157 const unsigned char *new_sha1, const char *prefix) 157 const unsigned char *new_sha1, const char *prefix)
158{ 158{
159 int i; 159 int i, save_context = ctx.qry.context;
160 160
161 html("<div class='diffstat-header'>"); 161 html("<div class='diffstat-header'>");
162 cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1, 162 cgit_diff_link("Diffstat", NULL, NULL, ctx.qry.head, ctx.qry.sha1,
163 ctx.qry.sha2, NULL, 0); 163 ctx.qry.sha2, NULL, 0);
164 if (prefix) 164 if (prefix)
165 htmlf(" (limited to '%s')", prefix); 165 htmlf(" (limited to '%s')", prefix);
166 html(" (");
167 ctx.qry.context = (save_context > 0 ? save_context : 3) << 1;
168 cgit_self_link("more", NULL, NULL, &ctx);
169 html("/");
170 ctx.qry.context = (save_context > 3 ? save_context : 3) >> 1;
171 cgit_self_link("less", NULL, NULL, &ctx);
172 ctx.qry.context = save_context;
173 html(" context)");
166 html("</div>"); 174 html("</div>");
167 html("<table summary='diffstat' class='diffstat'>"); 175 html("<table summary='diffstat' class='diffstat'>");
168 max_changes = 0; 176 max_changes = 0;
169 cgit_diff_tree(old_sha1, new_sha1, inspect_filepair, prefix); 177 cgit_diff_tree(old_sha1, new_sha1, inspect_filepair, prefix);
170 for(i = 0; i<files; i++) 178 for(i = 0; i<files; i++)
171 print_fileinfo(&items[i]); 179 print_fileinfo(&items[i]);
172 html("</table>"); 180 html("</table>");
173 html("<div class='diffstat-summary'>"); 181 html("<div class='diffstat-summary'>");
174 htmlf("%d files changed, %d insertions, %d deletions", 182 htmlf("%d files changed, %d insertions, %d deletions",
175 files, total_adds, total_rems); 183 files, total_adds, total_rems);
176 html("</div>"); 184 html("</div>");
177} 185}
178 186
179 187
180/* 188/*
181 * print a single line returned from xdiff 189 * print a single line returned from xdiff
182 */ 190 */
183static void print_line(char *line, int len) 191static void print_line(char *line, int len)
184{ 192{
185 char *class = "ctx"; 193 char *class = "ctx";
186 char c = line[len-1]; 194 char c = line[len-1];
187 195
188 if (line[0] == '+') 196 if (line[0] == '+')
189 class = "add"; 197 class = "add";
190 else if (line[0] == '-') 198 else if (line[0] == '-')
191 class = "del"; 199 class = "del";
192 else if (line[0] == '@') 200 else if (line[0] == '@')
193 class = "hunk"; 201 class = "hunk";
194 202
195 htmlf("<div class='%s'>", class); 203 htmlf("<div class='%s'>", class);
196 line[len-1] = '\0'; 204 line[len-1] = '\0';
197 html_txt(line); 205 html_txt(line);
198 html("</div>"); 206 html("</div>");
199 line[len-1] = c; 207 line[len-1] = c;
200} 208}
201 209
202static void header(unsigned char *sha1, char *path1, int mode1, 210static void header(unsigned char *sha1, char *path1, int mode1,
203 unsigned char *sha2, char *path2, int mode2) 211 unsigned char *sha2, char *path2, int mode2)
204{ 212{
205 char *abbrev1, *abbrev2; 213 char *abbrev1, *abbrev2;
206 int subproject; 214 int subproject;
207 215
208 subproject = (S_ISGITLINK(mode1) || S_ISGITLINK(mode2)); 216 subproject = (S_ISGITLINK(mode1) || S_ISGITLINK(mode2));
209 html("<div class='head'>"); 217 html("<div class='head'>");
210 html("diff --git a/"); 218 html("diff --git a/");
211 html_txt(path1); 219 html_txt(path1);
212 html(" b/"); 220 html(" b/");
213 html_txt(path2); 221 html_txt(path2);
214 222
215 if (is_null_sha1(sha1)) 223 if (is_null_sha1(sha1))
216 path1 = "dev/null"; 224 path1 = "dev/null";
217 if (is_null_sha1(sha2)) 225 if (is_null_sha1(sha2))
218 path2 = "dev/null"; 226 path2 = "dev/null";
219 227
220 if (mode1 == 0) 228 if (mode1 == 0)
221 htmlf("<br/>new file mode %.6o", mode2); 229 htmlf("<br/>new file mode %.6o", mode2);
222 230
223 if (mode2 == 0) 231 if (mode2 == 0)
224 htmlf("<br/>deleted file mode %.6o", mode1); 232 htmlf("<br/>deleted file mode %.6o", mode1);
225 233
226 if (!subproject) { 234 if (!subproject) {
227 abbrev1 = xstrdup(find_unique_abbrev(sha1, DEFAULT_ABBREV)); 235 abbrev1 = xstrdup(find_unique_abbrev(sha1, DEFAULT_ABBREV));
228 abbrev2 = xstrdup(find_unique_abbrev(sha2, DEFAULT_ABBREV)); 236 abbrev2 = xstrdup(find_unique_abbrev(sha2, DEFAULT_ABBREV));
229 htmlf("<br/>index %s..%s", abbrev1, abbrev2); 237 htmlf("<br/>index %s..%s", abbrev1, abbrev2);
230 free(abbrev1); 238 free(abbrev1);
231 free(abbrev2); 239 free(abbrev2);
232 if (mode1 != 0 && mode2 != 0) { 240 if (mode1 != 0 && mode2 != 0) {
233 htmlf(" %.6o", mode1); 241 htmlf(" %.6o", mode1);
234 if (mode2 != mode1) 242 if (mode2 != mode1)
235 htmlf("..%.6o", mode2); 243 htmlf("..%.6o", mode2);
236 } 244 }
237 html("<br/>--- a/"); 245 html("<br/>--- a/");
238 if (mode1 != 0) 246 if (mode1 != 0)
239 cgit_tree_link(path1, NULL, NULL, ctx.qry.head, 247 cgit_tree_link(path1, NULL, NULL, ctx.qry.head,
240 sha1_to_hex(old_rev_sha1), path1); 248 sha1_to_hex(old_rev_sha1), path1);
241 else 249 else
242 html_txt(path1); 250 html_txt(path1);
243 html("<br/>+++ b/"); 251 html("<br/>+++ b/");
244 if (mode2 != 0) 252 if (mode2 != 0)
245 cgit_tree_link(path2, NULL, NULL, ctx.qry.head, 253 cgit_tree_link(path2, NULL, NULL, ctx.qry.head,
246 sha1_to_hex(new_rev_sha1), path2); 254 sha1_to_hex(new_rev_sha1), path2);
247 else 255 else
248 html_txt(path2); 256 html_txt(path2);
249 } 257 }
250 html("</div>"); 258 html("</div>");
251} 259}
252 260
253static void print_ssdiff_link() 261static void print_ssdiff_link()
254{ 262{
255 if (!strcmp(ctx.qry.page, "diff")) { 263 if (!strcmp(ctx.qry.page, "diff")) {
256 if (use_ssdiff) 264 if (use_ssdiff)
257 cgit_diff_link("Unidiff", NULL, NULL, ctx.qry.head, 265 cgit_diff_link("Unidiff", NULL, NULL, ctx.qry.head,
258 ctx.qry.sha1, ctx.qry.sha2, ctx.qry.path, 1); 266 ctx.qry.sha1, ctx.qry.sha2, ctx.qry.path, 1);
259 else 267 else
260 cgit_diff_link("Side-by-side diff", NULL, NULL, 268 cgit_diff_link("Side-by-side diff", NULL, NULL,
261 ctx.qry.head, ctx.qry.sha1, 269 ctx.qry.head, ctx.qry.sha1,
262 ctx.qry.sha2, ctx.qry.path, 1); 270 ctx.qry.sha2, ctx.qry.path, 1);
263 } 271 }
264} 272}
265 273
266static void filepair_cb(struct diff_filepair *pair) 274static void filepair_cb(struct diff_filepair *pair)
267{ 275{
268 unsigned long old_size = 0; 276 unsigned long old_size = 0;
269 unsigned long new_size = 0; 277 unsigned long new_size = 0;
270 int binary = 0; 278 int binary = 0;
271 linediff_fn print_line_fn = print_line; 279 linediff_fn print_line_fn = print_line;
272 280
273 if (use_ssdiff) { 281 if (use_ssdiff) {
274 cgit_ssdiff_header_begin(); 282 cgit_ssdiff_header_begin();
275 print_line_fn = cgit_ssdiff_line_cb; 283 print_line_fn = cgit_ssdiff_line_cb;
276 } 284 }
277 header(pair->one->sha1, pair->one->path, pair->one->mode, 285 header(pair->one->sha1, pair->one->path, pair->one->mode,
278 pair->two->sha1, pair->two->path, pair->two->mode); 286 pair->two->sha1, pair->two->path, pair->two->mode);
279 if (use_ssdiff) 287 if (use_ssdiff)
280 cgit_ssdiff_header_end(); 288 cgit_ssdiff_header_end();
281 if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) { 289 if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) {
282 if (S_ISGITLINK(pair->one->mode)) 290 if (S_ISGITLINK(pair->one->mode))
283 print_line_fn(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); 291 print_line_fn(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52);
284 if (S_ISGITLINK(pair->two->mode)) 292 if (S_ISGITLINK(pair->two->mode))
285 print_line_fn(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); 293 print_line_fn(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52);
286 if (use_ssdiff) 294 if (use_ssdiff)
287 cgit_ssdiff_footer(); 295 cgit_ssdiff_footer();
288 return; 296 return;
289 } 297 }
290 if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, 298 if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size,
291 &new_size, &binary, print_line_fn)) 299 &new_size, &binary, ctx.qry.context, print_line_fn))
292 cgit_print_error("Error running diff"); 300 cgit_print_error("Error running diff");
293 if (binary) { 301 if (binary) {
294 if (use_ssdiff) 302 if (use_ssdiff)
295 html("<tr><td colspan='4'>Binary files differ</td></tr>"); 303 html("<tr><td colspan='4'>Binary files differ</td></tr>");
296 else 304 else
297 html("Binary files differ"); 305 html("Binary files differ");
298 } 306 }
299 if (use_ssdiff) 307 if (use_ssdiff)
300 cgit_ssdiff_footer(); 308 cgit_ssdiff_footer();
301} 309}
302 310
303void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) 311void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix)
304{ 312{
305 enum object_type type; 313 enum object_type type;
306 unsigned long size; 314 unsigned long size;
307 struct commit *commit, *commit2; 315 struct commit *commit, *commit2;
308 316
309 if (!new_rev) 317 if (!new_rev)
310 new_rev = ctx.qry.head; 318 new_rev = ctx.qry.head;
311 get_sha1(new_rev, new_rev_sha1); 319 get_sha1(new_rev, new_rev_sha1);
312 type = sha1_object_info(new_rev_sha1, &size); 320 type = sha1_object_info(new_rev_sha1, &size);
313 if (type == OBJ_BAD) { 321 if (type == OBJ_BAD) {
314 cgit_print_error(fmt("Bad object name: %s", new_rev)); 322 cgit_print_error(fmt("Bad object name: %s", new_rev));
315 return; 323 return;
316 } 324 }
317 commit = lookup_commit_reference(new_rev_sha1); 325 commit = lookup_commit_reference(new_rev_sha1);
318 if (!commit || parse_commit(commit)) 326 if (!commit || parse_commit(commit))
319 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(new_rev_sha1))); 327 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(new_rev_sha1)));
320 328
321 if (old_rev) 329 if (old_rev)
322 get_sha1(old_rev, old_rev_sha1); 330 get_sha1(old_rev, old_rev_sha1);
323 else if (commit->parents && commit->parents->item) 331 else if (commit->parents && commit->parents->item)
324 hashcpy(old_rev_sha1, commit->parents->item->object.sha1); 332 hashcpy(old_rev_sha1, commit->parents->item->object.sha1);
325 else 333 else
326 hashclr(old_rev_sha1); 334 hashclr(old_rev_sha1);
327 335
328 if (!is_null_sha1(old_rev_sha1)) { 336 if (!is_null_sha1(old_rev_sha1)) {
329 type = sha1_object_info(old_rev_sha1, &size); 337 type = sha1_object_info(old_rev_sha1, &size);
330 if (type == OBJ_BAD) { 338 if (type == OBJ_BAD) {
331 cgit_print_error(fmt("Bad object name: %s", sha1_to_hex(old_rev_sha1))); 339 cgit_print_error(fmt("Bad object name: %s", sha1_to_hex(old_rev_sha1)));
332 return; 340 return;
333 } 341 }
334 commit2 = lookup_commit_reference(old_rev_sha1); 342 commit2 = lookup_commit_reference(old_rev_sha1);
335 if (!commit2 || parse_commit(commit2)) 343 if (!commit2 || parse_commit(commit2))
336 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1))); 344 cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1)));
337 } 345 }
338 346
339 if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff)) 347 if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff))
340 use_ssdiff = 1; 348 use_ssdiff = 1;
341 349
342 print_ssdiff_link(); 350 print_ssdiff_link();
343 cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); 351 cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix);
344 352
345 if (use_ssdiff) { 353 if (use_ssdiff) {
346 html("<table summary='ssdiff' class='ssdiff'>"); 354 html("<table summary='ssdiff' class='ssdiff'>");
347 } else { 355 } else {
348 html("<table summary='diff' class='diff'>"); 356 html("<table summary='diff' class='diff'>");
349 html("<tr><td>"); 357 html("<tr><td>");
350 } 358 }
351 cgit_diff_tree(old_rev_sha1, new_rev_sha1, filepair_cb, prefix); 359 cgit_diff_tree(old_rev_sha1, new_rev_sha1, filepair_cb, prefix);
352 if (!use_ssdiff) 360 if (!use_ssdiff)
353 html("</td></tr>"); 361 html("</td></tr>");
354 html("</table>"); 362 html("</table>");
355} 363}