[PATCH 1/1] wined3d: Make 8bpp a valid depth in EnumAdapterModes

Vitaly Budovski vbudovski at gmail.com
Mon Apr 2 01:26:26 CDT 2007


Chris Robinson wrote:
> On Saturday 31 March 2007 02:25:52 am you wrote:
>   
>> You can change WineD3D to allow WINED3DFMT_P8, but you have to modify d3d8
>> and d3d9 to filter it out. This is the correct approach then. Injecting P8
>> in ddraw will work too, but it is not correct from the design point of
>> view(since then wined3d says "I can't do that", and ddraw says "But I
>> insist you can do it"
>>     
>
> How about the attached patch? If not given an explicit mode, DDraw will loop 
> over known good formats and not pass WINED3DFMT_UNKNOWN to WineD3D at all. 
> Someone can then fix D3D8 to not pass it either, then fix WineD3D to not have 
> special handling for enumerating WINED3DFMT_UNKNOWN. That should fix the 
> current DDraw problem fine, and not immediately risk breaking D3D8 in the 
> process.
>
> Vitaly Budovski, does this fix the problem as well?
>   

I think that's probably a good idea. Handling everything in 
WINED3DFMT_UNKNOWN causes far too many problems. This does fix the 
regression in AOE (as long as you add the WINED3DFMT_P8 format to the 
switch statements).

Thanks Chris!



More information about the wine-devel mailing list