Check for NULL-subject in patch view

While at it, make sure the commit message ends with a '\n'.

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

diff --git a/ui-patch.c b/ui-patch.c
index ef79c7cef02933107bd7e249c3e2b8d7de33fdaf..e7a010accb58f0489f839a19fd3007733199c6ac 100644 (file)
--- a/ui-patch.c
+++ b/ui-patch.c
@@ -96,7 +96,12 @@ void cgit_print_patch(char *hex, struct cacheitem *item)
        htmlf("From: %s%s\n", info->author, info->author_email);
        html("Date: ");
        cgit_print_date(info->author_date, "%a, %d  %b  %Y  %H:%M:%S  %z%n");
        htmlf("From: %s%s\n", info->author, info->author_email);
        html("Date: ");
        cgit_print_date(info->author_date, "%a, %d  %b  %Y  %H:%M:%S  %z%n");

-       htmlf("Subject: %s\n\n%s", info->subject, info->msg);
+       htmlf("Subject: %s\n\n", info->subject);
+       if (info->msg && *info->msg) {
+               htmlf("%s", info->msg);
+               if (info->msg[strlen(info->msg) - 1] != '\n')
+                       html("\n");
+       }

        html("---\n");
        cgit_diff_tree(old_sha1, sha1, filepair_cb, NULL);
        html("--\n");
        html("---\n");
        cgit_diff_tree(old_sha1, sha1, filepair_cb, NULL);
        html("--\n");