[PATCH v2] d3dx9: No need to fail if we don't support vertices reordering in D3DXMESHOPT_ATTRSORT

[Gijs Vermeulen]

From: Christian Costa <titan.costa(a)gmail.com>

A non optimized mesh does not prevent rendering as long as we return valid data to the application.
In our case we provided an identity remapping array when no vertices reordering is done.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=48529
Signed-off-by: Gijs Vermeulen <gijsvrm(a)gmail.com>
---
v2: d3dx9_36 -> d3dx9, sorry for the noise.

 dlls/d3dx9_36/mesh.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/dlls/d3dx9_36/mesh.c b/dlls/d3dx9_36/mesh.c
index 22f86f10d0..8f7c926c32 100644
--- a/dlls/d3dx9_36/mesh.c
+++ b/dlls/d3dx9_36/mesh.c
@@ -1689,11 +1689,7 @@ static HRESULT WINAPI d3dx9_mesh_OptimizeInplace(ID3DXMesh *iface, DWORD flags,
         if (FAILED(hr)) goto cleanup;
     } else if (flags & D3DXMESHOPT_ATTRSORT) {
         if (!(flags & D3DXMESHOPT_IGNOREVERTS))
-        {
             FIXME("D3DXMESHOPT_ATTRSORT vertex reordering not implemented.\n");
-            hr = E_NOTIMPL;
-            goto cleanup;
-        }
 
         hr = iface->lpVtbl->LockAttributeBuffer(iface, 0, &attrib_buffer);
         if (FAILED(hr)) goto cleanup;
-- 
2.28.0

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=79161

Your paranoid android.


=== debiant (build log) ===

Task: WineTest did not produce the wow32 report