Correct comment in prepare_packed_git_one.
After staring at the comment and the associated for loop, I realized the comment was completely bogus. The section of code its talking about is trying to avoid duplicate mapping of the same packfile. Signed-off-by: Shawn O. Pearce <spearce@spearce.org> Signed-off-by: Junio C Hamano <junkio@cox.net>
This commit is contained in:
parent
625e9421df
commit
54a15a8df2
@ -779,7 +779,7 @@ static void prepare_packed_git_one(char *objdir, int local)
|
||||
if (!has_extension(de->d_name, ".idx"))
|
||||
continue;
|
||||
|
||||
/* we have .idx. Is it a file we can map? */
|
||||
/* Don't reopen a pack we already have. */
|
||||
strcpy(path + len, de->d_name);
|
||||
for (p = packed_git; p; p = p->next) {
|
||||
if (!memcmp(path, p->pack_name, len + namelen - 4))
|
||||
@ -787,6 +787,9 @@ static void prepare_packed_git_one(char *objdir, int local)
|
||||
}
|
||||
if (p)
|
||||
continue;
|
||||
/* See if it really is a valid .idx file with corresponding
|
||||
* .pack file that we can map.
|
||||
*/
|
||||
p = add_packed_git(path, len + namelen, local);
|
||||
if (!p)
|
||||
continue;
|
||||
|
Loading…
Reference in New Issue
Block a user