[4/8] WineD3D: Destroying the resource frees all private data

H. Verbeet hverbeet at gmail.com
Thu May 10 15:33:58 CDT 2007


On 10/05/07, Stefan Dösinger <stefan at codeweavers.com> wrote:
> -        }
> +        };
Any particular reason for this change?

> +    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?

> -        list_add_tail(&This->resource.privateData, &data->entry);
> -        return WINED3D_OK;
> -
...
> +    list_init(&data->entry);
> +    list_add_tail(&This->resource.privateData, &data->entry);
>
>      return WINED3D_OK;
Is the added list_init() intentional here? Looks redundant to me, if
you're doing a list_add_tail() right afterwards anyway.



More information about the wine-patches mailing list