Re: d3dx9 [patch 1/2]: Implement D3DXSHEvalSphericalLight

2013-03-14 Thread Matteo Bruni
2013/3/14 Nozomi Kodama nozomi.kod...@yahoo.com:
 No news about these patches?


 Nozomi?



Hmm, I was under the impression that these patches were already picked
up in git, but I see I was mistaken.
Anyway they look okay to me, except for a bit in the tests from the
first patch that I noticed just now by reading the patches again:

+else if (j = order * order)
+expected = test[l].red_received[j];

That somewhat defies the test, since you are then comparing a value
with itself in the ok(). You can just use the original value (i.e.
1.01f + j for the red component) instead. Same issue with green and
blue of course.


Cheers,
Matteo.

 
 De : Nozomi Kodama nozomi.kod...@yahoo.com
 À : wine-devel@winehq.org wine-devel@winehq.org
 Envoyé le : Lundi 11 mars 2013 17h11
 Objet : RE: d3dx9 [patch 1/2]: Implement D3DXSHEvalSphericalLight

 Hello

 Any problem with the patch
 http://source.winehq.org/patches/data/94777

 and
 http://source.winehq.org/patches/data/94776 ?

 Nozomi.






Re: d3dx9 [patch 1/2]: Implement D3DXSHEvalSphericalLight

2013-03-13 Thread Nozomi Kodama
No news about these patches?


Nozomi?





 De : Nozomi Kodama nozomi.kod...@yahoo.com
À : wine-devel@winehq.org wine-devel@winehq.org 
Envoyé le : Lundi 11 mars 2013 17h11
Objet : RE: d3dx9 [patch 1/2]: Implement D3DXSHEvalSphericalLight
 

Hello

Any problem with the patch
http://source.winehq.org/patches/data/94777

and
http://source.winehq.org/patches/data/94776 ?


Nozomi.


RE: d3dx9 [patch 1/2]: Implement D3DXSHEvalSphericalLight

2013-03-11 Thread Nozomi Kodama
Hello

Any problem with the patch
http://source.winehq.org/patches/data/94777

and
http://source.winehq.org/patches/data/94776 ?


Nozomi.