Thread (14 messages) 14 messages, 3 authors, 2016-06-15

Re: What's cooking in git.git (Nov 2008, #06; Wed, 26)

From: Nguyen Thai Ngoc Duy <hidden>
Date: 2016-06-15 22:45:42

On 11/29/08, Nguyen Thai Ngoc Duy [off-list ref] wrote:
On 11/29/08, Daniel Barkalow [off-list ref] wrote:
 >  If there's any need for this to be distinguished from "assume unchanged",
 >  I think it should be used with, not instead of, the CE_VALID bit; and it
 >  could probably use some bit in the stat info section, since we don't need
 >  stat info if we know by assumption that the entry is valid.


Interesting. I'll think more about this.
As I said, CE_VALID implies all files are present. I could make
CE_NO_CHECKOUT to be used with CE_VALID, but I would need to check all
CE_VALID code path to make sure the behaviour remains if
CE_NO_CHECKOUT is absent. It's just more intrusive.

I have nothing against storing CE_NO_CHECKOUT in stat info except that
it seems inappropriate/hidden place to do. ce_flags is more obvious
choice. I haven't looked closely to stat info code in read-cache.c
though.
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help