changeset 3555:3f714fb7c43f draft

(svn r4429) - CodeChange[newgrf]: Action0, prop 0x0D. use calloc to allocate memory and zero it out and change the loop a little
author Darkvater <Darkvater@openttd.org>
date Sat, 15 Apr 2006 16:03:45 +0000
parents 95f3020d5695
children e2b102ead39a
files newgrf.c
diffstat 1 files changed, 16 insertions(+), 18 deletions(-) [+]
line wrap: on
line diff
--- a/newgrf.c
+++ b/newgrf.c
@@ -1014,29 +1014,27 @@
 				Bridge *bridge = &_bridge[brid + i];
 				byte tableid = grf_load_byte(&buf);
 				byte numtables = grf_load_byte(&buf);
-				byte table, sprite;
 
 				if (bridge->sprite_table == NULL) {
-					/* Allocate memory for sprite table pointers and set to NULL */
-					bridge->sprite_table = malloc(7 * sizeof(*bridge->sprite_table));
-					for (table = 0; table < 7; table++)
-						bridge->sprite_table[table] = NULL;
+					/* Allocate memory for sprite table pointers and zero out */
+					bridge->sprite_table = calloc(7, sizeof(*bridge->sprite_table));
 				}
 
-				for (table = tableid; table < tableid + numtables; table++) {
-					if (table < 7) {
-						if (bridge->sprite_table[table] == NULL) {
-							bridge->sprite_table[table] = malloc(32 * sizeof(**bridge->sprite_table));
-						}
-
-						for (sprite = 0; sprite < 32; sprite++)
-							bridge->sprite_table[table][sprite] = grf_load_dword(&buf);
-					} else {
-						grfmsg(GMS_WARN, "BridgeChangeInfo: Table %d >= 7, skipping.", table);
-						// Skip past invalid data.
-						for (sprite = 0; sprite < 32; sprite++)
-							grf_load_dword(&buf);
+				for (; numtables-- != 0; tableid++) {
+					byte sprite;
+
+					if (tableid >= 7) { // skip invalid data
+						grfmsg(GMS_WARN, "BridgeChangeInfo: Table %d >= 7, skipping.", tableid);
+						for (sprite = 0; sprite < 32; sprite++) grf_load_dword(&buf);
+						continue;
 					}
+
+					if (bridge->sprite_table[tableid] == NULL) {
+						bridge->sprite_table[tableid] = malloc(32 * sizeof(**bridge->sprite_table));
+					}
+
+					for (sprite = 0; sprite < 32; sprite++)
+						bridge->sprite_table[tableid][sprite] = grf_load_dword(&buf);
 				}
 			}
 			break;