Re: [PATCH 2/5] mstask: Implemented (Set|Get)ApplicationName. (Revision)

[Alexandre Julliard]

Roy Shea <royshea(a)gmail.com> writes:
> @@ -323,16 +324,62 @@ static HRESULT WINAPI MSTASK_ITask_SetApplicationName(
>          ITask* iface,
>          LPCWSTR pwszApplicationName)
>  {
> -    FIXME("(%p, %s): stub\n", iface, debugstr_w(pwszApplicationName));
> -    return E_NOTIMPL;
> +    DWORD n;
> +    TaskImpl *This = (TaskImpl *)iface;
> +
> +    TRACE("(%p, %s)\n", iface, debugstr_w(pwszApplicationName));
> +
> +    HeapFree(GetProcessHeap(), 0, This->applicationName);
> +    This->applicationName = NULL;

You need to preserve the previous value on error.

-- 
Alexandre Julliard
julliard(a)winehq.org