This did not really break anything in the past since spaces are ignored when rendering HTML. Remove the preceding space anyway to prevent from potential future problems.
Signed-off-by: Lukas Fleischer <c...@cryptocrack.de> --- This would get fixed anyway when switching to strip_prefix(). However, it is better to do this now to avoid any problems. It is also "required" for the following patch (otherwise, we would have to add a workaround for this bug to the test case which doesn't seem right and might lead to confusion when the bug suddenly disappears after the strip_prefix() migrate and the tests break). parsing.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/parsing.c b/parsing.c index 0062550..599f61e 100644 --- a/parsing.c +++ b/parsing.c @@ -156,7 +156,7 @@ struct commitinfo *cgit_parse_commit(struct commit *commit) } if (p && !prefixcmp(p, "committer ")) { - p = parse_user(p + 9, &ret->committer, &ret->committer_email, + p = parse_user(p + 10, &ret->committer, &ret->committer_email, &ret->committer_date); } -- 1.8.5.2 _______________________________________________ CGit mailing list CGit@lists.zx2c4.com http://lists.zx2c4.com/mailman/listinfo/cgit