Re: [PATCH 0/4] MR1681: uxtheme: Avoid using lstrcmpiW.

[Zhiyi Zhang (＠zhiyi)]

Zhiyi Zhang (@zhiyi) commented about dlls/uxtheme/stylemap.c:
> - {status_desc, L"STATUS"},
> - {tab_desc, L"TAB"},
> - {taskband_desc, L"TASKBAND"},
> - {taskbar_desc, L"TASKBAR"},
> - {taskdialog_desc, L"TASKDIALOG"},
> - {toolbar_desc, L"TOOLBAR"},
> - {tooltip_desc, L"TOOLTIP"},
> - {trackbar_desc, L"TRACKBAR"},
> - {traynotify_desc, L"TRAYNOTIFY"},
> - {treeview_desc, L"TREEVIEW"},
> - {window_desc, L"WINDOW"},
> - {0},
>
> };
>
> -BOOL MSSTYLES_LookupPartState(LPCWSTR pszClass, LPCWSTR pszPart, LPCWSTR pszState, int *iPartId, int *iStateId)
> +static BOOL lookup_class_part(const struct class_desc *desc, const WCHAR *part_name, const WCHAR *state_name, int *part_id, int *state_id)

Splitting this function doesn't look better to me.