Mercurial > hg > openttd
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;