author | Lars Hjemli <hjemli@gmail.com> | 2007-05-13 09:26:23 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-05-13 09:30:18 (UTC) |
commit | ae4c1ee11379e353f2a94201181a3a9dab75b646 (patch) (side-by-side diff) | |
tree | dddf3527d3a608c0c1983e462303f6898142116b /ui-commit.c | |
parent | 1b49de3c2c52e29c78f870826b6334abdadc0295 (diff) | |
download | cgit-ae4c1ee11379e353f2a94201181a3a9dab75b646.zip cgit-ae4c1ee11379e353f2a94201181a3a9dab75b646.tar.gz cgit-ae4c1ee11379e353f2a94201181a3a9dab75b646.tar.bz2 |
Use standard tree-diff function when showing diffstat for single commit
Remove unneeded code from ui-commit.c and replace with call to standard
diff-functions.
Also fix some whitespace issues.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-commit.c | 66 |
1 files changed, 15 insertions, 51 deletions
diff --git a/ui-commit.c b/ui-commit.c index 3618800..f1a22d3 100644 --- a/ui-commit.c +++ b/ui-commit.c @@ -16,3 +16,3 @@ void print_filepair(struct diff_filepair *pair) char *class; - + switch (pair->status) { @@ -54,4 +54,4 @@ void print_filepair(struct diff_filepair *pair) - if (pair->one->mode != pair->two->mode && - !is_null_sha1(pair->one->sha1) && + if (pair->one->mode != pair->two->mode && + !is_null_sha1(pair->one->sha1) && !is_null_sha1(pair->two->sha1)) { @@ -62,13 +62,13 @@ void print_filepair(struct diff_filepair *pair) htmlf("</td><td class='%s'>", class); - query = fmt("id=%s&id2=%s", sha1_to_hex(pair->one->sha1), - sha1_to_hex(pair->two->sha1)); - html_link_open(cgit_pageurl(cgit_query_repo, "diff", query), + query = fmt("id=%s&id2=%s", sha1_to_hex(pair->one->sha1), + sha1_to_hex(pair->two->sha1)); + html_link_open(cgit_pageurl(cgit_query_repo, "diff", query), NULL, NULL); - if (pair->status == DIFF_STATUS_COPIED || + if (pair->status == DIFF_STATUS_COPIED || pair->status == DIFF_STATUS_RENAMED) { html_txt(pair->two->path); - htmlf("</a> (%s from ", pair->status == DIFF_STATUS_COPIED ? + htmlf("</a> (%s from ", pair->status == DIFF_STATUS_COPIED ? "copied" : "renamed"); - query = fmt("id=%s", sha1_to_hex(pair->one->sha1)); - html_link_open(cgit_pageurl(cgit_query_repo, "view", query), + query = fmt("id=%s", sha1_to_hex(pair->one->sha1)); + html_link_open(cgit_pageurl(cgit_query_repo, "view", query), NULL, NULL); @@ -83,40 +83,5 @@ void print_filepair(struct diff_filepair *pair) //TODO: diffstat graph - - html("</td></tr>\n"); - files++; -} - -void diff_format_cb(struct diff_queue_struct *q, - struct diff_options *options, void *data) -{ - int i; - - for (i = 0; i < q->nr; i++) { - if (q->queue[i]->status == 'U') - continue; - print_filepair(q->queue[i]); - } -} - -void cgit_diffstat(struct commit *commit) -{ - struct diff_options opt; - int ret; - - diff_setup(&opt); - opt.output_format = DIFF_FORMAT_CALLBACK; - opt.detect_rename = 1; - opt.recursive = 1; - opt.format_callback = diff_format_cb; - diff_setup_done(&opt); - - if (commit->parents) - ret = diff_tree_sha1(commit->parents->item->object.sha1, - commit->object.sha1, - "", &opt); - else - ret = diff_root_tree_sha1(commit->object.sha1, "", &opt); - diffcore_std(&opt); - diff_flush(&opt); + html("</td></tr>\n"); + files++; } @@ -168,3 +133,3 @@ void cgit_print_commit(const char *hex) html_attr(cgit_pageurl(cgit_query_repo, "commit", query)); - htmlf("'>%s</a></td></tr>\n", + htmlf("'>%s</a></td></tr>\n", sha1_to_hex(p->item->object.sha1)); @@ -174,3 +139,3 @@ void cgit_print_commit(const char *hex) filename = fmt("%s-%s.zip", cgit_query_repo, hex); - html_attr(cgit_pageurl(cgit_query_repo, "snapshot", + html_attr(cgit_pageurl(cgit_query_repo, "snapshot", fmt("id=%s&name=%s", hex, filename))); @@ -178,3 +143,2 @@ void cgit_print_commit(const char *hex) } - html("</table>\n"); @@ -188,3 +152,3 @@ void cgit_print_commit(const char *hex) html("<tr><th colspan='3'>Affected files</tr>\n"); - cgit_diffstat(commit); + cgit_diff_commit(commit, print_filepair); htmlf("<tr><td colspan='3' class='summary'>" |