[5/5] ddraw: Emulate a 24 bit depth format without stencil or padding

[Stefan Dösinger]

Am 23.08.2011 um 00:53 schrieb Henri Verbeet:
> On 23 August 2011 00:38, Stefan Dösinger <stefandoesinger at gmx.at> wrote:
>>> How is WINED3DFMT_D24_UNORM different from WINED3DFMT_X8D24_UNORM?
>> It has a different application-visible ddraw format.
> I'm not sure that really justifies adding duplicate formats in
> wined3d.
I see 3 alternatives:

1) Just change the ddraw pixelformat for X8D24. That would be covered by test results from some Windows HW(Radeon 9000M, WinXP), but we might break some games

2) Leave the status quo and blame bug 22434 on the app. That would be covered by test results from another Windows machine(Radeon X1600, WinXP)

3) Advertise both X8D24 versions if X8D24 is supported. Change ddraw_surface_init to make sure we don't have to convert back from the wined3d format. Then, once device_parent_create_depth_stencil is gone, we'll never convert wined3d depth formats to ddraw formats and we can support both X8D24 versions in ddraw without having two versions in wined3d.

> Also, calling this D24_UNORM violates wined3d format naming
> conventions. D24_UNORM would be a 3 byte format without padding.
The ddraw format is already wrong. It claims to be 3 byte without padding, but as the test shows it actually has 1 padding byte. I think at the heart of this issue is that driver vendors couldn't agree how to represent a 3 byte depth with one byte padding format in a DDPIXELFORMAT structure.


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110823/35e76c82/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: PGP.sig
Type: application/pgp-signature
Size: 841 bytes
Desc: Signierter Teil der Nachricht
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20110823/35e76c82/attachment.pgp>