From: Alex Vandiver <alexmv@dropbox.com>
Date: Tue, 31 Oct 2017 18:43:10 GMT
Subject: Re: [PATCH v3 4/4] fsmonitor: Delay updating state until after split index is merged
Message-ID: <alpine.DEB.2.10.1710311139560.5248@alexmv-linux>
In-Reply-To: <alpine.DEB.2.21.1.1710311830330.6482@virtualbox>

On Tue, 31 Oct 2017, Junio C Hamano wrote:
> This makes local variable "int i;" in this function unused and gets
> compiler warning.

Apologies for leaving that detritus -- I saw you added a 'SQUASH??' commit
to deal with it, which LGTM.

On Tue, 31 Oct 2017, Johannes Schindelin wrote:
> ... to which end we introduced the DEVELOPER flag to catch these: if you
> call
> 
> 	make DEVELOPER=1

Aha!  Thanks for the tip; I'll be sure to use that from now on.
 - Alex