zdoom/raze
0
0
Fork 0
mirror of https://github.com/ZDoom/Raze.git synced 2024-11-16 09:21:36 +00:00
Code Issues Projects Releases Packages Wiki Activity

Tweak the conditions under which setbrightness() invalidates GL textures.

Browse source 
Previously, we checked whether the faded palette has changed (by way of CRC)
and invalidated the textures then unless the preserve flags were set.  This
however could lead to wrongly invalidating them under unfortunate circumstances,
e.g. basepal change from CON + tints at the same time before r2620 which
reverted r2232.

Now, only invalidate them if the corresponding preserve flags are clear AND
 * the base palette has really changed OR
 * the palette CRC changed and we were running on software gamma
The latter means that performance-killing invalidations may still happen on
GL platforms lacking HW gamma (for ATI, it's currently only disabled in 8-bit
fullscreen).

Also have a new global 'basepalreset' to fake a basepal change for
setbrightness(), currently used when changing renderers so that going from
Polymer to Polymost and back again will invalidate the Polymer textures
on the second change, potentially re-applying a basepal highpal. (Still
with me?)

git-svn-id: https://svn.eduke32.com/eduke32@2636 1a8010ca-5511-0410-912e-c29ae57300e0
This commit is contained in:
helixhorned 2012-05-05 22:22:34 +00:00
parent 1a2c5ee040
commit ebbb3e304d
1 changed files with 23 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

40
polymer/eduke32/build/src/engine.c
View file

@ -2342,6 +2342,7 @@ int32_t totalclocklock;
char apptitle[256] = "Build Engine";
static uint8_t **basepaltableptr;
static uint8_t basepalreset=1;
uint8_t basepalcount;
uint8_t curbasepal;
@ -13530,18 +13531,21 @@ void setbasepaltable(uint8_t **thebasepaltable, uint8_t thebasepalcount)
// 16: don't reset palfade*
void setbrightness(char dabrightness, uint8_t dapalid, uint8_t flags)
{
int32_t i, j, paldidchange=0;
int32_t i, j, nohwgamma;
const uint8_t *dapal;
int32_t paldidchange, palsumdidchange;
// uint32_t lastbright = curbrightness;
Bassert((flags&(1+4))==0);
if (dapalid >= basepalcount)
dapalid = 0;
paldidchange = (curbasepal != dapalid || basepalreset);
curbasepal = dapalid;
basepalreset = 0;
dapal = basepaltableptr[dapalid];
dapal = basepaltableptr[curbasepal];
if (!(flags&4))
{
@ -13550,9 +13554,8 @@ void setbrightness(char dabrightness, uint8_t dapalid, uint8_t flags)
// vid_gamma = 1.0 + ((float)curbrightness / 10.0);
}
j = 0;
if (setgamma())
j = curbrightness;
nohwgamma = setgamma();
j = nohwgamma ? curbrightness : 0;
for (i=0; i<256; i++)
{
@ -13576,9 +13579,9 @@ void setbrightness(char dabrightness, uint8_t dapalid, uint8_t flags)
static uint32_t lastpalettesum=0;
uint32_t newpalettesum = crc32once((uint8_t *)curpalettefaded, sizeof(curpalettefaded));
paldidchange = (newpalettesum != lastpalettesum);
palsumdidchange = (newpalettesum != lastpalettesum);
if (paldidchange || newpalettesum != g_lastpalettesum)
if (palsumdidchange || newpalettesum != g_lastpalettesum)
{
if ((flags&1) == 0) setpalette(0,256);
}
@ -13589,19 +13592,20 @@ void setbrightness(char dabrightness, uint8_t dapalid, uint8_t flags)
#ifdef USE_OPENGL
if (rendmode >= 3)
{
// XXX: paldidchange should really be "oldpalnum!=newpalnum".
// Currently changing the palette while having a tint effect invalidates
// everything, which is bad!
// only reset the textures if the corresponding preserve flags are clear and
// Only reset the textures if the corresponding preserve flags are clear and
// either (a) the new palette is different to the last, or (b) the brightness
// changed and we couldn't set it using hardware gamma
if (!(flags&2) && paldidchange)
// changed and we couldn't set it using hardware gamma.
// XXX: no-HW-gamma OpenGL platforms will exhibit bad performance with
// simultaneous basepal and tint changes?
const int32_t doinvalidate = (paldidchange || (palsumdidchange && nohwgamma));
if (!(flags&2) && doinvalidate)
gltexinvalidateall();
if (!(flags&8) && paldidchange)
if (!(flags&8) && doinvalidate)
gltexinvalidate8();
#ifdef POLYMER
if ((rendmode == 4) && paldidchange)
if ((rendmode == 4) && doinvalidate)
polymer_texinvalidate();
#endif
}
@ -15993,6 +15997,8 @@ int32_t setrendermode(int32_t renderer)
else renderer = 3;
# endif
basepalreset = 1;
rendmode = renderer;
if (rendmode >= 3)
glrendmode = rendmode;