Henri,
You said to move all assembler, shader and compiler tests over to the d3dcompiler dll. When you said this did you mean only grammar tests, or did you also mean visual tests as well? I've hit sort of a roadblock, where I don't know exactly what to place where. Right now all the tests are visual, so I need to use either the d3d9 or d3d10 API. However, the d3dcompiler dll is version agnostic, so I'm unsure of which version of d3d I should use. As far as I can tell, the wine d3d 10 implementation is much less mature than its d3d9 one, which makes me guess I should put it in one of the d3d9 dlls. D3dx9 wouldn't be the right place for it, though, since I'm using the d3dcompiler calls instead of D3DXCompileShader. So should I move the visual tests to d3d9? Or somewhere else?
To complicate things, the next thing on my list is implementing grammar tests. These should definitely go in d3dcompiler, but that still doesn't answer the question of where to put the visual tests?
Travis.
On Wed, Aug 25, 2010 at 9:11 AM, Dan Kegel dank@kegel.com wrote:
Henri wrote:
You still have basic formatting errors like trailing spaces, mixing tabs and spaces, inconsistent "*" placement, etc. ... unnecessary cast... wine-patches reviews are supposed to catch non-trivial interactions with other parts of the code, not trivial whitespace errors.
Looking at http://wiki.winehq.org/SubmittingPatches, I didn't see a nice, concise checklist of things to check for, so I added one at the top, and linked to Henri's post. Improvements welcome.
- Dan