Harden the page renderers

1 file changed, 7 insertions, 4 deletions

diff --git a/source/ui-repolist.c b/source/ui-repolist.c
index cd22afd..031cb0d 100644
--- a/source/ui-repolist.c
+++ b/source/ui-repolist.c
@@ -175,8 +175,10 @@ static int any_repos_visible(void)
 	return 0;
 }
 
-// currenturl is passed in because it is the same for every heading and every
-// row, and working it out here would mean an allocation and a free per cell.
+/*
+ * currenturl is passed in because it is the same for every heading and every
+ * row, and working it out here would mean an allocation and a free per cell.
+ */
 static void print_column_header(const char *title, const char *column, const char *currenturl)
 {
 	htmlf("<th class='left col-%s'><a href='", column);
@@ -208,7 +210,7 @@ static int section_changed(const char *section, const char *last)
 		return 0;
 	if (!section || !last)
 		return 1;
-	return strcmp(section, last) != 0;
+	return strcmp(section, last);
 }
 
 static void print_section_row(const char *section, int columns)
@@ -269,7 +271,8 @@ static void print_pager(int total, int pagelen, char *search, char *sort)
 	for (i = 0, ofs = 0; ofs < total; i++, ofs = i * pagelen) {
 		class = (ctx.qry.ofs == ofs) ? "current" : NULL;
 		html("<li>");
-		cgit_index_link(cgit_fmt("[%d]", i + 1), cgit_fmt("Page %d", i + 1), class, search, sort, ofs, 0);
+		cgit_index_link(cgit_fmt("[%d]", i + 1), cgit_fmt("Page %d", i + 1), class, search, sort,
+			ofs, 0);
 		html("</li>");
 	}
 	html("</ul>\n");