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