Re: [PATCH] commit: Optimize number of lstat() calls
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Feb 4, 2017, 10:42 UTC
- Message-ID
- <alpine.DEB.2.20.1702041142320.3496@virtualbox>
- In-Reply-To
- <xmqqd1ey5shc.fsf@gitster.mtv.corp.google.com>
Hi Junio,
On Fri, 3 Feb 2017, Junio C Hamano wrote:
Show 12 quoted lines
> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes: > > > I could imagine that there is a third option we should consider, too: only > > lstat() and update the paths that match the pathspec(s) provided on the > > command line (this is the semantic meaning of the --only option, after > > all: "I am only interested in these paths as far as this commit is > > concerned"). What do you think? > > I wondered that myself when I read the first message from Matthew > and noticed that we always refresh the entire index. But if it is > OK to leave the entire index un-refreshed, that would even be > simpler ;-)
Hah! You're right, that would be much simpler ;-)
Ciao, Johannes