1 files changed, 8 insertions, 87 deletions

diff --git a/extension/src/LoadGameCompatibility.cpp b/extension/src/LoadGameCompatibility.cpp
index 3da5f08..bed7442 100644
--- a/extension/src/LoadGameCompatibility.cpp
+++ b/extension/src/LoadGameCompatibility.cpp
@@ -3,89 +3,17 @@
 #include <godot_cpp/classes/file_access.hpp>
 #include <godot_cpp/variant/utility_functions.hpp>
 
-#include "openvic/utility/BMP.hpp"
+#include "openvic-simulation/utility/BMP.hpp"
 
 #include "Utilities.hpp"
 
 using namespace godot;
 using namespace OpenVic;
 
-Error GameSingleton::_load_province_identifier_file_compatibility_mode(String const& file_path) {
-	UtilityFunctions::print("Loading compatibility mode province identifier file: ", file_path);
-
-	const Ref<FileAccess> file = FileAccess::open(file_path, FileAccess::ModeFlags::READ);
-	Error err = FileAccess::get_open_error();
-	if (err != OK || file.is_null()) {
-		UtilityFunctions::push_error("Failed to load compatibility mode province identifier file: ", file_path);
-		if (err == OK) err = FAILED;
-	} else {
-		int line_number = 0;
-		while (!file->eof_reached()) {
-			const PackedStringArray line = file->get_csv_line(";");
-			line_number++;
-
-			if (line.is_empty() || (line.size() == 1 && line[0].is_empty()))
-				continue;
-
-			if (line_number < 2) continue; // skip header line
-			std::string identifier;
-			colour_t colour = NULL_COLOUR;
-			if (line.size() > 0) {
-				identifier = godot_to_std_string(line[0]);
-				if (identifier.empty()) {
-					identifier = std::to_string(game_manager.map.get_province_count() + 1);
-				} else {
-					bool successful = false;
-					const uint64_t val = StringUtils::string_to_uint64(identifier, &successful, 10);
-					if (successful) {
-						if (val <= Province::NULL_INDEX || val > Province::MAX_INDEX) {
-							UtilityFunctions::push_error("Invalid province index ", line[0], " (out of valid range ", Province::NULL_INDEX, " < index <= ", Province::MAX_INDEX, ")");
-							err = FAILED;
-						}
-					} else {
-						UtilityFunctions::push_error("Invalid province index ", line[0], " (not a valid integer)");
-						err = FAILED;
-					}
-				}
-				for (int i = 1; i < 4; ++i) {
-					if (line.size() > i) {
-						if (line[i].is_valid_int()) {
-							const int64_t int_val = line[i].to_int();
-							if (int_val >= NULL_COLOUR && int_val <= FULL_COLOUR) {
-								colour = (colour << 8) | int_val;
-								continue;
-							}
-						} else if (line[i].is_valid_float()) {
-							const double double_val = line[i].to_float();
-							if (std::trunc(double_val) == double_val) {
-								const int64_t int_val = double_val;
-								if (int_val >= NULL_COLOUR && int_val <= FULL_COLOUR) {
-									colour = (colour << 8) | int_val;
-									continue;
-								}
-							}
-						}
-					}
-					colour = NULL_COLOUR;
-					break;
-				}
-			}
-			if (identifier.empty() || colour == NULL_COLOUR) {
-				UtilityFunctions::push_error("Invalid province ID-colour entry \"", line, "\" on line ", line_number, " in file: ", file_path);
-				err = FAILED;
-				continue;
-			}
-			if (game_manager.map.add_province(identifier, colour) != SUCCESS) err = FAILED;
-		}
-	}
-	game_manager.map.lock_provinces();
-	return err;
-}
-
 Error GameSingleton::_load_terrain_variants_compatibility_mode(String const& terrain_image_path, String const& terrain_texturesheet_path) {
 	// Read BMP's palette to determine terrain variant colours which texture they're associated with
 	BMP bmp;
-	if (bmp.open(godot_to_std_string(terrain_image_path).c_str()) != SUCCESS || bmp.read_header() != SUCCESS || bmp.read_palette() != SUCCESS) {
+	if (!(bmp.open(godot_to_std_string(terrain_image_path).c_str()) && bmp.read_header() && bmp.read_palette())) {
 		UtilityFunctions::push_error("Failed to read BMP palette from compatibility mode terrain image: ", terrain_image_path);
 		return FAILED;
 	}
@@ -111,10 +39,11 @@ Error GameSingleton::_load_terrain_variants_compatibility_mode(String const& ter
 	const int32_t slice_size = sheet_width / SHEET_DIMS;
 
 	{
+		static constexpr colour_t TERRAIN_WATER_INDEX_COLOUR = 0xFFFFFF;
 		Ref<Image> water_image = Image::create(slice_size, slice_size, false, terrain_sheet->get_format());
 		ERR_FAIL_NULL_V_EDMSG(water_image, FAILED, "Failed to create water terrain image");
 		water_image->fill({ 0.1f, 0.1f, 0.5f });
-		terrain_variants.add_item({ "terrain_water", 0xFFFFFF, water_image });
+		terrain_variants.add_item({ "terrain_water", TERRAIN_WATER_INDEX_COLOUR, water_image });
 	}
 	Error err = OK;
 	for (int32_t idx = 0; idx < PALETTE_SIZE; ++idx) {
@@ -125,7 +54,7 @@ Error GameSingleton::_load_terrain_variants_compatibility_mode(String const& ter
 			err = FAILED;
 			continue;
 		}
-		if (terrain_variants.add_item({ "terrain_" + std::to_string(idx), palette[idx], terrain_image }) != SUCCESS) err = FAILED;
+		if (!terrain_variants.add_item({ "terrain_" + std::to_string(idx), palette[idx], terrain_image })) err = FAILED;
 	}
 	terrain_variants.lock();
 	if (_generate_terrain_texture_array() != OK) return FAILED;
@@ -133,7 +62,6 @@ Error GameSingleton::_load_terrain_variants_compatibility_mode(String const& ter
 }
 
 Error GameSingleton::load_defines_compatibility_mode(PackedStringArray const& file_paths) {
-	static const std::filesystem::path province_identifier_file = "map/definition.csv";
 	static const std::filesystem::path province_image_file = "map/provinces.bmp";
 	static const std::filesystem::path terrain_image_file = "map/terrain.bmp";
 	static const std::filesystem::path terrain_texture_file = "map/terrain/texturesheet.tga";
@@ -145,23 +73,16 @@ Error GameSingleton::load_defines_compatibility_mode(PackedStringArray const& fi
 
 	Error err = OK;
 
-	if (dataloader.set_roots(roots) != SUCCESS) {
+	if (!dataloader.set_roots(roots)) {
 		Logger::error("Failed to set dataloader roots!");
 		err = FAILED;
 	}
 
-	if (dataloader.load_defines(game_manager) != SUCCESS) {
+	if (!dataloader.load_defines(game_manager)) {
 		UtilityFunctions::push_error("Failed to load defines!");
 		err = FAILED;
 	}
 
-	if (_load_province_identifier_file_compatibility_mode(
-		std_to_godot_string(dataloader.lookup_file(province_identifier_file).string())
-		) != OK) {
-		UtilityFunctions::push_error("Failed to load province identifiers!");
-		err = FAILED;
-	}
-	game_manager.map.lock_water_provinces();
 	game_manager.map.lock_regions();
 	if (_load_terrain_variants_compatibility_mode(
 		std_to_godot_string(dataloader.lookup_file(terrain_image_file).string()),
@@ -177,7 +98,7 @@ Error GameSingleton::load_defines_compatibility_mode(PackedStringArray const& fi
 		UtilityFunctions::push_error("Failed to load map images!");
 		err = FAILED;
 	}
-	if (game_manager.load_hardcoded_defines() != SUCCESS) {
+	if (!game_manager.load_hardcoded_defines()) {
 		UtilityFunctions::push_error("Failed to hardcoded defines!");
 		err = FAILED;
 	}