git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Introduction of a ce_namelen field

From
Thomas Gummerer <t.gummerer@gmail.com>
Date
Jul 6, 2012, 16:07 UTC
Message-ID
<1341590850-15653-1-git-send-email-t.gummerer@gmail.com>
In-Reply-To
<7vtxxns4z4.fsf@alter.siamese.dyndns.org>

Thanks to the review of Junio, Duy and Thomas here is a reroll of the patches, where the name length is separated from the flags in the in-memory format and which includes a little bit of a performance optimization by using the ce_namelen field instead of strlen() in a couple of places thanks to the suggestion of Duy.

[PATCH/RFC v2 1/2] Strip namelen out of ce_flags into a ce_namelen [PATCH/RFC v2 2/2] Replace strlen(ce->name) with ce_namlen

builtin/apply.c | 3 ++- builtin/blame.c | 3 ++- builtin/checkout.c | 3 ++- builtin/update-index.c | 9 ++++++--- cache.h | 18 ++++++------------ read-cache.c | 58 +++++++++++++++++++++++++++++++++++----------------------- tree.c | 7 ++++--- unpack-trees.c | 5 +++-- 8 files changed, 60 insertions(+), 46 deletions(-)

Previous: Junio C HamanoNext: Thomas Gummerer
Message 6 of 18 in “Replace ce_namelen() with a ce_namelen field”
  1. Replace ce_namelen() with a ce_namelen fieldThomas Gummerer, Jul 4, 2012
  2. Thomas RastJul 4, 2012
  3. Nguyen Thai Ngoc DuyJul 4, 2012
  4. Nguyen Thai Ngoc DuyJul 4, 2012
  5. Junio C HamanoJul 4, 2012
  6. Introduction of a ce_namelen fieldThomas Gummerer, Jul 6, 2012
  7. 1/2 Strip namelen out of ce_flags into a ce_namelen fieldThomas Gummerer, Jul 6, 2012
  8. Junio C HamanoJul 9, 2012
  9. 0/3 Introduction of a ce_namelen fieldThomas Gummerer, Jul 11, 2012
  10. 1/3 read-cache.c: Handle long filenames correctlyThomas Gummerer, Jul 11, 2012
  11. Nguyen Thai Ngoc DuyJul 11, 2012
  12. Junio C HamanoJul 11, 2012
  13. 2/3 Strip namelen out of ce_flags into a ce_namelen fieldThomas Gummerer, Jul 11, 2012
  14. 3/3 Replace strlen() with ce_namelen()Thomas Gummerer, Jul 11, 2012
  15. Junio C HamanoJul 11, 2012
  16. Junio C HamanoJul 11, 2012
  17. 2/2 Replace strlen() with ce_namelen()Thomas Gummerer, Jul 6, 2012
  18. Junio C HamanoJul 9, 2012

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.