Mercurial > hg > openttd

changeset 15526:5f8dc60f1939 draft

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
(svn r20185) -Codechange: reduce the scope of _palette_animation_counter
author rubidium <rubidium@openttd.org>
date Mon, 19 Jul 2010 16:05:29 +0000
parents b4110108e039
children 87b3b2d2fdf1
files src/gfx.cpp src/openttd.cpp src/variables.h
diffstat 3 files changed, 10 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/src/gfx.cpp
+++ b/src/gfx.cpp
@@ -1212,11 +1212,15 @@
 	_pal_count_dirty = 256;
 }
 
-#define EXTR(p, q) (((uint16)(_palette_animation_counter * (p)) * (q)) >> 16)
-#define EXTR2(p, q) (((uint16)(~_palette_animation_counter * (p)) * (q)) >> 16)
+#define EXTR(p, q) (((uint16)(palette_animation_counter * (p)) * (q)) >> 16)
+#define EXTR2(p, q) (((uint16)(~palette_animation_counter * (p)) * (q)) >> 16)
 
 void DoPaletteAnimations()
 {
+	/* Animation counter for the palette animation. */
+	static int palette_animation_counter = 0;
+	palette_animation_counter += 8;
+
 	Blitter *blitter = BlitterFactoryBase::GetCurrentBlitter();
 	const Colour *s;
 	const ExtraPaletteValues *ev = &_extra_palette_values;
@@ -1226,12 +1230,12 @@
 	const int colour_rotation_amount = (_use_palette == PAL_DOS) ? PALETTE_ANIM_SIZE_DOS : PALETTE_ANIM_SIZE_WIN;
 	Colour old_val[PALETTE_ANIM_SIZE_DOS];
 	const int oldval_size = colour_rotation_amount * sizeof(*old_val);
-	const uint old_tc = _palette_animation_counter;
+	const uint old_tc = palette_animation_counter;
 	uint i;
 	uint j;
 
 	if (blitter != NULL && blitter->UsePaletteAnimation() == Blitter::PALETTE_ANIMATION_NONE) {
-		_palette_animation_counter = 0;
+		palette_animation_counter = 0;
 	}
 
 	Colour *palette_pos = &_cur_palette[PALETTE_ANIM_SIZE_START];  // Points to where animations are taking place on the palette
@@ -1277,7 +1281,7 @@
 
 	/* Radio tower blinking */
 	{
-		byte i = (_palette_animation_counter >> 1) & 0x7F;
+		byte i = (palette_animation_counter >> 1) & 0x7F;
 		byte v;
 
 		if (i < 0x3f) {
@@ -1337,7 +1341,7 @@
 	}
 
 	if (blitter != NULL && blitter->UsePaletteAnimation() == Blitter::PALETTE_ANIMATION_NONE) {
-		_palette_animation_counter = old_tc;
+		palette_animation_counter = old_tc;
 	} else {
 		if (memcmp(old_val, &_cur_palette[PALETTE_ANIM_SIZE_START], oldval_size) != 0) {
 			/* Did we changed anything on the palette? Seems so.  Mark it as dirty */
--- a/src/openttd.cpp
+++ b/src/openttd.cpp
@@ -1323,7 +1323,6 @@
 
 	extern int _caret_timer;
 	_caret_timer += 3;
-	_palette_animation_counter += 8;
 	CursorTick();
 
 #ifdef ENABLE_NETWORK
--- a/src/variables.h
+++ b/src/variables.h
@@ -19,9 +19,6 @@
 /* Amount of game ticks */
 VARDEF uint16 _tick_counter;
 
-/* NOSAVE: Used in palette animations only, not really important. */
-VARDEF int _palette_animation_counter;
-
 VARDEF uint32 _realtime_tick;
 
 VARDEF byte _display_opt;