From a7f551cae007c0b5b877ba729668d15eb7f36caa Mon Sep 17 00:00:00 2001 From: Anu <anulithic@gmail.com> Date: Mon, 24 Feb 2025 17:44:35 -0800 Subject: [PATCH] Clarify some comments --- src/js/utilsSlave.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/js/utilsSlave.js b/src/js/utilsSlave.js index f12cd3a9ede..f70c055af63 100644 --- a/src/js/utilsSlave.js +++ b/src/js/utilsSlave.js @@ -1674,7 +1674,7 @@ globalThis.nationOf = function(nation) { country = "Saint Kitts and Nevis"; break; case "Kurdish": - country = "Kurdistan"; // should we say "the Kurdish people" or something instead? Kurdistan is either a vaguely-defined transnational region, or a very tenuously self-governing part of Iraq + country = "Kurdistan"; // We can presume Kurdistan has become a relatively more independent sovereign state than in real life at time of writing break; case "Laotian": country = "Laos"; @@ -1769,7 +1769,7 @@ globalThis.nationOf = function(nation) { case "Vincentian": country = "Saint Vincent and the Grenadines"; break; - case "Zairian": + case "Zairian": // this is a little anachronistic, as the country hasn't been called Zaire since the 1990s, but FC-Author chose to keep it more distinct from the other Congo, and we're sticking with it country = "Zaire"; // other options: the Democratic Republic of the Congo, Congo-Kinshasa, DR Congo, Big Congo break; case "Rhodesian": // no slave's nationality should ever be directly assigned Rhodesian, but it is occasionally used as an alias, so catch it just in case @@ -1780,7 +1780,7 @@ globalThis.nationOf = function(nation) { country = "no country"; // best we can do, shouldn't ever come up break; default: - country = "an unknown country"; // if any new nationalities are ever added, catch them so they can be added + country = "an unknown country"; // if any new nationalities are ever added, catch them so we know to account for them here break; } return country; -- GitLab