From: Martin Koegler Date: Sun, 13 Jan 2008 18:11:10 GMT Subject: [PATCH 2/2] git-fsck: remove commit test already done by parse_commit_buffer Message-ID: <1200247870151-git-send-email-mkoegler@auto.tuwien.ac.at> In-Reply-To: <12002478702664-git-send-email-mkoegler@auto.tuwien.ac.at> Signed-off-by: Martin Koegler --- builtin-fsck.c | 12 ------------ 1 files changed, 0 insertions(+), 12 deletions(-) diff --git a/builtin-fsck.c b/builtin-fsck.c index e4874f6..a77d8c0 100644 --- a/builtin-fsck.c +++ b/builtin-fsck.c @@ -360,18 +360,6 @@ static int fsck_commit(struct commit *commit) fprintf(stderr, "Checking commit %s\n", sha1_to_hex(commit->object.sha1)); - if (memcmp(buffer, "tree ", 5)) - return objerror(&commit->object, "invalid format - expected 'tree' line"); - if (get_sha1_hex(buffer+5, tree_sha1) || buffer[45] != '\n') - return objerror(&commit->object, "invalid 'tree' line format - bad sha1"); - buffer += 46; - while (!memcmp(buffer, "parent ", 7)) { - if (get_sha1_hex(buffer+7, sha1) || buffer[47] != '\n') - return objerror(&commit->object, "invalid 'parent' line format - bad sha1"); - buffer += 48; - } - if (memcmp(buffer, "author ", 7)) - return objerror(&commit->object, "invalid format - expected 'author' line"); free(commit->buffer); commit->buffer = NULL; if (!commit->tree) -- 1.4.4.4