About: d3dx9: Store transform matrix per-sprite.

Matteo Bruni matteo.mystral at gmail.com
Tue Oct 19 08:30:30 CDT 2010


2010/10/19 Joris Huizer <joris_huizer at yahoo.com>:
> Hello,
>
> In this patch ID3DXSpriteImpl_Flush is adapted; the new loop being like:
>
>    int i, count, start;
>    /*  ... */
>    for(start=0;start<This->sprite_count;start+=count,count=0) {
>        i=start;
>        while(i<This->sprite_count &&
>              (count==0 || This->sprites[i].texture==This->sprites[i-1].texture)) {
>         /* filling in array */
>      }
>      /* rest of the outer loop, using array and count */
>    }
>
> It seems count is used uninitialised here on the first iteration ?
>
> HTH,
> Joris
>

Argh! You're absolutely right. Apparently it managed to work fine for
me so I didn't notice that silly mistake...
Thank you, I'll send a fix immediately.



More information about the wine-devel mailing list