Re: About: [PATCH 2/2] ntdll: Check for case-insensitive volumes.

[Charles Davis]

On 10/18/10 8:10 AM, Joris Huizer wrote:
> Hello,
>
> In the proposed patch "[PATCH 2/2] ntdll: Check for case-insensitive volumes.", I found this piece:
> - /* Add a new entry */
> - for (i = 0; i < sizeof(fs_cache)/sizeof(fs_cache[0]); i++)
> -
>    if (fs_cache[i].dev == 0)
> -
>    {
> -
>        /* This entry is empty, use it */
> -
>        fs_cache[i].dev = dev;
> -
>        fs_cache[i].fsid = fsid;
> -
>        fs_cache[i].case_sensitive = case_sensitive;
> -
>    }
>
> Maybe I'm missing something, but shouldn't you return from the function when finding a cache entry to fill? This seems to be filling every empty 
> cache entry?

Ah, you're right. I'll fix this in try 4.

Chip