[PATCH 1/5] ddraw: Only ddraw7 disallows SetPalette on sublevels.

Henri Verbeet hverbeet at gmail.com
Thu Apr 24 06:25:35 CDT 2014


On 24 April 2014 12:10, Stefan Dösinger <stefan at codeweavers.com> wrote:
> -    if (!(surface->surface_desc.u4.ddpfPixelFormat.dwFlags & (DDPF_PALETTEINDEXED1 | DDPF_PALETTEINDEXED2
> -            | DDPF_PALETTEINDEXED4 | DDPF_PALETTEINDEXED8 | DDPF_PALETTEINDEXEDTO8)))
> -        return DDERR_INVALIDPIXELFORMAT;
> -
>      if (surface->surface_desc.ddsCaps.dwCaps2 & DDSCAPS2_MIPMAPSUBLEVEL)
>          return DDERR_NOTONMIPMAPSUBLEVEL;
>
Note that this inverts the ordering of the pixel format and sub-level
validation. That might even be correct, but the associated test
doesn't cover that case at all.



More information about the wine-devel mailing list