On Fri, Oct 31, 2014 at 2:00 PM, Bruno Jesus <00cpxxx@gmail.com> wrote:

On Fri, Oct 31, 2014 at 3:11 AM, Nikolay Sivov <nsivov@codeweavers.com> wrote:
>> /* FIXME: This flag is undocumented and unknown by our CompareString.
>> - * We need a define for it.
>> */
>> - dwFlags = 0x10000000;
>> + dwFlags = LOCALE_RETURN_GENITIVE_NAMES;
>> if (!bCase) dwFlags |= NORM_IGNORECASE;
>
>
> It makes no sense to use this flag from CompareString(). I think it's better
> to remove it along with this comment.

Well I didn't want to touch that, my logic was that dwFlags was being
set to LOCALE_USE_CP_ACP and right after that it was set to
0x10000000. So I looked at the other flags:

#define LOCALE_NOUSEROVERRIDE 0x80000000
#define LOCALE_USE_CP_ACP 0x40000000
#define LOCALE_RETURN_NUMBER 0x20000000
#define LOCALE_RETURN_GENITIVE_NAMES 0x10000000

And noticed that 0x10000000 is LOCALE_RETURN_GENITIVE_NAMES. This
patch purpose was solely to remove the magic number. Please feel free
to remove anything you wish. For the sake of history these commits are
related to it:
https://source.winehq.org/git/wine.git/commitdiff/ac323a20
https://source.winehq.org/git/wine.git/commitdiff/64d68b10

Sure, I get that. My point was that such knowledge was probably gained by tracing with native comctl32, and it's not allowed.

Also if Wine doesn't support such flag for CompareString() anyway no need to bother setting it,

because virtual benefits of running Wine's comctl32 on Windows are insignificant.

I think I'll send a patch to remove that, yeah. Thanks for bringing this issue up.

Best regards,
Bruno