Fixed unexpected tags in html output.

At least those that were catched by tidy.

Signed-off-by: Lars Hjemli <hjemli@gmail.com>

diff --git a/ui-commit.c b/ui-commit.c
index 59eeb1dfba0400fe6e8c90327bbcdb566fcc3a7d..6b135aa716fc2b643f111a5336dd33b998d94d09 100644 (file)
--- a/ui-commit.c
+++ b/ui-commit.c
@@ -103,7 +103,7 @@ void print_fileinfo(struct fileinfo *info)
              info->removed * 100.0 / max_changes);
        htmlf("<td class='none' style='width: %.1f%%;'/>",
              (max_changes - info->removed - info->added) * 100.0 / max_changes);
              info->removed * 100.0 / max_changes);
        htmlf("<td class='none' style='width: %.1f%%;'/>",
              (max_changes - info->removed - info->added) * 100.0 / max_changes);

-       html("</tr></table></a></td></tr>\n");
+       html("</tr></table></td></tr>\n");

 }
 
 void cgit_count_diff_lines(char *line, int len)
 }
 
 void cgit_count_diff_lines(char *line, int len)

diff --git a/ui-diff.c b/ui-diff.c
index 3ed0a465b378e877c21bb21856cac94d827a6c74..e6b957c5b61f0ca764efc6a7d4d66dd941f47c67 100644 (file)
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -38,7 +38,6 @@ static void header(unsigned char *sha1, char *path1, int mode1,
        int subproject;
 
        subproject = (S_ISDIRLNK(mode1) || S_ISDIRLNK(mode2));
        int subproject;
 
        subproject = (S_ISDIRLNK(mode1) || S_ISDIRLNK(mode2));

-       html("<tr><td>");

        html("<div class='head'>");
        html("diff --git a/");
        html_txt(path1);
        html("<div class='head'>");
        html("diff --git a/");
        html_txt(path1);


@@ -102,8 +101,10 @@ 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'>");
                commit = lookup_commit_reference(sha1);
                if (commit && !parse_commit(commit)) {
                        html("<table class='diff'>");

+                       html("<tr><td>");

                        cgit_diff_commit(commit, filepair_cb);
                        cgit_diff_commit(commit, filepair_cb);

-                       html("</td></tr></table>");
+                       html("</td></tr>");
+                       html("</table>");

                }
                return;
        }
                }
                return;
        }


@@ -127,7 +128,7 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex,
                header(sha1, path, 0644, sha2, path, 0644);
                if (cgit_diff_files(sha1, sha2, print_line))
                        cgit_print_error("Error running diff");
                header(sha1, path, 0644, sha2, path, 0644);
                if (cgit_diff_files(sha1, sha2, print_line))
                        cgit_print_error("Error running diff");

-               html("</tr></td>");
+               html("</td></tr>");

                break;
        case OBJ_TREE:
                cgit_diff_tree(sha1, sha2, filepair_cb);
                break;
        case OBJ_TREE:
                cgit_diff_tree(sha1, sha2, filepair_cb);


@@ -137,5 +138,5 @@ void cgit_print_diff(const char *head, const char *old_hex, const char *new_hex,
                                     typename(type)));
                break;
        }
                                     typename(type)));
                break;
        }

-       html("</td></tr></table>");
+       html("</table>");

 }
 }

diff --git a/ui-tree.c b/ui-tree.c
index 032a96607d3ce37cd5f00166293d2eb20615c972..006ca4762dafb504446d5abfa26ef4ff99c5a68a 100644 (file)
--- a/ui-tree.c
+++ b/ui-tree.c
@@ -50,7 +50,7 @@ static int print_entry(const unsigned char *sha1, const char *base,
                                          cgit_query_path ? cgit_query_path : "",
                                          pathname)));
        }
                                          cgit_query_path ? cgit_query_path : "",
                                          pathname)));
        }

-       htmlf("'>%s</a></div></td>", name);
+       htmlf("'>%s</a></td>", name);

        htmlf("<td class='filesize'>%li</td>", size);
 
        html("<td class='links'><a href='");
        htmlf("<td class='filesize'>%li</td>", size);
 
        html("<td class='links'><a href='");