d3dx9: Implement ID3DXSprite_OnLostDevice/OnResetDevice

Henri Verbeet hverbeet at gmail.com
Sun Nov 16 17:14:36 CST 2008

2008/11/15 Tony Wasserka <tony.wasserka at freenet.de>:
> +    if(This->sprites) {
> +        int i;
> +        for(i=0;i<This->sprite_count;i++)
> +            if(This->sprites[i].texture)
This if should be redundant. I looked at some of the other sprite
code, and noticed you've got similar constructions in other places.
You also don't handle HeapAlloc/HeapReAlloc failures in most places.

> +        HeapFree(GetProcessHeap(), 0, This->sprites);
Why do you free the sprites array here?

More information about the wine-devel mailing list