aboutsummaryrefslogtreecommitdiffstats
path: root/fsck.c
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2018-05-13 12:35:37 -0400
committerJeff King <peff@peff.net>2018-05-21 23:55:12 -0400
commited9c3220621d634d543bc4dd998d12167dfc57d4 (patch)
tree50393155b7372b6eb048e3b91ee708623ca3da69 /fsck.c
parentdb5a58c1bda5b20169b9958af1e8b05ddd178b01 (diff)
downloadgit-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.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/fsck.c b/fsck.c
index 5c8c12dde3..f7be966bde 100644
--- a/fsck.c
+++ b/fsck.c
@@ -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");