More refactoring and duplicate code removal

1 files changed, 0 insertions, 69 deletions

diff --git a/src/openvic/map/Region.cpp b/src/openvic/map/Region.cpp
deleted file mode 100644
index 33092c5..0000000
--- a/src/openvic/map/Region.cpp
+++ /dev/null
@@ -1,69 +0,0 @@
-#include "Region.hpp"
-
-using namespace OpenVic;
-
-bool ProvinceSet::add_province(Province* province) {
-	if (locked) {
-		Logger::error("Cannot add province to province set - locked!");
-		return false;
-	}
-	if (province == nullptr) {
-		Logger::error("Cannot add province to province set - null province!");
-		return false;
-	}
-	if (contains_province(province)) {
-		Logger::error("Cannot add province ", province->get_identifier(), " to province set - already in the set!");
-		return false;
-	}
-	provinces.push_back(province);
-	return true;
-}
-
-void ProvinceSet::lock(bool log) {
-	if (locked) {
-		Logger::error("Failed to lock province set - already locked!");
-	} else {
-		locked = true;
-		if (log) Logger::info("Locked province set with ", size(), " provinces");
-	}
-}
-
-bool ProvinceSet::is_locked() const {
-	return locked;
-}
-
-void ProvinceSet::reset() {
-	provinces.clear();
-	locked = false;
-}
-
-bool ProvinceSet::empty() const {
-	return provinces.empty();
-}
-
-size_t ProvinceSet::size() const {
-	return provinces.size();
-}
-
-void ProvinceSet::reserve(size_t size) {
-	if (locked) {
-		Logger::error("Failed to reserve space for ", size, " items in province set - already locked!");
-	} else {
-		provinces.reserve(size);
-	}
-}
-
-bool ProvinceSet::contains_province(Province const* province) const {
-	return province && std::find(provinces.begin(), provinces.end(), province) != provinces.end();
-}
-
-std::vector<Province*> const& ProvinceSet::get_provinces() const {
-	return provinces;
-}
-
-Region::Region(const std::string_view new_identifier) : HasIdentifier { new_identifier } {}
-
-colour_t Region::get_colour() const {
-	if (provinces.empty()) return FULL_COLOUR << 16;
-	return provinces.front()->get_colour();
-}