Changeset - r12266:094a74dc3a85
[Not reviewed]
master
0 1 0
rubidium - 15 years ago 2009-06-29 09:32:58
rubidium@openttd.org
(svn r16691) -Codechange: what's the point of getting the fifth dparam and then using that as the second dparam later on? Especially when the fifth dparam is never set and the second dparam is never used.
1 file changed with 2 insertions and 5 deletions:
0 comments (0 inline, 0 general)
src/texteff.cpp
Show inline comments
 
@@ -26,7 +26,6 @@ struct TextEffect {
 
	int32 bottom;
 
	uint16 duration;
 
	uint64 params_1;
 
	uint64 params_2;
 
	TextEffectMode mode;
 
};
 

	
 
@@ -84,7 +83,6 @@ TextEffectID AddTextEffect(StringID msg,
 
	te->y = y - 5;
 
	te->bottom = y + 5;
 
	te->params_1 = GetDParam(0);
 
	te->params_2 = GetDParam(4);
 
	te->mode = mode;
 

	
 
	GetString(buffer, msg, lastof(buffer));
 
@@ -106,7 +104,6 @@ void UpdateTextEffect(TextEffectID te_id
 
	te = &_text_effect_list[te_id];
 
	te->string_id = msg;
 
	te->params_1 = GetDParam(0);
 
	te->params_2 = GetDParam(4);
 

	
 
	/* Update width of text effect */
 
	char buffer[100];
 
@@ -171,7 +168,7 @@ void DrawTextEffects(DrawPixelInfo *dpi)
 
						dpi->left + dpi->width  > te->x &&
 
						dpi->top  + dpi->height > te->y) {
 
					if (te->mode == TE_RISING || (_settings_client.gui.loading_indicators && !IsTransparencySet(TO_LOADING))) {
 
						AddStringToDraw(te->x, te->y, te->string_id, te->params_1, te->params_2);
 
						AddStringToDraw(te->x, te->y, te->string_id, te->params_1, INVALID_STRING_ID);
 
					}
 
				}
 
			}
 
@@ -186,7 +183,7 @@ void DrawTextEffects(DrawPixelInfo *dpi)
 
						dpi->left + dpi->width  > te->x &&
 
						dpi->top  + dpi->height > te->y) {
 
					if (te->mode == TE_RISING || (_settings_client.gui.loading_indicators && !IsTransparencySet(TO_LOADING))) {
 
						AddStringToDraw(te->x, te->y, (StringID)(te->string_id - 1), te->params_1, te->params_2);
 
						AddStringToDraw(te->x, te->y, (StringID)(te->string_id - 1), te->params_1, INVALID_STRING_ID);
 
					}
 
				}
 
			}
0 comments (0 inline, 0 general)