Re: GSoC-2011: Implement Missing Mesh Functions in Wine’s D3DX9

[Michael Mc Donnell]

Hi Stefan

This is my first try at implementing the UpdateSemantics mesh method.
It works fine here on my local machine, passes the new tests, and
works with my little interactive demo(not included). I still have a
few questions though.

This implementation allocates new heap memory and I suspect that it
would be faster to re-use the already allocated memory. That would,
however, require quite a bit of re-engineering. Should I submit the
current version or wait and try to do an optimized version?

Btw should I roll all or some of the patches up into a single patch?

Any other comments about the patches in general?

Cheers,
Michael Mc Donnell
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-d3dx9-test-Added-simple-test-for-UpdateSemantics-met.patch
Type: text/x-patch
Size: 5673 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110524/b1b2b56b/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-d3dx9-tests-UpdateSemantics-must-write-declaration.patch
Type: text/x-patch
Size: 1587 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110524/b1b2b56b/attachment-0005.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-d3dx9-tests-UpdateSemantics-ignores-bigger-vertex-si.patch
Type: text/x-patch
Size: 1019 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110524/b1b2b56b/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-d3dx9-Implement-UpdateSemantics-mesh-method.patch
Type: text/x-patch
Size: 2881 bytes
Desc: not available
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110524/b1b2b56b/attachment-0007.bin>