Re: [PATCH] wscript: Do case insensitive check for WSH/WScript item names

[Jacek Caban]

On 04.11.2016 06:36, Nikolay Sivov wrote:
> Signed-off-by: Nikolay Sivov <nsivov(a)codeweavers.com>
> ---
>
> For https://bugs.winehq.org/show_bug.cgi?id=41651
>
> programs/wscript/main.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/programs/wscript/main.c b/programs/wscript/main.c
> index 3537f9d..e4bd676 100644
> --- a/programs/wscript/main.c
> +++ b/programs/wscript/main.c
> @@ -87,7 +87,7 @@ static HRESULT WINAPI ActiveScriptSite_GetItemInfo(IActiveScriptSite *iface,
>  {
>      WINE_TRACE("(%s %x %p %p)\n", wine_dbgstr_w(pstrName), dwReturnMask, ppunkItem, ppti);
>
> -    if(strcmpW(pstrName, wshW) && strcmpW(pstrName, wscriptW))
> +    if(strcmpiW(pstrName, wshW) && strcmpiW(pstrName, wscriptW))
>          return E_FAIL;
>
> if(dwReturnMask & SCRIPTINFO_ITYPEINFO) {

Looks good, but please add a test case.


Thanks,

Jacek