[PATCH 1/5] mfplat/sample: Simplify code for setting DirectX manager.

Nikolay Sivov nsivov at codeweavers.com
Tue Mar 22 23:57:12 CDT 2022


On 3/18/22 16:27, Giovanni Mascellani wrote:
>
> -    allocator->d3d9_device_manager = NULL;
> -    allocator->dxgi_device_manager = NULL;
>   
> -    if (dxgi_device_manager)
> -        allocator->dxgi_device_manager = dxgi_device_manager;
> -    else if (d3d9_device_manager)
> -        allocator->d3d9_device_manager = d3d9_device_manager;
> +    allocator->d3d9_device_manager = d3d9_device_manager;
> +    allocator->dxgi_device_manager = dxgi_device_manager;
>   
>       LeaveCriticalSection(&allocator->cs);
>   
I don't see a point in changing this. The way it's done now makes it 
clear that only one object is set.



More information about the wine-devel mailing list