Re: [PATCH v3 0/2] MR4140: uxtheme/tests: Add some tests for OpenThemeData() with various app/class names.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/system.c:
> if(bThemeActive)
>      {
> +        const WCHAR *p;
> +
>          pszAppName = UXTHEME_GetWindowProperty(hwnd, atSubAppName, szAppBuff, ARRAY_SIZE(szAppBuff));
>          /* If SetWindowTheme was used on the window, that overrides the class list passed to this function */
>          pszUseClassList = UXTHEME_GetWindowProperty(hwnd, atSubIdList, szClassBuff, ARRAY_SIZE(szClassBuff));
>          if(!pszUseClassList)
>              pszUseClassList = pszClassList;
>
> -        if (pszUseClassList)
> +        p = wcschr(pszClassList, ':');
> +        if (p)

Let's use StrStrW to check "::" instead. It's clearer that way.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4140#note_49623