Make print_branch() handle refs not pointing at commits

[ps-cgit] / parsing.c

diff --git a/parsing.c b/parsing.c
index 4420e5867cacf716df3aec1649241acd3354dced..30e7648958789fb72930c405d1ba8d8c5a382f27 100644 (file)
--- a/parsing.c
+++ b/parsing.c
@@ -168,7 +168,7 @@ void cgit_parse_url(const char *url)
                if (p) {
                        p[0] = '\0';
                        if (p[1])
                if (p) {
                        p[0] = '\0';
                        if (p[1])

-                               cgit_query_path = xstrdup(p + 1);
+                               cgit_query_path = trim_end(p + 1, '/');

                }
                cgit_cmd = cgit_get_cmd_index(cmd + 1);
                cgit_query_page = xstrdup(cmd + 1);
                }
                cgit_cmd = cgit_get_cmd_index(cmd + 1);
                cgit_query_page = xstrdup(cmd + 1);


@@ -200,6 +200,9 @@ struct commitinfo *cgit_parse_commit(struct commit *commit)
        ret->subject = NULL;
        ret->msg = NULL;
 
        ret->subject = NULL;
        ret->msg = NULL;
 

+       if (p == NULL)
+               return ret;
+

        if (strncmp(p, "tree ", 5))
                die("Bad commit: %s", sha1_to_hex(commit->object.sha1));
        else
        if (strncmp(p, "tree ", 5))
                die("Bad commit: %s", sha1_to_hex(commit->object.sha1));
        else


@@ -230,18 +233,26 @@ struct commitinfo *cgit_parse_commit(struct commit *commit)
                p = strchr(t, '\n') + 1;
        }
 
                p = strchr(t, '\n') + 1;
        }
 

+       while (*p && (*p != '\n'))
+               p = strchr(p, '\n') + 1; // skip unknown header fields
+

        while (*p == '\n')
                p = strchr(p, '\n') + 1;
 
        t = strchr(p, '\n');
        while (*p == '\n')
                p = strchr(p, '\n') + 1;
 
        t = strchr(p, '\n');

-       if (t && *t) {
-               ret->subject = substr(p, t);
+       if (t) {
+               if (*t == '\0')
+                       ret->subject = "** empty **";
+               else
+                       ret->subject = substr(p, t);

                p = t + 1;
 
                while (*p == '\n')
                        p = strchr(p, '\n') + 1;
                p = t + 1;
 
                while (*p == '\n')
                        p = strchr(p, '\n') + 1;

-               ret->msg = p;
-       }
+               ret->msg = xstrdup(p);
+       } else
+               ret->subject = substr(p, p+strlen(p));
+

        return ret;
 }
 
        return ret;
 }
 


@@ -284,6 +295,9 @@ struct taginfo *cgit_parse_tag(struct tag *tag)
                p = strchr(p, '\n') + 1;
        }
 
                p = strchr(p, '\n') + 1;
        }
 

+       while (p && *p && (*p != '\n'))
+               p = strchr(p, '\n') + 1; // skip unknown tag fields
+

        while (p && (*p == '\n'))
                p = strchr(p, '\n') + 1;
        if (p && *p)
        while (p && (*p == '\n'))
                p = strchr(p, '\n') + 1;
        if (p && *p)