Re: [PATCH v3 4/4] fsmonitor: Delay updating state until after split index is merged
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Oct 31, 2017, 17:31 UTC
- Message-ID
- <alpine.DEB.2.21.1.1710311830330.6482@virtualbox>
- In-Reply-To
- <xmqqa80728lo.fsf@gitster.mtv.corp.google.com>
Hi,
On Tue, 31 Oct 2017, Junio C Hamano wrote:
Show 28 quoted lines
> Alex Vandiver <alexmv@dropbox.com> writes:
>
> > diff --git a/fsmonitor.c b/fsmonitor.c
> > index 4ea44dcc6..417759224 100644
> > --- a/fsmonitor.c
> > +++ b/fsmonitor.c
> > @@ -49,20 +49,7 @@ int read_fsmonitor_extension(struct index_state *istate, const void *data,
> > ewah_free(fsmonitor_dirty);
> > return error("failed to parse ewah bitmap reading fsmonitor index extension");
> > }
> > -
> > - if (git_config_get_fsmonitor()) {
> > - /* Mark all entries valid */
> > - for (i = 0; i < istate->cache_nr; i++)
> > - istate->cache[i]->ce_flags |= CE_FSMONITOR_VALID;
> > -
> > - /* Mark all previously saved entries as dirty */
> > - ewah_each_bit(fsmonitor_dirty, fsmonitor_ewah_callback, istate);
> > -
> > - /* Now mark the untracked cache for fsmonitor usage */
> > - if (istate->untracked)
> > - istate->untracked->use_fsmonitor = 1;
> > - }
> > - ewah_free(fsmonitor_dirty);
> > + istate->fsmonitor_dirty = fsmonitor_dirty;
>
> This makes local variable "int i;" in this function unused and gets
> compiler warning.... to which end we introduced the DEVELOPER flag to catch these: if you call
make DEVELOPER=1
and compile with GCC or Clang, it will elevate such warnings to errors, and we highly encourage contributors to build their patched source code with said flag.
Thanks, Johannes