[PATCH 2/7] d3dx9_36/tests: Added HLSL test suite

[Travis Athougies]

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 at 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
>
>
>



-- 
Travis Athougies