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

Re: [PATCH v1] name-hash: properly fold directory names in adjust_dirname_case()

From
David Turner <novalis@novalis.org>
Date
Feb 8, 2018, 01:02 UTC
Message-ID
<1518051751.15704.476.camel@novalis.org>
In-Reply-To
<20180208004156.36224-1-benpeart@microsoft.com>
On Wed, 2018-02-07 at 19:41 -0500, Ben Peart wrote:
Show 12 quoted lines
> Correct the pointer arithmetic in adjust_dirname_case() so that it
> calls
> find_dir_entry() with the correct string length.  Previously passing
> in
> "dir1/foo" would pass a length of 6 instead of the correct 4.  This
> resulted in
> find_dir_entry() never finding the entry and so the subsequent memcpy
> that would
> fold the name to the version with the correct case never executed.
> 
> Add a test to validate the corrected behavior with name folding of
> directories.
Looks good to me.
Previous: Ben PeartNext: Torsten Bögershausen
Message 2 of 8 in “name-hash: properly fold directory names in adjust_dirname_case()”
  1. name-hash: properly fold directory names in adjust_dirname_case()Ben Peart, Feb 8, 2018
  2. David TurnerFeb 8, 2018
  3. Torsten BögershausenFeb 8, 2018
  4. Ben PeartFeb 8, 2018
  5. Junio C HamanoFeb 8, 2018
  6. Junio C HamanoFeb 8, 2018
  7. name-hash: properly fold directory names in adjust_dirname_case()Ben Peart, Feb 8, 2018
  8. Torsten BögershausenFeb 9, 2018

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.