<div>Hi,</div><div><br></div><div>Attached is a series of patches to add better tests and fix Wine's implementation of the conversion between D3DVIEWPORT and D3DVIEWPORT2 structures in IDirect3DViewport / IDirect3DViewport2 / IDirect3DViewport3 implementation.</div>

<div><br></div><div>I'm sending this to to wine-devel because next commit round will be on Monday, so I hope to get some feedback until then and fix any issues before sending to wine-patches. I'm also working on fixing IDirect3DViewport::TransformVertices, which at the very least does not work if viewport data is set using D3DVIEWPORT2. This will fix bug <a href="http://bugs.winehq.org/show_bug.cgi?id=11819">http://bugs.winehq.org/show_bug.cgi?id=11819</a>, which is actually caused by Wine's buggy implementation of TransformVertices and has nothing to do with the z-buffer locking not begin implemented. I'm working on some tests and a fix, and will post them here when they are ready.</div>

<div><br></div><div>Octavian</div><div><br></div><div>--</div><div><br></div><div>Octavian Voicu (5):</div><div>  ddraw/tests: Remove some redundant d3d viewport tests.</div><div>  ddraw/tests: Add macro to verify viewport data bounds.</div>

<div>  ddraw/tests: Make ViewportTest loop through an array of tests.</div><div>  ddraw: Add error checking for IDirect3DViewportImpl_[GS]etViewport,</div><div>    with tests.</div><div>  ddraw: Add more viewport tests and make them pass under Windows and</div>

<div>    Wine.</div><div><br></div><div> dlls/ddraw/tests/d3d.c |  329 ++++++++++++++++++++++++++++--------------------</div><div> dlls/ddraw/viewport.c  |  114 ++++++++++-------</div><div> 2 files changed, 260 insertions(+), 183 deletions(-)</div>

<div><br></div>