Re: [PATCH 0/1] MR4031: d3dx9: D3DXLoadMeshHierarchyFromXInMemory support empty mesh.

[Matteo Bruni (＠Mystral)]

Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/mesh.c:
> hr = filedata_get_name(filedata, &name);
>      if (FAILED(hr)) goto cleanup;
>
> -    hr = alloc_hier->lpVtbl->CreateMeshContainer(alloc_hier, name, &mesh_data,
> +    if (mesh_data.pMesh)
> +        hr = alloc_hier->lpVtbl->CreateMeshContainer(alloc_hier, name, &mesh_data,
>              materials ? ID3DXBuffer_GetBufferPointer(materials) : NULL,
>              effects ? ID3DXBuffer_GetBufferPointer(effects) : NULL,
>              num_materials,

I prefer to have brackets around multiline (even though single statement) blocks.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4031#note_47925