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

[Alexandre Julliard]

Roy Shea  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.