[PATCH 1/2] parsing.c: Remove leading space from committer
Lukas Fleischer
cgit at cryptocrack.de
Wed Jan 15 23:18:47 CET 2014
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 <cgit at 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
More information about the CGit
mailing list