Hi, > from IWineD3DDeviceImpl_CreatePalette: > + memset(&palVersion, 0, sizeof(LOGPALETTE)); > + object->hpal = CreatePalette(&palVersion); > + memcpy(&object->palVersion, &palVersion, sizeof(LOGPALETTE)); > + /* FIXME Using the palVersion member from IWineD3DPaletteImpl for > CreatePalette + * directly causes a heap corruption. Why? In original > ddraw this was a WORD, + * which failed too. Confused ... > + */ > + object->palNumEntries = IWineD3DPaletteImpl_Size(Flags); > > That can't really work. This only creates a palette with zero entries. > You need to set palNumEntries before calling CreatePalette. > Attached patch( applies on top of yours ) should fix it( i.e. do it the > same way it is done in the current implementation). > But the text colors in AoE are still lacking :/. > Needs some more investigation ... I'll look at this, the whole palette code is a bit messy. My suspicion is that IDirectDrawPaletteImpl_SetEntries / IWineD3DPaletteImpl_SetEntries doesn't update all surfaces that use a palette. I also have to take IWineD3DDevice::SetCurrentTexturePalette, IWineD3DDevice::GetCurrentTexturePalette as well as IWineD3DDevice::SetPaletteEntries and GetPaletteEntries in account.
pgp6mJk8g7LRl.pgp
Description: PGP signature
