|
diff --git a/ui-diff.c b/ui-diff.c index e0a72f7..1656b77 100644 --- a/ ui-diff.c+++ b/ ui-diff.c |
|
@@ -114,33 +114,33 @@ static void count_diff_lines(char *line, int len) |
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 | |
121 | static void inspect_filepair(struct diff_filepair *pair) |
121 | static 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, 0, count_diff_lines); |
130 | &binary, 0, ctx.qry.ignorews, 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; |
@@ -161,33 +161,34 @@ void cgit_print_diffstat(const unsigned char *old_sha1, |
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(" ("); |
166 | html(" ("); |
167 | ctx.qry.context = (save_context > 0 ? save_context : 3) << 1; |
167 | ctx.qry.context = (save_context > 0 ? save_context : 3) << 1; |
168 | cgit_self_link("more", NULL, NULL, &ctx); |
168 | cgit_self_link("more", NULL, NULL, &ctx); |
169 | html("/"); |
169 | html("/"); |
170 | ctx.qry.context = (save_context > 3 ? save_context : 3) >> 1; |
170 | ctx.qry.context = (save_context > 3 ? save_context : 3) >> 1; |
171 | cgit_self_link("less", NULL, NULL, &ctx); |
171 | cgit_self_link("less", NULL, NULL, &ctx); |
172 | ctx.qry.context = save_context; |
172 | ctx.qry.context = save_context; |
173 | html(" context)"); |
173 | html(" context)"); |
174 | html("</div>"); |
174 | html("</div>"); |
175 | html("<table summary='diffstat' class='diffstat'>"); |
175 | html("<table summary='diffstat' class='diffstat'>"); |
176 | max_changes = 0; |
176 | max_changes = 0; |
177 | cgit_diff_tree(old_sha1, new_sha1, inspect_filepair, prefix); |
177 | cgit_diff_tree(old_sha1, new_sha1, inspect_filepair, prefix, |
| |
178 | ctx.qry.ignorews); |
178 | for(i = 0; i<files; i++) |
179 | for(i = 0; i<files; i++) |
179 | print_fileinfo(&items[i]); |
180 | print_fileinfo(&items[i]); |
180 | html("</table>"); |
181 | html("</table>"); |
181 | html("<div class='diffstat-summary'>"); |
182 | html("<div class='diffstat-summary'>"); |
182 | htmlf("%d files changed, %d insertions, %d deletions", |
183 | htmlf("%d files changed, %d insertions, %d deletions", |
183 | files, total_adds, total_rems); |
184 | files, total_adds, total_rems); |
184 | html("</div>"); |
185 | html("</div>"); |
185 | } |
186 | } |
186 | |
187 | |
187 | |
188 | |
188 | /* |
189 | /* |
189 | * print a single line returned from xdiff |
190 | * print a single line returned from xdiff |
190 | */ |
191 | */ |
191 | static void print_line(char *line, int len) |
192 | static void print_line(char *line, int len) |
192 | { |
193 | { |
193 | char *class = "ctx"; |
194 | char *class = "ctx"; |
@@ -283,33 +284,34 @@ static void filepair_cb(struct diff_filepair *pair) |
283 | print_line_fn = cgit_ssdiff_line_cb; |
284 | print_line_fn = cgit_ssdiff_line_cb; |
284 | } |
285 | } |
285 | header(pair->one->sha1, pair->one->path, pair->one->mode, |
286 | header(pair->one->sha1, pair->one->path, pair->one->mode, |
286 | pair->two->sha1, pair->two->path, pair->two->mode); |
287 | pair->two->sha1, pair->two->path, pair->two->mode); |
287 | if (use_ssdiff) |
288 | if (use_ssdiff) |
288 | cgit_ssdiff_header_end(); |
289 | cgit_ssdiff_header_end(); |
289 | if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) { |
290 | if (S_ISGITLINK(pair->one->mode) || S_ISGITLINK(pair->two->mode)) { |
290 | if (S_ISGITLINK(pair->one->mode)) |
291 | if (S_ISGITLINK(pair->one->mode)) |
291 | print_line_fn(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); |
292 | print_line_fn(fmt("-Subproject %s", sha1_to_hex(pair->one->sha1)), 52); |
292 | if (S_ISGITLINK(pair->two->mode)) |
293 | if (S_ISGITLINK(pair->two->mode)) |
293 | print_line_fn(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); |
294 | print_line_fn(fmt("+Subproject %s", sha1_to_hex(pair->two->sha1)), 52); |
294 | if (use_ssdiff) |
295 | if (use_ssdiff) |
295 | cgit_ssdiff_footer(); |
296 | cgit_ssdiff_footer(); |
296 | return; |
297 | return; |
297 | } |
298 | } |
298 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, |
299 | if (cgit_diff_files(pair->one->sha1, pair->two->sha1, &old_size, |
299 | &new_size, &binary, ctx.qry.context, print_line_fn)) |
300 | &new_size, &binary, ctx.qry.context, |
| |
301 | ctx.qry.ignorews, print_line_fn)) |
300 | cgit_print_error("Error running diff"); |
302 | cgit_print_error("Error running diff"); |
301 | if (binary) { |
303 | if (binary) { |
302 | if (use_ssdiff) |
304 | if (use_ssdiff) |
303 | html("<tr><td colspan='4'>Binary files differ</td></tr>"); |
305 | html("<tr><td colspan='4'>Binary files differ</td></tr>"); |
304 | else |
306 | else |
305 | html("Binary files differ"); |
307 | html("Binary files differ"); |
306 | } |
308 | } |
307 | if (use_ssdiff) |
309 | if (use_ssdiff) |
308 | cgit_ssdiff_footer(); |
310 | cgit_ssdiff_footer(); |
309 | } |
311 | } |
310 | |
312 | |
311 | void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) |
313 | void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefix) |
312 | { |
314 | { |
313 | enum object_type type; |
315 | enum object_type type; |
314 | unsigned long size; |
316 | unsigned long size; |
315 | struct commit *commit, *commit2; |
317 | struct commit *commit, *commit2; |
@@ -343,21 +345,22 @@ void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefi |
343 | if (!commit2 || parse_commit(commit2)) |
345 | if (!commit2 || parse_commit(commit2)) |
344 | cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1))); |
346 | cgit_print_error(fmt("Bad commit: %s", sha1_to_hex(old_rev_sha1))); |
345 | } |
347 | } |
346 | |
348 | |
347 | if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff)) |
349 | if ((ctx.qry.ssdiff && !ctx.cfg.ssdiff) || (!ctx.qry.ssdiff && ctx.cfg.ssdiff)) |
348 | use_ssdiff = 1; |
350 | use_ssdiff = 1; |
349 | |
351 | |
350 | print_ssdiff_link(); |
352 | print_ssdiff_link(); |
351 | cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); |
353 | cgit_print_diffstat(old_rev_sha1, new_rev_sha1, prefix); |
352 | |
354 | |
353 | if (use_ssdiff) { |
355 | if (use_ssdiff) { |
354 | html("<table summary='ssdiff' class='ssdiff'>"); |
356 | html("<table summary='ssdiff' class='ssdiff'>"); |
355 | } else { |
357 | } else { |
356 | html("<table summary='diff' class='diff'>"); |
358 | html("<table summary='diff' class='diff'>"); |
357 | html("<tr><td>"); |
359 | html("<tr><td>"); |
358 | } |
360 | } |
359 | cgit_diff_tree(old_rev_sha1, new_rev_sha1, filepair_cb, prefix); |
361 | cgit_diff_tree(old_rev_sha1, new_rev_sha1, filepair_cb, prefix, |
| |
362 | ctx.qry.ignorews); |
360 | if (!use_ssdiff) |
363 | if (!use_ssdiff) |
361 | html("</td></tr>"); |
364 | html("</td></tr>"); |
362 | html("</table>"); |
365 | html("</table>"); |
363 | } |
366 | } |
|