Jeff King <peff@xxxxxxxx> writes: > We check in fsck_commit_buffer() that commit->tree isn't NULL, which in > ... > So this code isn't helping anything. And it makes the fsck code slightly > more confusing and rigid (e.g., it requires that any commit structs have > already been parsed). Let's drop it. Thanks; I recall we discussed this during a review of a topic.