Jeff Hostetler <git@xxxxxxxxxxxxxxxxx> writes: > +static int add_excludes_from_buffer(char *buf, size_t size, > + const char *base, int baselen, > + struct exclude_list *el); > + > /* > * Given a file with name "fname", read it (either from disk, or from > * an index if 'istate' is non-null), parse it and store the > @@ -754,9 +758,9 @@ static int add_excludes(const char *fname, const char *base, int baselen, > struct sha1_stat *sha1_stat) > { > struct stat st; > - int fd, i, lineno = 1; > + int fd; > size_t size = 0; > - char *buf, *entry; > + char *buf; > > fd = open(fname, O_RDONLY); > if (fd < 0 || fstat(fd, &st) < 0) { The post-context of this hunk is quite interesting in that there is a call to read_skip_worktree_file_from_index(); which essentially pretends as if we read from the filesystem but in fact it grabs the blob object name registered in the index and reads from it. The reason why it is interesting is because this patch adds yet nother "let's instead read from a blob object" function and there is no sign to make the existing one take advantage of the new function seen in this patch.