Move logic for age comparision from cmp_tag_age into cmp_age()

[ps-cgit] / ui-summary.c

diff --git a/ui-summary.c b/ui-summary.c
index c6846285c77644f5cf69fd8537aa7660d8624dd3..05170cc9fd50e51ced8d74a03193019a0b44ba98 100644 (file)
--- a/ui-summary.c
+++ b/ui-summary.c
@@ -10,6 +10,28 @@
 
 static int header;
 
+static int cmp_age(int age1, int age2)
+{
+       if (age1 != 0 && age2 != 0)
+               return age2 - age1;
+
+       if (age1 == 0 && age2 == 0)
+               return 0;
+
+       if (age1 == 0)
+               return +1;
+
+       return -1;
+}
+
+static int cmp_tag_age(const void *a, const void *b)
+{
+       struct refinfo *r1 = *(struct refinfo **)a;
+       struct refinfo *r2 = *(struct refinfo **)b;
+
+       return cmp_age(r1->tag->tagger_date, r2->tag->tagger_date);
+}
+
 static void cgit_print_branch(struct refinfo *ref)
 {
        struct commit *commit;
@@ -145,7 +167,7 @@ static void cgit_print_branches()
                cgit_print_branch(list.refs[i]);
 }
 
-static void cgit_print_tags()
+static void cgit_print_tags(int maxcount)
 {
        struct reflist list;
        int i;
@@ -156,8 +178,13 @@ static void cgit_print_tags()
        for_each_tag_ref(cgit_refs_cb, &list);
        if (list.count == 0)
                return;
+       qsort(list.refs, list.count, sizeof(*list.refs), cmp_tag_age);
+       if (!maxcount)
+               maxcount = list.count;
+       else if (maxcount > list.count)
+               maxcount = list.count;
        print_tag_header();
-       for(i=0; i<list.count; i++)
+       for(i=0; i<maxcount; i++)
                print_tag(list.refs[i]);
 }
 
@@ -188,6 +215,6 @@ void cgit_print_summary()
                html("<tr class='nohover'><td colspan='4'>&nbsp;</td></tr>");
        cgit_print_branches();
        html("<tr class='nohover'><td colspan='4'>&nbsp;</td></tr>");
-       cgit_print_tags();
+       cgit_print_tags(cgit_summary_tags);
        html("</table>");
 }