author | Lars Hjemli <hjemli@gmail.com> | 2007-06-05 09:44:47 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-06-05 09:44:53 (UTC) |
commit | 4a6201e318d1db69d126f08a3d80ce2d5cf7ac01 (patch) (side-by-side diff) | |
tree | e551dce744ec35e21f0ecae8fcda37fdb58ab8d1 /ui-diff.c | |
parent | 51a960a3ca6b0cade97db287a342996c3e1de48d (diff) | |
download | cgit-4a6201e318d1db69d126f08a3d80ce2d5cf7ac01.zip cgit-4a6201e318d1db69d126f08a3d80ce2d5cf7ac01.tar.gz cgit-4a6201e318d1db69d126f08a3d80ce2d5cf7ac01.tar.bz2 |
ui-diff: emit table/tr/td at better locations
This fixes a bug which made the diff-link for each parent of a commit
output a diff without correct styles.
It also adds an error-message if the head commit specified on the querystring
isn't a valid commit object.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-diff.c | 16 |
1 files changed, 7 insertions, 9 deletions
@@ -98,2 +98,5 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, + html("<table class='diff'>"); + html("<tr><td>"); + if (head && !old_hex && !new_hex) { @@ -101,9 +104,6 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, commit = lookup_commit_reference(sha1); - if (commit && !parse_commit(commit)) { - html("<table class='diff'>"); - html("<tr><td>"); + if (commit && !parse_commit(commit)) cgit_diff_commit(commit, filepair_cb); - html("</td></tr>"); - html("</table>"); - } + else + cgit_print_error(fmt("Bad commit: %s", head)); return; @@ -123,6 +123,4 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, - html("<table class='diff'>"); switch(type) { case OBJ_BLOB: - html("<tr><td>"); header(sha1, path, 0644, sha2, path, 0644); @@ -130,3 +128,2 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, cgit_print_error("Error running diff"); - html("</td></tr>"); break; @@ -140,2 +137,3 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, } + html("</td></tr>"); html("</table>"); |