Re: [PATCH 05/23] fsck: stop checking commit->tree value

Jeff King [off-list ref] 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.