[PATCH] d3dx9_36: Implement D3DXCheckTextureRequirements + tests
Christian Costa
titan.costa at wanadoo.fr
Mon Apr 19 13:35:19 CDT 2010
is_pow2 was in util.c, then remove. That's why I put then in texture.c.
But that's better to put them in util.c
just in case there are used from another part.
BTW, is your name Paulo, Jéremie or David ? ;-)
paulo lesgaz a écrit :
> Maybe this function should be in util.c.
>
> A+
>
> David
>
>
> --- En date de : *Lun 19.4.10, Henri Verbeet /<hverbeet at gmail.com>/* a
> écrit :
>
>
> De: Henri Verbeet <hverbeet at gmail.com>
> Objet: Re: [PATCH] d3dx9_36: Implement
> D3DXCheckTextureRequirements + tests
> À: wine-devel at winehq.org
> Date: Lundi 19 avril 2010, 11h43
>
> On 19 April 2010 09:43, Christian Costa <titan.costa at wanadoo.fr
> </mc/compose?to=titan.costa at wanadoo.fr>> wrote:
> > +BOOL is_pow2(UINT num)
> > +{
> > + UINT i;
> > +
> > + if (!num) return FALSE;
> > + for (i = 0; num >> 1; i++) num >>= 1;
> > +
> > + return (num << i) == num;
> > +}
> This should at least be static. You can also just replace it with
> "return !(num & (num - 1));". You don't care about 0, since you
> already check that earlier in D3DXCheckTextureRequirements().
> make_pow2() can probably be simplified as well.
>
>
>
> ------------------------------------------------------------------------
>
>
>
More information about the wine-devel
mailing list