Re: WineD3D: Implement detection of ATI cards with Mesa DRI drivers [try 4]

[Roderick Colenbrander]

> Roderick Colenbrander wrote:
> > I think the code should be like this (in pseudo code) this way also
>
> other cards will at least be classified:
> > case VENDOR_MESA:
> >     if(D3D9_CAPABLE(gl_info)
> >        if(strstr(gl_info->gl_renderer, "R300"))
> >            card = radeon_9500;
> >        else
> >            card = generic_nv_card;
> >     if(D3D8_CAPABLE(...)
> >        if(strstr(gl_info->gl_renderer, "R200"))
> >            card = ...
>
> You are right, but please notice that with my latest patch, the same 
> behavior is already achieved, by not "break"-ing at the end of 
> VENDOR_MESA case if we didn't match one of ATI module names.

The main reason why the D3DX_CAPABLE macros should be used before checking the renderer string is that not in all cases all extensions are around while the rendering string can still be the same.

Roderick