diff options
| author | Jeff King <peff@peff.net> | 2018-05-13 12:35:37 -0400 |
|---|---|---|
| committer | Jeff King <peff@peff.net> | 2018-05-21 23:55:12 -0400 |
| commit | ed9c3220621d634d543bc4dd998d12167dfc57d4 (patch) | |
| tree | 50393155b7372b6eb048e3b91ee708623ca3da69 /fsck.c | |
| parent | db5a58c1bda5b20169b9958af1e8b05ddd178b01 (diff) | |
| download | git-ed9c3220621d634d543bc4dd998d12167dfc57d4.tar.gz | |
fsck: simplify ".git" check
There's no need for us to manually check for ".git"; it's a
subset of the other filesystem-specific tests. Dropping it
makes our code slightly shorter. More importantly, the
existing code may make a reader wonder why ".GIT" is not
covered here, and whether that is a bug (it isn't, as it's
also covered in the filesystem-specific tests).
Signed-off-by: Jeff King <peff@peff.net>
Diffstat (limited to 'fsck.c')
| -rw-r--r-- | fsck.c | 4 |
1 files changed, 1 insertions, 3 deletions
@@ -561,9 +561,7 @@ static int fsck_tree(struct tree *item, struct fsck_options *options) has_empty_name |= !*name; has_dot |= !strcmp(name, "."); has_dotdot |= !strcmp(name, ".."); - has_dotgit |= (!strcmp(name, ".git") || - is_hfs_dotgit(name) || - is_ntfs_dotgit(name)); + has_dotgit |= is_hfs_dotgit(name) || is_ntfs_dotgit(name); has_zero_pad |= *(char *)desc.buffer == '0'; if (update_tree_entry_gently(&desc)) { retval += report(options, &item->object, FSCK_MSG_BAD_TREE, "cannot be parsed as a tree"); |