Thread (4 messages) flat view 4 messages, 2 authors, 2016-06-15

Re: [PATCH/RFC] Replace ce_namelen() with a ce_namelen field

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:54:13

Possibly related (same subject, not in this thread)

Thomas Gummerer [off-list ref] writes:
Replace the ce_namelen() function in cache.h with a ce_namelen
field in struct cache_entry.
NAK, at least in this form that duplicates the same information in
ce_namelen (new field) and bits in ce_flags, making it easier for
them to go out of sync with new bugs (or misconversions in this
patch, if there is one).

At least, keep the ce_namelen() inline used in the existing calling
site, as the whole point of it is that they do not have to care how
the name length is stored or computed.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help