[PATCH 3/6] d3drm: Implement IDirect3DRM*_CreateTexture. (v3)
Stefan Dösinger
stefandoesinger at gmail.com
Sun May 8 10:57:16 CDT 2016
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256
Am 2016-05-08 um 11:31 schrieb Aaryaman Vasishta:
> + if (!d3drm_validate_image(image))
> + return D3DRMERR_BADVALUE;
Is this necessary? InitFromImage does it anyway I think.
>
> if (FAILED(hr = d3drm_texture_create(&object)))
> return hr;
You could call the public CreateObject method here, but I don't have a strong opinion on this.
> + object->d3drm = &d3drm->IDirect3DRM_iface;
Afaics InitFromImage does it anyway. It also causes problems when InitFromImage fails for some reason - you'll decref the d3drm object in the error path below.
> + if (FAILED(hr = IDirect3DRMTexture3_InitFromImage(&object->IDirect3DRMTexture3_iface, image)))
> + {
> + d3drm_texture_destroy(object);
> + return hr;
> + }
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2
iQIcBAEBCAAGBQJXL2HbAAoJEN0/YqbEcdMw9JcP/iXXJN5PuvyuLqki+eKgQHCv
fdRtBfdm57foN7+hfOcG9hZGgVWpdPMj5YFZmNhCaWKlnNjcxKDJE/ZsjsRaYXmD
pKPb6mLbNcZVBHyNeqbRcwlgJBiDLYIHM8BiT5DwssQK2lo8UmhVGJgmNZNPlGy8
sw/drdMg0lW11N6rOZR/fXcm5NY9F7oIAuI9945uElbOXg3byBvLAQdpPE95yJ5p
8rAQotCsbnqzGFEyRw3E1+0uKplBoaXd6ML+jhDrHK9Eqw0XWUtUsCuWiaYkTDGX
82XXWjhjsxQ84GkSKOweqtum6HjGDSnSZz12YTDMbspLywIpKzSMO5+O/dtxsolI
a5scAHlpYQ/bLK3TCWOkwNEsMkXBRjM8+j+PX1I3SQICdl3GmNxVo/b4ZU+0Wqt5
GwzyerUB2an5Dyf/+l7jqH0SxWgcrIiwa/ZTCxttAafDSooFhERwj402WhNQcZLN
BD9Jef52AYCJrduV3b3udvTyPuWCrPkkrm3NTSO6dbCIZG9EzMcujJZnA0D52y2E
qwfpfsfNNUJ7hkS1322u48i7GaSCZNoK6ISlA9RqtjVoRCGZKG/nGTwQgznrclB1
mvR24DUvAmogeTV0Akm20wUN3/n5sIkz47b0vb9Z5Jjg1v+90nTQ43pcMT0GCJhP
cN0YfnUAgpi7yD4yfA1E
=JJ9X
-----END PGP SIGNATURE-----
More information about the wine-devel
mailing list