author | Johan Herland <johan@herland.net> | 2010-06-24 15:52:57 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2010-07-18 08:53:48 (UTC) |
commit | 2cc8b99f083014c58d8937bfa4dcd2bc47cd7e58 (patch) (unidiff) | |
tree | 48d555b15c0432bb98712d746100bf6c0fe2be4a /ui-diff.c | |
parent | d20313e3daf855ee5d4808e050f54614c200d7b1 (diff) | |
download | cgit-2cc8b99f083014c58d8937bfa4dcd2bc47cd7e58.zip cgit-2cc8b99f083014c58d8937bfa4dcd2bc47cd7e58.tar.gz cgit-2cc8b99f083014c58d8937bfa4dcd2bc47cd7e58.tar.bz2 |
Add URL parameter 'ignorews' for optionally ignoring whitespace in diffs
The new ctx.qry.ignorews variable is passed via cgit_diff_files() and
cgit_diff_tree() to Git's diff machinery. This is equivalent to passing
--ignore-all-space to 'git diff'.
Signed-off-by: Johan Herland <johan@herland.net>
-rw-r--r-- | ui-diff.c | 11 |
1 files changed, 7 insertions, 4 deletions
@@ -126,9 +126,9 @@ static void inspect_filepair(struct diff_filepair *pair) | |||
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 |
@@ -173,9 +173,10 @@ void cgit_print_diffstat(const unsigned char *old_sha1, | |||
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'>"); |
@@ -295,9 +296,10 @@ static void filepair_cb(struct diff_filepair *pair) | |||
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>"); |
@@ -355,9 +357,10 @@ void cgit_print_diff(const char *new_rev, const char *old_rev, const char *prefi | |||
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 | } |