author | Ondrej Jirman <ondrej.jirman@zonio.net> | 2007-05-26 00:19:38 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-05-31 08:21:50 (UTC) |
commit | 3ce6fc16ef4670c535c6e35d48c7f980ff8a5532 (patch) (side-by-side diff) | |
tree | 7a3359b2e2488f0eaf350ba6a5e7865bbbabc969 | |
parent | 1a63cfcc3d83919e790e7e279eb35fc75adb0e3c (diff) | |
download | cgit-3ce6fc16ef4670c535c6e35d48c7f980ff8a5532.zip cgit-3ce6fc16ef4670c535c6e35d48c7f980ff8a5532.tar.gz cgit-3ce6fc16ef4670c535c6e35d48c7f980ff8a5532.tar.bz2 |
Handle single-line and empty commit subjects
If commit object ends with \0 after subject line, then info->subject
was not set.
This commit fixes this and also sets subject to ** empty ** if it
would otherwise be empty, so that there is something to click on.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | parsing.c | 11 |
1 files changed, 8 insertions, 3 deletions
@@ -231,20 +231,25 @@ struct commitinfo *cgit_parse_commit(struct commit *commit) } 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 = strdup("** empty **"); + else + ret->subject = substr(p, t); p = t + 1; while (*p == '\n') p = strchr(p, '\n') + 1; ret->msg = p; - } + } else + ret->subject = substr(p, p+strlen(p)); + return ret; } struct taginfo *cgit_parse_tag(struct tag *tag) { |