Matteo Bruni (@Mystral) commented about dlls/d3dx9_36/tests/texture.c:
texture = NULL; hr = D3DXCreateTextureFromFileInMemoryEx(device, dds_24bit, sizeof(dds_24bit), 32, 32, 6, 0, D3DFMT_UNKNOWN, D3DPOOL_DEFAULT, D3DX_DEFAULT, test_filter_values[i].filter, 0, NULL, NULL, &texture);
todo_wine_if(FAILED(hr)) ok(hr == D3D_OK, "Unexpected hr %#lx.\n", hr);
ok(hr == D3D_OK, "Unexpected hr %#lx.\n", hr); if (texture)
Now this `if` also becomes unnecessary; similarly below.