author | Ondrej Jirman <ondrej.jirman@zonio.net> | 2007-05-25 23:14:25 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-05-31 08:18:00 (UTC) |
commit | 0928d8827a714f3908efa7eb9eb4cde28761af26 (patch) (unidiff) | |
tree | 67175d206caf7e220a996f699cf56dfa5c0c945b | |
parent | 2a09c0d184ff64676e173072c074649666f4f735 (diff) | |
download | cgit-0928d8827a714f3908efa7eb9eb4cde28761af26.zip cgit-0928d8827a714f3908efa7eb9eb4cde28761af26.tar.gz cgit-0928d8827a714f3908efa7eb9eb4cde28761af26.tar.bz2 |
Fixed unexpected tags in html output.
At least those that were catched by tidy.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | ui-commit.c | 2 | ||||
-rw-r--r-- | ui-diff.c | 9 | ||||
-rw-r--r-- | ui-tree.c | 2 |
3 files changed, 7 insertions, 6 deletions
diff --git a/ui-commit.c b/ui-commit.c index 59eeb1d..6b135aa 100644 --- a/ui-commit.c +++ b/ui-commit.c | |||
@@ -98,17 +98,17 @@ void print_fileinfo(struct fileinfo *info) | |||
98 | html("</td><td class='graph'>"); | 98 | html("</td><td class='graph'>"); |
99 | htmlf("<table width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes)); | 99 | htmlf("<table width='%d%%'><tr>", (max_changes > 100 ? 100 : max_changes)); |
100 | htmlf("<td class='add' style='width: %.1f%%;'/>", | 100 | htmlf("<td class='add' style='width: %.1f%%;'/>", |
101 | info->added * 100.0 / max_changes); | 101 | info->added * 100.0 / max_changes); |
102 | htmlf("<td class='rem' style='width: %.1f%%;'/>", | 102 | htmlf("<td class='rem' style='width: %.1f%%;'/>", |
103 | info->removed * 100.0 / max_changes); | 103 | info->removed * 100.0 / max_changes); |
104 | htmlf("<td class='none' style='width: %.1f%%;'/>", | 104 | htmlf("<td class='none' style='width: %.1f%%;'/>", |
105 | (max_changes - info->removed - info->added) * 100.0 / max_changes); | 105 | (max_changes - info->removed - info->added) * 100.0 / max_changes); |
106 | html("</tr></table></a></td></tr>\n"); | 106 | html("</tr></table></td></tr>\n"); |
107 | } | 107 | } |
108 | 108 | ||
109 | void cgit_count_diff_lines(char *line, int len) | 109 | void cgit_count_diff_lines(char *line, int len) |
110 | { | 110 | { |
111 | if (line && (len > 0)) { | 111 | if (line && (len > 0)) { |
112 | if (line[0] == '+') | 112 | if (line[0] == '+') |
113 | lines_added++; | 113 | lines_added++; |
114 | else if (line[0] == '-') | 114 | else if (line[0] == '-') |
@@ -33,17 +33,16 @@ static void print_line(char *line, int len) | |||
33 | 33 | ||
34 | static void header(unsigned char *sha1, char *path1, int mode1, | 34 | static void header(unsigned char *sha1, char *path1, int mode1, |
35 | unsigned char *sha2, char *path2, int mode2) | 35 | unsigned char *sha2, char *path2, int mode2) |
36 | { | 36 | { |
37 | char *abbrev1, *abbrev2; | 37 | char *abbrev1, *abbrev2; |
38 | int subproject; | 38 | int subproject; |
39 | 39 | ||
40 | subproject = (S_ISDIRLNK(mode1) || S_ISDIRLNK(mode2)); | 40 | subproject = (S_ISDIRLNK(mode1) || S_ISDIRLNK(mode2)); |
41 | html("<tr><td>"); | ||
42 | html("<div class='head'>"); | 41 | html("<div class='head'>"); |
43 | html("diff --git a/"); | 42 | html("diff --git a/"); |
44 | html_txt(path1); | 43 | html_txt(path1); |
45 | html(" b/"); | 44 | html(" b/"); |
46 | html_txt(path2); | 45 | html_txt(path2); |
47 | 46 | ||
48 | if (is_null_sha1(sha1)) | 47 | if (is_null_sha1(sha1)) |
49 | path1 = "dev/null"; | 48 | path1 = "dev/null"; |
@@ -97,18 +96,20 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, | |||
97 | unsigned long size; | 96 | unsigned long size; |
98 | struct commit *commit; | 97 | struct commit *commit; |
99 | 98 | ||
100 | if (head && !old_hex && !new_hex) { | 99 | if (head && !old_hex && !new_hex) { |
101 | get_sha1(head, sha1); | 100 | get_sha1(head, sha1); |
102 | commit = lookup_commit_reference(sha1); | 101 | commit = lookup_commit_reference(sha1); |
103 | if (commit && !parse_commit(commit)) { | 102 | if (commit && !parse_commit(commit)) { |
104 | html("<table class='diff'>"); | 103 | html("<table class='diff'>"); |
104 | html("<tr><td>"); | ||
105 | cgit_diff_commit(commit, filepair_cb); | 105 | cgit_diff_commit(commit, filepair_cb); |
106 | html("</td></tr></table>"); | 106 | html("</td></tr>"); |
107 | html("</table>"); | ||
107 | } | 108 | } |
108 | return; | 109 | return; |
109 | } | 110 | } |
110 | 111 | ||
111 | get_sha1(old_hex, sha1); | 112 | get_sha1(old_hex, sha1); |
112 | get_sha1(new_hex, sha2); | 113 | get_sha1(new_hex, sha2); |
113 | 114 | ||
114 | type = sha1_object_info(sha1, &size); | 115 | type = sha1_object_info(sha1, &size); |
@@ -122,20 +123,20 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex, | |||
122 | 123 | ||
123 | html("<table class='diff'>"); | 124 | html("<table class='diff'>"); |
124 | switch(type) { | 125 | switch(type) { |
125 | case OBJ_BLOB: | 126 | case OBJ_BLOB: |
126 | html("<tr><td>"); | 127 | html("<tr><td>"); |
127 | header(sha1, path, 0644, sha2, path, 0644); | 128 | header(sha1, path, 0644, sha2, path, 0644); |
128 | if (cgit_diff_files(sha1, sha2, print_line)) | 129 | if (cgit_diff_files(sha1, sha2, print_line)) |
129 | cgit_print_error("Error running diff"); | 130 | cgit_print_error("Error running diff"); |
130 | html("</tr></td>"); | 131 | html("</td></tr>"); |
131 | break; | 132 | break; |
132 | case OBJ_TREE: | 133 | case OBJ_TREE: |
133 | cgit_diff_tree(sha1, sha2, filepair_cb); | 134 | cgit_diff_tree(sha1, sha2, filepair_cb); |
134 | break; | 135 | break; |
135 | default: | 136 | default: |
136 | cgit_print_error(fmt("Unhandled object type: %s", | 137 | cgit_print_error(fmt("Unhandled object type: %s", |
137 | typename(type))); | 138 | typename(type))); |
138 | break; | 139 | break; |
139 | } | 140 | } |
140 | html("</td></tr></table>"); | 141 | html("</table>"); |
141 | } | 142 | } |
@@ -45,17 +45,17 @@ static int print_entry(const unsigned char *sha1, const char *base, | |||
45 | } else { | 45 | } else { |
46 | html("class='ls-blob'><a href='"); | 46 | html("class='ls-blob'><a href='"); |
47 | html_attr(cgit_pageurl(cgit_query_repo, "view", | 47 | html_attr(cgit_pageurl(cgit_query_repo, "view", |
48 | fmt("h=%s&id=%s&path=%s%s", curr_rev, | 48 | fmt("h=%s&id=%s&path=%s%s", curr_rev, |
49 | sha1_to_hex(sha1), | 49 | sha1_to_hex(sha1), |
50 | cgit_query_path ? cgit_query_path : "", | 50 | cgit_query_path ? cgit_query_path : "", |
51 | pathname))); | 51 | pathname))); |
52 | } | 52 | } |
53 | htmlf("'>%s</a></div></td>", name); | 53 | htmlf("'>%s</a></td>", name); |
54 | htmlf("<td class='filesize'>%li</td>", size); | 54 | htmlf("<td class='filesize'>%li</td>", size); |
55 | 55 | ||
56 | html("<td class='links'><a href='"); | 56 | html("<td class='links'><a href='"); |
57 | html_attr(cgit_pageurl(cgit_query_repo, "log", | 57 | html_attr(cgit_pageurl(cgit_query_repo, "log", |
58 | fmt("h=%s&path=%s%s", | 58 | fmt("h=%s&path=%s%s", |
59 | curr_rev, | 59 | curr_rev, |
60 | cgit_query_path ? cgit_query_path : "", | 60 | cgit_query_path ? cgit_query_path : "", |
61 | pathname))); | 61 | pathname))); |