Re: [PATCH v5 0/3] MR4915: d3d9: Implement Direct3DCreate9On12().

[Jan Sikorski (＠jsikorski)]

Jan Sikorski (@jsikorski) commented about dlls/d3d9/d3d9on12.c:
> +{
> - /* IUnknown */
> - d3d9on12_QueryInterface,
> - d3d9on12_AddRef,
> - d3d9on12_Release,
> - /* IDirect3DDevice9On12 */
> - d3d9on12_GetD3D12Device,
> - d3d9on12_UnwrapUnderlyingResource,
> - d3d9on12_ReturnUnderlyingResource,
>
> +};
> -
>
> +BOOL d3d9on12_init( struct d3d9on12 **d3d9on12, D3D9ON12_ARGS *override_list, UINT override_entries )
> +{
> - struct d3d9on12 *object;
> -
> - if (!override_list || !override_list->Enable9On12 || !override_entries)

Do we want to dereference the list before checking `override_entries`? I see that was changed previously, I'm not sure why, to me the current version is confusing and I don't see a test that justifies it.