From: Johannes Schindelin <johannes.schindelin@gmx.de>
Date: Thu, 24 Feb 2022 16:51:08 GMT
Subject: Re: [PATCH v5 28/30] fsmonitor: force update index after large responses
Message-ID: <nycvar.QRO.7.76.6.2202241749340.11118@tvgsbejvaqbjf.bet>
In-Reply-To: <aec44a21afda7f058aca274dcb93c1eec442784a.1644612979.git.gitgitgadget@gmail.com>

Hi Jeff,

On Fri, 11 Feb 2022, Jeff Hostetler via GitGitGadget wrote:

> [...]
> + *
> + * NEEDSWORK: Does this need to be a config value?
> + */
> +static int fsmonitor_force_update_threshold = 100;
> +
>  void refresh_fsmonitor(struct index_state *istate)
>  {
>  	struct strbuf query_result = STRBUF_INIT;

We gave this ample opportunity to rise to a need, and no need arose, to
introduce a config option.

Therefore I am in favor of simply dropping the comment.

Ciao,
Dscho