[PATCH 3/5] Replace self-closing syntax on non-void elements

Chris Mayo aklhfex at gmail.com
Mon Feb 18 21:46:42 CET 2019


Self-closing syntax not valid in HTML5 for normal elements, that could
contain content.

Signed-off-by: Chris Mayo <aklhfex at gmail.com>
---
 ui-diff.c  | 4 ++--
 ui-log.c   | 2 +-
 ui-stats.c | 4 ++--
 ui-tree.c  | 2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/ui-diff.c b/ui-diff.c
index eb0a395..62319e9 100644
--- a/ui-diff.c
+++ b/ui-diff.c
@@ -343,7 +343,7 @@ void cgit_print_diff_ctrls(void)
 	html("<form method='get'>");
 	cgit_add_hidden_formfields(1, 0, ctx.qry.page);
 	html("<table>");
-	html("<tr><td colspan='2'/></tr>");
+	html("<tr><td colspan='2'></tr>");
 	html("<tr>");
 	html("<td class='label'>context:</td>");
 	html("<td class='ctrl'>");
@@ -374,7 +374,7 @@ void cgit_print_diff_ctrls(void)
 	html_intoption(1, "ssdiff", curr);
 	html_intoption(2, "stat only", curr);
 	html("</select></td></tr>");
-	html("<tr><td/><td class='ctrl'>");
+	html("<tr><td><td class='ctrl'>");
 	html("<noscript><input type='submit' value='reload'/></noscript>");
 	html("</td></tr></table>");
 	html("</form>");
diff --git a/ui-log.c b/ui-log.c
index 3bcb657..cce4d5a 100644
--- a/ui-log.c
+++ b/ui-log.c
@@ -309,7 +309,7 @@ static void print_commit(struct commit *commit, struct rev_info *revs)
 			html("</td>\n");
 		}
 		else
-			html("<td/>"); /* Empty 'Age' column */
+			html("<td>"); /* Empty 'Age' column */
 
 		/* Print msgbuf into remainder of table row */
 		htmlf("<td colspan='%d'%s>\n", columns - (revs->graph ? 1 : 0),
diff --git a/ui-stats.c b/ui-stats.c
index 7acd358..f5a2b6e 100644
--- a/ui-stats.c
+++ b/ui-stats.c
@@ -391,7 +391,7 @@ void cgit_show_stats(void)
 	html("<b>stat options</b>");
 	html("<form method='get'>");
 	cgit_add_hidden_formfields(1, 0, "stats");
-	html("<table><tr><td colspan='2'/></tr>");
+	html("<table><tr><td colspan='2'></tr>");
 	if (ctx.repo->max_stats > 1) {
 		html("<tr><td class='label'>Period:</td>");
 		html("<td class='ctrl'><select name='period' onchange='this.form.submit();'>");
@@ -408,7 +408,7 @@ void cgit_show_stats(void)
 	html_intoption(100, "100", top);
 	html_intoption(-1, "all", top);
 	html("</select></td></tr>");
-	html("<tr><td/><td class='ctrl'>");
+	html("<tr><td><td class='ctrl'>");
 	html("<noscript><input type='submit' value='Reload'/></noscript>");
 	html("</td></tr></table>");
 	html("</form>");
diff --git a/ui-tree.c b/ui-tree.c
index 08a8505..f8cbf7a 100644
--- a/ui-tree.c
+++ b/ui-tree.c
@@ -269,7 +269,7 @@ static void ls_head(void)
 	html("<th class='left'>Mode</th>");
 	html("<th class='left'>Name</th>");
 	html("<th class='right'>Size</th>");
-	html("<th/>");
+	html("<th>");
 	html("</tr>\n");
 }
 
-- 
2.20.1



More information about the CGit mailing list