From 2f3ea885c5eeb8d53c1b2951580abc642fcc7bf1 Mon Sep 17 00:00:00 2001 From: Randy Heit Date: Sat, 10 Mar 2012 00:26:42 +0000 Subject: [PATCH] - 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) --- src/r_data/r_translate.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/r_data/r_translate.cpp b/src/r_data/r_translate.cpp index 06350e0b1..0e29801e5 100644 --- a/src/r_data/r_translate.cpp +++ b/src/r_data/r_translate.cpp @@ -887,7 +887,7 @@ static void R_CreatePlayerTranslation (float h, float s, float v, const FPlayerC { 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); }