- Fixed: The identity translation for players should not be remapped through GPalette.Remap[].

The graphics are already mapped through this, so we don't need to do it again when translating.
  Moreover, if there was no duplicate of color 0, but there was a different color with a 
  duplicate, this means we end up drawing players with the duplicated color wherever it should
  be color 0. (Standard translations already had this right.)

SVN r3417 (trunk)
This commit is contained in:
Randy Heit 2012-03-10 00:26:42 +00:00
parent 29369b4d34
commit 2f3ea885c5

View file

@ -887,7 +887,7 @@ static void R_CreatePlayerTranslation (float h, float s, float v, const FPlayerC
{ {
for (i = 0; i < table->NumEntries; ++i) for (i = 0; i < table->NumEntries; ++i)
{ {
table->Remap[i] = GPalette.Remap[i]; table->Remap[i] = i;
} }
memcpy(table->Palette, GPalette.BaseColors, sizeof(*table->Palette) * table->NumEntries); memcpy(table->Palette, GPalette.BaseColors, sizeof(*table->Palette) * table->NumEntries);
} }