[PATCH 1/2] quartz/videorenderer: Get rid of CreateRenderingSubsystem().

[Zebediah Figura]

On 10/18/2019 12:34 AM, Chip Davis wrote:
> October 17, 2019 11:17 PM, "Zebediah Figura" <z.figura12 at gmail.com> wrote:
> 
>> diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c
>> index 87365d9212..f4d6945380 100644
>> --- a/dlls/quartz/videorenderer.c
>> +++ b/dlls/quartz/videorenderer.c
>> @@ -873,8 +858,11 @@ HRESULT VideoRenderer_create(IUnknown *outer, void **out)
>>      if (FAILED(hr))
>>          goto fail;
>>
>> -    if (!CreateRenderingSubsystem(pVideoRenderer)) {
>> -        hr = E_FAIL;
>> +    pVideoRenderer->hEvent = CreateEventW(NULL, TRUE, FALSE, NULL);
>> +
>> +    if (FAILED(BaseWindowImpl_PrepareWindow(&pVideoRenderer->baseControlWindow.baseWindow)))
> 
> I think you want to set 'hr' here.

I consciously didn't so as to preserve the current behaviour, but it is
better that way. I'll send an updated patch.

> 
>> +    {
>> +        CloseHandle(pVideoRenderer->hEvent);
>>          goto fail;
>>      }
>>
>> -- 
>> 2.23.0
> 
> 
> Chip
>