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

Roderick Colenbrander thunderbird2k at gmx.net
Fri Nov 23 15:38:47 CST 2007


> 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
-- 
Psssst! Schon vom neuen GMX MultiMessenger gehört?
Der kann`s mit allen: http://www.gmx.net/de/go/multimessenger



More information about the wine-devel mailing list