wininet: parse cookie information from cookie value

Aric Stewart aric at codeweavers.com
Mon Feb 16 07:34:10 CST 2009


Thanks for the tip.

Resent.
-aric

Rob Shearman wrote:
> 2009/2/13 Aric Stewart <aric at codeweavers.com>:
>> +        if (CompareStringW(GetThreadLocale(), NORM_IGNORECASE, ptr, 6,
>> +                           szDomain, 6) == 2)
> 
> LOCALE_INVARIANT should be used when comparing with a constant string.
> See here for the reasons why:
> http://blogs.msdn.com/michkap/archive/2004/12/29/344136.aspx
> http://blogs.msdn.com/larryosterman/archive/2004/07/01/171078.aspx
> http://msdn.microsoft.com/en-us/library/dd373795(VS.85).aspx
> 
> It doesn't look like it's implemented in Wine, so I guess you should
> just use MAKELCID(LANG_ENGLISH, SUBLANG_DEFAULT) with a FIXME to use
> LOCALE_INVARIANT when it's implemented.
> 
> And, yes, there's probably lots of other places where we do case
> insensitive comparisons incorrectly like this, but we should stop
> introducing new ones.



More information about the wine-devel mailing list