2008/11/17 Stefan Dösinger <[EMAIL PROTECTED]>:
> - } else if (fixup ||
> + } else if (device->vertex_pipe->can_convert_d3dcolor || fixup ||
Shouldn't this be redundant if you already handle color conversion in
IWineD3DVertexBufferImpl_FindDecl()?
> - } else {
> + } else if(!device->vertex_pipe->can_convert_d3dcolor) {
I don't think this is how we want to do this. I think it makes more
sense to ask the pipeline if it can handle a particular type for a
particular usage. You can handle FLOAT16 support in the same way then.
Also note that EXT_vertex_array_bgra adds support for D3DCOLOR, but
only for diffuse, specular and generic attributes. That should be
enough for anything we come across, but with a "can_convert_d3dcolor"
flag like that you can't handle that extension properly.
