About: d3dx9: Store transform matrix per-sprite.

[Joris Huizer]

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

About: d3dx9: Store transform matrix per-sprite.

[Matteo Bruni]

2010/10/19 Joris Huizer :
> 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.