[7/10] WineD3D: Test for supported queries

Juan Lang juan_lang at yahoo.com
Thu Feb 15 12:13:07 CST 2007

Hi Stefan,

+    hr = IDirect3D9_CreateDevice( pD3d, D3DADAPTER_DEFAULT,
+                                  D3DCREATE_SOFTWARE_VERTEXPROCESSING,
&d3dpp, &pDevice );
+    ok(SUCCEEDED(hr), "Failed to create IDirect3D9Device (%s)\n",
+    if (FAILED(hr)) goto cleanup;


+    cleanup:
+    if(pDevice) IDirect3DDevice9_Release(pDevice);

The label cleanup isn't used anywhere except in the failure to create the
device.  In this case, wouldn't:
    if (FAILED(hr)) return;
be clearer, and omit the unneeded label?  There's no reason to release
pDevice in creating it failed.


Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail beta.

More information about the wine-devel mailing list