diff --git a/ui-commit.c b/ui-commit.c index 3618800..f1a22d3 100644 --- a/ui-commit.c +++ b/ui-commit.c @@ -14,7 +14,7 @@ void print_filepair(struct diff_filepair *pair) { char *query; char *class; - + switch (pair->status) { case DIFF_STATUS_ADDED: class = "add"; @@ -52,25 +52,25 @@ void print_filepair(struct diff_filepair *pair) html_filemode(pair->two->mode); } - 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)) { html("["); html_filemode(pair->one->mode); html("]"); } htmlf("", 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(" (%s from ", pair->status == DIFF_STATUS_COPIED ? + htmlf(" (%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); html_txt(pair->one->path); html(")"); @@ -81,44 +81,9 @@ void print_filepair(struct diff_filepair *pair) html(""); //TODO: diffstat graph - + html("\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); + files++; } void cgit_print_commit(const char *hex) @@ -166,17 +131,16 @@ void cgit_print_commit(const char *hex) "%s\n", + htmlf("'>%s\n", sha1_to_hex(p->item->object.sha1)); } if (cgit_repo->snapshots) { htmlf("download%s", filename); } - html("\n"); html("
"); html_txt(info->subject); @@ -186,7 +150,7 @@ void cgit_print_commit(const char *hex) html("
"); html(""); html("\n"); - cgit_diffstat(commit); + cgit_diff_commit(commit, print_filepair); htmlf("\n", files, files > 1 ? "s" : ""); html("
Affected files
" "%d file%s changed
");