https://bugs.winehq.org/show_bug.cgi?id=41383
Józef Kucia joseph.kucia@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |joseph.kucia@gmail.com
--- Comment #1 from Józef Kucia joseph.kucia@gmail.com --- (In reply to Axel D from comment #0)
There seems to be a mistake: in the non vertexblends case, ffp_normal_matrix is set to the transpose of the inverse of the modelview matrix. That is in accordance to the spec. However in the vertexblends case, the modelview matrices are used non-inversed.
That's what you would expect when implementing vertex blending. The inverse of model view matrices was used in the the initial version of the patch [1]. However, some testing suggested that normals are simply transformed by model view matrices [2]. It could be a good idea to add some tests for normals transformation when vertex blending is enabled.
[1] - https://www.winehq.org/pipermail/wine-patches/2015-July/140489.html [2] - https://www.winehq.org/pipermail/wine-devel/2015-July/108239.html