[4/8] WineD3D: Destroying the resource frees all private data
Stefan Dösinger
stefan at codeweavers.com
Thu May 10 16:46:20 CDT 2007
Am Donnerstag 10 Mai 2007 22:33 schrieb H. Verbeet:
> On 10/05/07, Stefan Dösinger <stefan at codeweavers.com> wrote:
> > - }
> > + };
>
> Any particular reason for this change?
I don't think so. Must have slipped in accidentally.
> > + if (Flags & WINED3DSPD_IUNKNOWN) {
> > + if(SizeOfData != 4) {
> > + WARN("IUnknown data with size %d, returning
> > WINED3DERR_INVALIDCALL\n", SizeOfData); + return
> > WINED3DERR_INVALIDCALL;
>
> This looks rather questionable to me. Are you making an assumption
> about the size of a pointer here or is 4 some magic constant?
Indeed :-/ I just hardcoded 4 as sizeof IUnknown * thoughlessly. We should
test that on 64 bit machines, but I guess that sizeof(IUnknown *) is more
correct. Does anyone have a 64 bit windows set up to test that?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20070510/935587cb/attachment.pgp
More information about the wine-devel
mailing list