Re: [PATCH v2 4/4] submodule: cache superproject gitdir during 'update'

From
Emily Shaffer <hidden>
Date
2021-08-19 18:02:28
Reply to
Jonathan Tan, 2021-07-27
On Tue, Jul 27, 2021 at 10:51:51AM -0700, Jonathan Tan wrote:
quoted
A cached path to the superproject's gitdir might be added during 'git
[snip]
quoted
+		# Cache a pointer to the superproject's gitdir. This may have
Patches 3 and 4 look good to me except that for me, a cache is something
that lets us avoid an expensive computation. But as far as I know, we're
not planning to perform the expensive computation in the absence of a
cache. Maybe the word "record" would fit better.

This is nit-picky, though, and if others think that the word "cache"
fits here, that's fine by me too.
Thanks. I switched the commit messages around to use 'record' and 'hint'
when appropriate instead. I'll send a reroll with those nits as soon as
it passes CI.

 - Emily
Thread (38 messages) 38 messages, 6 authors, last reply 2021-10-14
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help