[PATCH 4/6] gameux: add storing Title registry value

Nikolay Sivov nsivov at codeweavers.com
Fri Sep 3 06:16:36 CDT 2010


  On 9/3/2010 13:25, Mariusz Pluciński wrote:
> +            hr = IXMLDOMElement_get_text(lpXMLElement,&bstrValue);
> +            if(SUCCEEDED(hr))
> +            {
> +                nValueLen = SysStringLen(bstrValue)+1;
> +
> +                GameData->sName = CoTaskMemAlloc(nValueLen*sizeof(WCHAR));
> +                if(GameData->sName == NULL)
> +                    hr = E_OUTOFMEMORY;
> +
> +                if(SUCCEEDED(hr))
> +                    lstrcpynW(GameData->sName, bstrValue, nValueLen);
> +
> +                SysFreeString(bstrValue);
> +            }
Maybe it's better to store it as BSTR to avoid such checks/allocations.



More information about the wine-devel mailing list