about summary refs log tree commit
path: root/ui-repolist.c
diff options
context:
space:
mode:
authorJason A. Donenfeld <Jason@zx2c4.com>2012-07-12 19:13:39 +0200
committerJason A. Donenfeld <Jason@zx2c4.com>2012-07-12 20:01:46 +0200
commit184c5655b2e350dbd0dd8be75d3f370f22aa6dee (patch)
treebcd4fef53e416f443666ec50f21adef33fc95e76 /ui-repolist.c
parentfc9181ff3d3ebbe0159871f6a49438e60bb17f58 (diff)
downloadcgit-184c5655b2e350dbd0dd8be75d3f370f22aa6dee.tar.gz
ui-repolist: Case insensitive sorting and age sort
Add two options, one for doing the ordinary name sorts in a
case-insensitive manner, and another for choosing to sort repos in each
section by age instead of by name.
Diffstat (limited to 'ui-repolist.c')
-rw-r--r--ui-repolist.c22
1 files changed, 18 insertions, 4 deletions
diff --git a/ui-repolist.c b/ui-repolist.c
index d946f32..4e2bdf4 100644
--- a/ui-repolist.c
+++ b/ui-repolist.c
@@ -9,6 +9,7 @@
 #include "cgit.h"
 #include "html.h"
 #include "ui-shared.h"
+#include <strings.h>
 
 time_t read_agefile(char *path)
 {
@@ -131,8 +132,12 @@ void print_pager(int items, int pagelen, char *search, char *sort)
 
 static int cmp(const char *s1, const char *s2)
 {
-        if (s1 && s2)
-                return strcmp(s1, s2);
+        if (s1 && s2) {
+                if (ctx.cfg.case_sensitive_sort)
+                        return strcmp(s1, s2);
+                else
+                        return strcasecmp(s1, s2);
+        }
         if (s1 && !s2)
                 return -1;
         if (s2 && !s1)
@@ -145,10 +150,19 @@ static int sort_section(const void *a, const void *b)
         const struct cgit_repo *r1 = a;
         const struct cgit_repo *r2 = b;
         int result;
+        time_t t;
 
         result = cmp(r1->section, r2->section);
-        if (!result)
-                result = cmp(r1->name, r2->name);
+        if (!result) {
+                if (!strcmp(ctx.cfg.section_sort, "age")) {
+                        // get_repo_modtime caches the value in r->mtime, so we don't
+                        // have to worry about inefficiencies here.
+                        if (get_repo_modtime(r1, &t) && get_repo_modtime(r2, &t))
+                                result = r2->mtime - r1->mtime;
+                }
+                if (!result)
+                        result = cmp(r1->name, r2->name);
+        }
         return result;
 }