Add cgit_commit_link() + support for id=sha1 to commit view

This adds a function to generate links to the commit page and extends said
page to use id from querystring as primary revision specified (fallback to
h).

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

1 files changed, 9 insertions, 9 deletions

diff --git a/ui-summary.c b/ui-summary.c
index 29b76e3..03dd078 100644
--- a/ui-summary.c
+++ b/ui-summary.c
@@ -17,4 +17,6 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1,
         struct commitinfo *info;
-        char buf[256], *url;
+        char buf[256];
+        char *ref;
 
+        ref = xstrdup(refname);
         strncpy(buf, refname, sizeof(buf));
@@ -27,3 +29,3 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1,
                 html("<tr><td>");
-                cgit_log_link(refname, NULL, NULL, refname, NULL, NULL);
+                cgit_log_link(ref, NULL, NULL, ref, NULL, NULL);
                 html("</td><td>");
@@ -33,7 +35,3 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1,
                 html("</td><td>");
-                url = cgit_pageurl(cgit_query_repo, "commit",
-                                   fmt("h=%s", sha1_to_hex(sha1)));
-                html_link_open(url, NULL, NULL);
-                html_ntxt(cgit_max_msg_len, info->subject);
-                html_link_close();
+                cgit_commit_link(info->subject, NULL, NULL, ref, NULL);
                 html("</td></tr>\n");
@@ -47,2 +45,3 @@ static int cgit_print_branch_cb(const char *refname, const unsigned char *sha1,
         }
+        free(ref);
         return 0;
@@ -56,4 +55,5 @@ static void cgit_print_object_ref(struct object *obj)
         if (obj->type == OBJ_COMMIT) {
-                page = "commit";
-                arg = "h";
+                cgit_commit_link(fmt("commit %s", sha1_to_hex(obj->sha1)), NULL, NULL,
+                                 cgit_query_head, sha1_to_hex(obj->sha1));
+                return;
         } else if (obj->type == OBJ_TREE) {