ui-repolist: extract get_repo_modtime() from print_modtime()

The new function is then used by both print_modtime() and
cgit_reposort_modtime().

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

1 files changed, 19 insertions, 27 deletions

diff --git a/ui-repolist.c b/ui-repolist.c
index 312a7ee..436a774 100644
--- a/ui-repolist.c
+++ b/ui-repolist.c
@@ -29,5 +29,5 @@ time_t read_agefile(char *path)
 }
 
-static void print_modtime(struct cgit_repo *repo)
+static int get_repo_modtime(const struct cgit_repo *repo, time_t *mtime)
 {
         char *path;
@@ -36,12 +36,21 @@ static void print_modtime(struct cgit_repo *repo)
         path = fmt("%s/%s", repo->path, ctx.cfg.agefile);
         if (stat(path, &s) == 0) {
-                cgit_print_age(read_agefile(path), -1, NULL);
-                return;
+                *mtime = read_agefile(path);
+                return 1;
         }
 
         path = fmt("%s/refs/heads/%s", repo->path, repo->defbranch);
-        if (stat(path, &s) != 0)
-                return;
-        cgit_print_age(s.st_mtime, -1, NULL);
+        if (stat(path, &s) == 0) {
+                *mtime = s.st_mtime;
+                return 1;
+        }
+        return 0;
+}
+
+static void print_modtime(struct cgit_repo *repo)
+{
+        time_t t;
+        if (get_repo_modtime(repo, &t))
+                cgit_print_age(t, -1, NULL);
 }
 
@@ -97,27 +106,10 @@ static int cgit_reposort_modtime(const void *a, const void *b)
         const struct cgit_repo *r1 = a;
         const struct cgit_repo *r2 = b;
-        char *path;
-        struct stat s;
         time_t t1, t2;
-        path = fmt("%s/%s", r1->path, ctx.cfg.agefile);
-        if (stat(path, &s) == 0) {
-                t1 = read_agefile(path);
-        } else {
-                path = fmt("%s/refs/heads/%s", r1->path, r1->defbranch);
-                if (stat(path, &s) != 0)
-                        return 0;
-                t1 =s.st_mtime;
-        }
 
-        path = fmt("%s/%s", r2->path, ctx.cfg.agefile);
-        if (stat(path, &s) == 0) {
-                t2 = read_agefile(path);
-        } else {
-                path = fmt("%s/refs/heads/%s", r2->path, r2->defbranch);
-                if (stat(path, &s) != 0)
-                        return 0;
-                t2 =s.st_mtime;
-        }
-        return t2-t1;
+        t1 = t2 = 0;
+        get_repo_modtime(r1, &t1);
+        get_repo_modtime(r2, &t2);
+        return t2 - t1;
 }