refs: dont make a table when there are no tags, add spacing between

diff --git a/stagit.c b/stagit.c
index 9d8efadf29f620f0af3810e9d3e4dcdfd9bb4394..4b568295f0afd8f359dfe5f826cc3cd982b716bd 100644 (file)
--- a/stagit.c
+++ b/stagit.c
@@ -881,17 +881,24 @@ writetags(FILE *fp)
        const git_oid *id = NULL;
        size_t i, len;
 
-       fputs("<h2>Tags</h2><table id=\"branches\"><thead>\n<tr><td>Tag</td>"
-             "<td>Age</td><td>Commit message</td>"
-             "<td>Author</td><td>Files</td><td class=\"num\">+</td>"
-             "<td class=\"num\">-</td></tr>\n</thead><tbody>\n", fp);
-
        /* summary page with branches and tags */
        memset(&tagnames, 0, sizeof(tagnames));
-       git_tag_list(&tagnames, repo);
+       if (git_tag_list(&tagnames, repo))
+               return -1;
+       if (!tagnames.count) {
+               git_strarray_free(&tagnames);
+               return 0;
+       }
+
        /* sort names */
        qsort(tagnames.strings, tagnames.count, sizeof(char *),
              (int (*)(const void *, const void *))&tagcompare);
+
+       fputs("<h2>Tags</h2><table id=\"branches\"><thead>\n<tr><td>Tag</td>"
+             "<td>Age</td><td>Commit message</td>"
+             "<td>Author</td><td>Files</td><td class=\"num\">+</td>"
+             "<td class=\"num\">-</td></tr>\n</thead><tbody>\n", fp);
+
        for (i = 0; i < tagnames.count; i++) {
                if (git_revparse_single(&obj, repo, tagnames.strings[i]))
                        continue;
@@ -946,6 +953,7 @@ writerefs(FILE *fp)
 
        if ((ret = writebranches(fp)))
                return ret;
+       fputs("<br/>", fp);
        return writetags(fp);
 }