Re: [PATCH 2/2] ntdll: Check for case-insensitive volumes. (resend try 4)

[Alexandre Julliard]

Charles Davis  writes:
> On 10/27/10 3:49 AM, Alexandre Julliard wrote:
> > You still have extra stat() calls in the fast path,
>
> I couldn't get rid of all of them. The one in lookup_unix_name() has to
> stay AFAIK; otherwise SFNs break in some weird way that I can't seem to
> figure out. (kernel32's path test, for example, has 4 failures, all of
> them related to short paths.)

That's of course the sign of a bigger problem, namely that you don't
handle short paths at all on case insensitive file systems.

If the tests didn't catch this sooner, it means that you should probably
first write more tests with various case and short names combinations.