openttd-mini: comparison oldloader.c

equal deleted inserted replaced

-:929df0b3484a
+:5569d866ea85
 	while (chunk->type != OC_END) {
 		ptr = chunk->ptr;
 		for (i = 0; i < chunk->amount; i++) {
-			if (ls->failed)
+			if (ls->failed) return false;
-				return false;
 			/* Handle simple types */
 			if ((chunk->type & 0xFF) != 0) {
 				switch (chunk->type & 0xFF) {
-					case OC_NULL:
+					/* Just read the byte and forget about it */
-						/* Just read the byte and forget about it */
+					case OC_NULL: ReadByte(ls); break;
-						ReadByte(ls);
-						break;
 					case OC_CHUNK:
 						/* Call function, with 'i' as parameter to tell which item we
 						 * are going to read */
 						if (!chunk->proc(ls, i)) return false;
 						break;
 					case OC_ASSERT:
 						DEBUG(oldloader, 4)("[OldLoader] Assert point: %x / %x", ls->total_read, chunk->offset + _bump_assert_value);
-						if (ls->total_read != chunk->offset + _bump_assert_value) {
+						if (ls->total_read != chunk->offset + _bump_assert_value) ls->failed = true;
-							ls->failed = true;
-						}
-						break;
 				}
 			} else {
 				uint32 res = 0;
 				/* Reading from the file: bit 16 to 23 have the FILE */
 {
 	/* Check for shared orders, and link them correctly */
 	Vehicle* v;
 	FOR_ALL_VEHICLES(v) {
-		Vehicle* u;
+		Vehicle *u;
 		if (v->type == 0) continue;
 		FOR_ALL_VEHICLES_FROM(u, v->index + 1) {
 			if (u->type == 0) continue;
 extern TileIndex _animated_tile_list[256];
 extern char _name_array[512][32];
 extern uint16 _custom_sprites_base;
-static byte   _old_vehicle_multipler;
+static byte   _old_vehicle_multiplier;
 static uint8  _old_map3[OLD_MAP_SIZE * 2];
 static bool   _new_ttdpatch_format;
 static uint32 _old_town_index;
 static uint16 _old_string_id;
 static uint16 _old_string_id_2;
 	if (_read_ttdpatch_flags) return;
 	_read_ttdpatch_flags = true;
 	/* TTDPatch misuses _old_map3 for flags.. read them! */
-	_old_vehicle_multipler = _old_map3[0];
+	_old_vehicle_multiplier = _old_map3[0];
 	/* Somehow.... there was an error in some savegames, so 0 becomes 1
 	and 1 becomes 2. The rest of the values are okay */
-	if (_old_vehicle_multipler < 2) _old_vehicle_multipler++;
+	if (_old_vehicle_multiplier < 2) _old_vehicle_multiplier++;
 	/* TTDPatch incraeses the Vehicle-part in the middle of the game,
 	so if the multipler is anything else but 1, the assert fails..
 	bump the assert value so it doesn't!
 	(1 multipler == 850 vehicles
 	1 vehicle   == 128 bytes */
-	_bump_assert_value = (_old_vehicle_multipler - 1) * 850 * 128;
+	_bump_assert_value = (_old_vehicle_multiplier - 1) * 850 * 128;
 	/* Check if we have a modern TTDPatch savegame (has extra data all around) */
 	_new_ttdpatch_format = (memcmp(&_old_map3[0x1FFFA], "TTDp", 4) == 0);
 	/* Clean the misused places */
 	for (i = 0x1FE00; i < 0x20000; i++) _old_map3[i] = 0;
 	if (_new_ttdpatch_format)
 		DEBUG(oldloader, 1)("[OldLoader] Found TTDPatch game");
-	DEBUG(oldloader, 1)("[OldLoader] Vehicle-multipler is set to %d (%d vehicles)", _old_vehicle_multipler, _old_vehicle_multipler * 850);
+	DEBUG(oldloader, 1)("[OldLoader] Vehicle-multiplier is set to %d (%d vehicles)", _old_vehicle_multiplier, _old_vehicle_multiplier * 850);
 }
 static const OldChunks town_chunk[] = {
 	OCL_SVAR(   OC_TILE, Town, xy ),
 	OCL_SVAR( OC_FILE_U16 | OC_VAR_U32, Town, population ),
 static uint16 _old_order;
 static const OldChunks order_chunk[] = {
 	OCL_VAR ( OC_UINT16,   1, &_old_order ),
 	OCL_END()
 };
 static bool LoadOldOrder(LoadgameState *ls, int num)
 {
 	if (!AddBlockIfNeeded(&_order_pool, num))
 		error("Orders: failed loading savegame: too many orders");
-	if (!LoadChunk(ls, NULL, order_chunk))
+	if (!LoadChunk(ls, NULL, order_chunk)) return false;
-		return false;
 	AssignOrder(GetOrder(num), UnpackOldOrder(_old_order));
 	/* Relink the orders to eachother (in TTD(Patch) the orders for one
 	vehicle are behind eachother, with OT_NOTHING as indication that
 static const OldChunks depot_chunk[] = {
 	OCL_SVAR(   OC_TILE, Depot, xy ),
 	OCL_VAR ( OC_UINT32,   1, &_old_town_index ),
 	OCL_END()
 };
 static bool LoadOldDepot(LoadgameState *ls, int num)
 {
 	if (!AddBlockIfNeeded(&_depot_pool, num))
 		error("Depots: failed loading savegame: too many depots");
-	if (!LoadChunk(ls, GetDepot(num), depot_chunk))
+	if (!LoadChunk(ls, GetDepot(num), depot_chunk)) return false;
-		return false;
 	if (GetDepot(num)->xy != 0) {
 		GetDepot(num)->town_index = REMAP_TOWN_IDX(_old_town_index);
 	}
 static const OldChunks price_chunk[] = {
 	OCL_VAR (  OC_INT32,   1, &_old_price ),
 	OCL_VAR ( OC_UINT16,   1, &_old_price_frac ),
 	OCL_END()
 };
 static bool LoadOldPrice(LoadgameState *ls, int num)
 {
-	if (!LoadChunk(ls, NULL, price_chunk))
+	if (!LoadChunk(ls, NULL, price_chunk)) return false;
-		return false;
 	/* We use a struct to store the prices, but they are ints in a row..
 	so just access the struct as an array of int32's */
 	((int32*)&_price)[num] = _old_price;
 	_price_frac[num] = _old_price_frac;
 	OCL_VAR ( OC_UINT16,   1, &_old_price_frac ),
 	OCL_NULL( 2 ),         // Junk
 	OCL_END()
 };
 static bool LoadOldCargoPaymentRate(LoadgameState *ls, int num)
 {
-	if (!LoadChunk(ls, NULL, cargo_payment_rate_chunk))
+	if (!LoadChunk(ls, NULL, cargo_payment_rate_chunk)) return false;
-		return false;
 	_cargo_payment_rates[num] = -_old_price;
 	_cargo_payment_rates_frac[num] = _old_price_frac;
 	return true;
 	OCL_SVAR(  OC_UINT8, GoodsEntry, last_speed ),
 	OCL_SVAR(  OC_UINT8, GoodsEntry, last_age ),
 	OCL_END()
 };
 static bool LoadOldGood(LoadgameState *ls, int num)
 {
 	Station *st = GetStation(_current_station_id);
 	return LoadChunk(ls, &st->goods[num], goods_chunk);
 }
 	OCL_NULL( 9 ), // Random junk at the end of this chunk
 	OCL_END()
 };
 static bool LoadOldIndustry(LoadgameState *ls, int num)
 {
 	Industry *i;
 	if (!AddBlockIfNeeded(&_industry_pool, num))
 		error("Industries: failed loading savegame: too many industries");
 	i = GetIndustry(num);
-	if (!LoadChunk(ls, i, industry_chunk))
+	if (!LoadChunk(ls, i, industry_chunk)) return false;
-		return false;
 	if (i->xy != 0) {
 		i->town = GetTown(REMAP_TOWN_IDX(_old_town_index));
 	}
 static const OldChunks player_yearly_chunk[] = {
 	OCL_VAR(  OC_INT32,   1, &_old_yearly ),
 	OCL_END()
 };
 static bool OldPlayerYearly(LoadgameState *ls, int num)
 {
 	int i;
 	Player *p = GetPlayer(_current_player_id);
 	for (i = 0; i < 13; i++) {
-		if (!LoadChunk(ls, NULL, player_yearly_chunk))
+		if (!LoadChunk(ls, NULL, player_yearly_chunk)) return false;
-			return false;
 		p->yearly_expenses[num][i] = _old_yearly;
 	}
 	return true;
 	OCL_SVAR( OC_INT32, PlayerEconomyEntry, performance_history ),
 	OCL_SVAR( OC_FILE_I32 | OC_VAR_I64, PlayerEconomyEntry, company_value ),
 	OCL_END()
 };
 static bool OldPlayerEconomy(LoadgameState *ls, int num)
 {
 	int i;
 	Player *p = GetPlayer(_current_player_id);
-	if (!LoadChunk(ls, &p->cur_economy, player_economy_chunk))
+	if (!LoadChunk(ls, &p->cur_economy, player_economy_chunk)) return false;
-		return false;
 	/* Don't ask, but the number in TTD(Patch) are inversed to OpenTTD */
 	p->cur_economy.income   = -p->cur_economy.income;
 	p->cur_economy.expenses = -p->cur_economy.expenses;
 	for (i = 0; i < 24; i++) {
-		if (!LoadChunk(ls, &p->old_economy[i], player_economy_chunk))
+		if (!LoadChunk(ls, &p->old_economy[i], player_economy_chunk)) return false;
-			return false;
 		p->old_economy[i].income   = -p->old_economy[i].income;
 		p->old_economy[i].expenses = -p->old_economy[i].expenses;
 	}
 	OCL_NULL( 8 ),  // Junk...
 	OCL_END()
 };
 static bool OldLoadAIBuildRec(LoadgameState *ls, int num)
 {
 	Player *p = GetPlayer(_current_player_id);
 	switch (num) {
 	OCL_SVAR(  OC_UINT8, PlayerAI, railtype_to_use ),
 	OCL_SVAR(  OC_UINT8, PlayerAI, route_type_mask ),
 	OCL_END()
 };
 static bool OldPlayerAI(LoadgameState *ls, int num)
 {
 	Player *p = GetPlayer(_current_player_id);
 	return LoadChunk(ls, &p->ai, player_ai_chunk);
 {
 	Player *p = GetPlayer(num);
 	_current_player_id = num;
-	if (!LoadChunk(ls, p, player_chunk))
+	if (!LoadChunk(ls, p, player_chunk)) return false;
-		return false;
 	p->name_1 = RemapOldStringID(_old_string_id);
 	p->president_name_1 = RemapOldStringID(_old_string_id_2);
 	p->money64 = p->player_money;
 	OCL_NULL( 5 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_road_chunk[] = {
 	OCL_SVAR(  OC_UINT8, VehicleRoad, state ),
 	OCL_SVAR(  OC_UINT8, VehicleRoad, frame ),
 	OCL_SVAR( OC_UINT16, VehicleRoad, blocked_ctr ),
 	OCL_SVAR(  OC_UINT8, VehicleRoad, overtaking ),
 	OCL_NULL( 1 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_ship_chunk[] = {
 	OCL_SVAR(  OC_UINT8, VehicleShip, state ),
 	OCL_NULL( 9 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_air_chunk[] = {
 	OCL_SVAR(  OC_UINT8, VehicleAir, pos ),
 	OCL_SVAR(  OC_FILE_U8 | OC_VAR_U16, VehicleAir, targetairport ),
 	OCL_SVAR( OC_UINT16, VehicleAir, crashed_counter ),
 	OCL_SVAR(  OC_UINT8, VehicleAir, state ),
 	OCL_NULL( 5 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_special_chunk[] = {
 	OCL_SVAR( OC_UINT16, VehicleSpecial, unk0 ),
 	OCL_SVAR(  OC_UINT8, VehicleSpecial, unk2 ),
 	OCL_NULL( 7 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_disaster_chunk[] = {
 	OCL_SVAR( OC_UINT16, VehicleDisaster, image_override ),
 	OCL_SVAR( OC_UINT16, VehicleDisaster, unk2 ),
 	OCL_NULL( 6 ), // Junk
 	OCL_END()
 };
 static const OldChunks vehicle_empty_chunk[] = {
 	OCL_NULL( 10 ), // Junk
 	OCL_END()
 };
 static bool LoadOldVehicleUnion(LoadgameState *ls, int num)
 {
 	Vehicle *v = GetVehicle(_current_vehicle_id);
 	uint temp = ls->total_read;
 	bool res;
 	OCL_NULL( 20 ), // Junk at end of struct (TTDPatch has some data in it)
 	OCL_END()
 };
 static bool LoadOldVehicle(LoadgameState *ls, int num)
 {
 	uint i;
 	/* Read the TTDPatch flags, because we need some info from it */
 	ReadTTDPatchFlags();
-	for (i = 0; i < _old_vehicle_multipler; i++) {
+	for (i = 0; i < _old_vehicle_multiplier; i++) {
 		Vehicle *v;
-		_current_vehicle_id = num * _old_vehicle_multipler + i;
+		_current_vehicle_id = num * _old_vehicle_multiplier + i;
 		if (!AddBlockIfNeeded(&_vehicle_pool, _current_vehicle_id))
 			error("Vehicles: failed loading savegame: too many vehicles");
 		v = GetVehicle(_current_vehicle_id);
-		if (!LoadChunk(ls, v, vehicle_chunk))
+		if (!LoadChunk(ls, v, vehicle_chunk)) return false;
-			return false;
 		/* This should be consistent, else we have a big problem... */
 		if (v->index != _current_vehicle_id) {
 			DEBUG(oldloader, 0)("[OldLoader] -- Loading failed - vehicle-array is invalid");
 			return false;
 			v->next = GetVehicle(_old_next_ptr);
 		v->string_id = RemapOldStringID(_old_string_id);
 		/* Vehicle-subtype is different in TTD(Patch) */
-		if (v->type == VEH_Special)
+		if (v->type == VEH_Special) v->subtype = v->subtype >> 1;
-			v->subtype = v->subtype >> 1;
 	}
 	return true;
 }
 	OCL_NULL( 6 ),         // Width of sign, no longer in use
 	OCL_END()
 };
 static bool LoadOldSign(LoadgameState *ls, int num)
 {
 	if (!AddBlockIfNeeded(&_sign_pool, num))
 		error("Signs: failed loading savegame: too many signs");
 	OCL_NULL( 1 ),         // Junk
 	OCL_END()
 };
 static bool LoadOldEngine(LoadgameState *ls, int num)
 {
 	if (!LoadChunk(ls, GetEngine(num), engine_chunk)) return false;
 	/* Make sure wagons are marked as do-not-age */
 	OCL_SVAR(  OC_FILE_U8 | OC_VAR_U16, Subsidy, from ),
 	OCL_SVAR(  OC_FILE_U8 | OC_VAR_U16, Subsidy, to ),
 	OCL_END()
 };
-static bool LoadOldSubsidy(LoadgameState *ls, int num)
+static inline bool LoadOldSubsidy(LoadgameState *ls, int num)
 {
 	return LoadChunk(ls, &_subsidies[num], subsidy_chunk);
 }
 static const OldChunks game_difficulty_chunk[] = {
 	OCL_SVAR( OC_FILE_U16 | OC_VAR_I32, GameDifficulty, economy ),
 	OCL_SVAR( OC_FILE_U16 | OC_VAR_I32, GameDifficulty, line_reverse_mode ),
 	OCL_SVAR( OC_FILE_U16 | OC_VAR_I32, GameDifficulty, disasters ),
 	OCL_END()
 };
-static bool LoadOldGameDifficulty(LoadgameState *ls, int num)
+static inline bool LoadOldGameDifficulty(LoadgameState *ls, int num)
 {
 	return LoadChunk(ls, &_opt.diff, game_difficulty_chunk);
 }
 		_m[i * 4 + 3].extra = GB(b, 6, 2);
 	}
 	return !ls->failed;
 }
 static bool LoadOldMapPart2(LoadgameState *ls, int num)
 {
 	uint i;
 	for (i = 0; i < OLD_MAP_SIZE; i++) {
 		_m[i].m5 = ReadByte(ls);
 	}
 	return !ls->failed;
 }
 static uint32 _old_cur_town_ctr;
 static const OldChunks main_chunk[] = {
 	OCL_ASSERT( 0 ),
 	OCL_VAR ( OC_UINT16,   1, &_date ),
 	/* Below any (if available) extra chunks from TTDPatch can follow */
 	OCL_END()
 };
 static bool LoadOldMain(LoadgameState *ls)
 {
 	int i;
 	/* The first 49 is the name of the game + checksum, skip it */
 	return true;
 }
 void GetOldSaveGameName(char *title, const char *file)
 {
-	FILE *f;
+	FILE *f = fopen(file, "rb");
-	f = fopen(file, "rb");
 	title[0] = 0;
 	title[48] = 0;
-	if (f == NULL)
+	if (f == NULL) return;
-		return;
+	if (fread(title, 1, 48, f) != 48) snprintf(title, 48, "Corrupt file");
-	if (fread(title, 1, 48, f) != 48)
-		snprintf(title, 48, "Corrupt file");
 	fclose(f);
 }
 void GetOldScenarioGameName(char *title, const char *file)

changeset 3626	5569d866ea85
parent 3580	14bf6e7715c8
child 3627	3da11e0769c5