From 5ca2f2ec3d00b0df8785d12283e1eff4e6a414f1 Mon Sep 17 00:00:00 2001
From: Ingo Schwarze <schwarze@openbsd.org>
Date: Thu, 24 Jul 2014 20:30:45 +0000
Subject: Sort result pages first by section number, then by name. By moving
 the sort from cgi.c to mansearch.c, we get two advantages: Easier access to
 the data needed for sorting, in particular the section number, and the
 apropos(1) command line utility profits as well.

Feature requested by deraadt@.
---
 cgi.c       | 11 -----------
 mansearch.c | 20 ++++++++++++++++++++
 mansearch.h |  1 +
 3 files changed, 21 insertions(+), 11 deletions(-)

diff --git a/cgi.c b/cgi.c
index 5a20f9ce..388ab61b 100644
--- a/cgi.c
+++ b/cgi.c
@@ -53,7 +53,6 @@ struct	req {
 };
 
 static	void		 catman(const struct req *, const char *);
-static	int	 	 cmp(const void *, const void *);
 static	void		 format(const struct req *, const char *);
 static	void		 html_print(const char *);
 static	void		 html_printquery(const struct req *);
@@ -593,8 +592,6 @@ pg_searchres(const struct req *req, struct manpage *r, size_t sz)
 		return;
 	}
 
-	qsort(r, sz, sizeof(struct manpage), cmp);
-
 	resp_begin_html(200, NULL);
 	resp_searchform(req);
 	puts("<DIV CLASS=\"results\">");
@@ -1059,14 +1056,6 @@ main(void)
 	return(EXIT_SUCCESS);
 }
 
-static int
-cmp(const void *p1, const void *p2)
-{
-
-	return(strcasecmp(((const struct manpage *)p1)->names,
-	    ((const struct manpage *)p2)->names));
-}
-
 /*
  * Scan for indexable paths.
  */
diff --git a/mansearch.c b/mansearch.c
index eaae414f..96e07388 100644
--- a/mansearch.c
+++ b/mansearch.c
@@ -94,6 +94,7 @@ static	void		 exprfree(struct expr *);
 static	struct expr	*exprspec(struct expr *, uint64_t,
 				 const char *, const char *);
 static	struct expr	*exprterm(const struct mansearch *, char *, int);
+static	int		 manpage_compare(const void *, const void *);
 static	void		 sql_append(char **sql, size_t *sz,
 				const char *newstr, int count);
 static	void		 sql_match(sqlite3_context *context,
@@ -331,6 +332,7 @@ mansearch(const struct mansearch *search,
 				    maxres, sizeof(struct manpage));
 			}
 			mpage = *res + cur;
+			mpage->sec = 10;
 			mpage->form = mp->form;
 			buildnames(mpage, db, s, mp->pageid,
 			    paths->paths[i], mp->form);
@@ -347,6 +349,7 @@ mansearch(const struct mansearch *search,
 		sqlite3_close(db);
 		ohash_delete(&htab);
 	}
+	qsort(*res, cur, sizeof(struct manpage), manpage_compare);
 	rc = 1;
 out:
 	if (-1 != fd) {
@@ -360,6 +363,18 @@ out:
 	return(rc);
 }
 
+static int
+manpage_compare(const void *vp1, const void *vp2)
+{
+	const struct manpage	*mp1, *mp2;
+	int			 diff;
+
+	mp1 = vp1;
+	mp2 = vp2;
+	diff = mp1->sec - mp2->sec;
+	return(diff ? diff : strcasecmp(mp1->names, mp2->names));
+}
+
 static void
 buildnames(struct manpage *mpage, sqlite3 *db, sqlite3_stmt *s,
 		uint64_t pageid, const char *path, int form)
@@ -392,6 +407,11 @@ buildnames(struct manpage *mpage, sqlite3 *db, sqlite3_stmt *s,
 		arch = sqlite3_column_text(s, 1);
 		name = sqlite3_column_text(s, 2);
 
+		/* Remember the first section found. */
+
+		if (9 < mpage->sec && '1' <= *sec && '9' >= *sec)
+			mpage->sec = (*sec - '1') + 1;
+
 		/* If the section changed, append the old one. */
 
 		if (NULL != prevsec &&
diff --git a/mansearch.h b/mansearch.h
index aa625108..7c5a7c00 100644
--- a/mansearch.h
+++ b/mansearch.h
@@ -74,6 +74,7 @@ struct	manpage {
 	char		*file; /* to be prefixed by manpath */
 	char		*names; /* a list of names with sections */
 	char		*output; /* user-defined additional output */
+	int		 sec; /* section number, 10 means invalid */
 	int		 form; /* 0 == catpage */
 };
 
-- 
cgit