BLADERUNNER: Possible fix for clang warning for subtitles

1 files changed, 11 insertions, 41 deletions

diff --git a/engines/bladerunner/subtitles.cpp b/engines/bladerunner/subtitles.cpp
index 8edf27cec2..3a21625080 100644
--- a/engines/bladerunner/subtitles.cpp
+++ b/engines/bladerunner/subtitles.cpp
@@ -55,41 +55,7 @@ namespace BladeRunner {
  * DONE - OK - CHECK what happens in VQA when no corresponding TRE subs file?
  */
 
-const Common::String Subtitles::SUBTITLES_FONT_FILENAME_EXTERNAL = "SUBTLS_E.FON";
-
-/*
-* All entries need to have the language code appended (after a '_').
-* And all entries should get the suffix extension ".TRx"; the last letter in extension "TR*" should also be the language code
-* If/When adding new Text Resources here --> Update kMaxTextResourceEntries and also update method getIdxForSubsTreName()
-*/
-const Common::String Subtitles::SUBTITLES_FILENAME_PREFIXES[kMaxTextResourceEntries] = {
-	"INGQUO",           // 0 // (in-game subtitles, not VQA subtitles)
-	"WSTLGO",           // 1 // all game (language) versions have the English ('E') version of WSTLGO
-	"BRLOGO",           // 2 // all game (language) versions have the English ('E') version of BRLOGO
-	"INTRO",            // 3
-	"MW_A",             // 4
-	"MW_B01",           // 5
-	"MW_B02",           // 6
-	"MW_B03",           // 7
-	"MW_B04",           // 8
-	"MW_B05",           // 9
-	"INTRGT",           // 10
-	"MW_C01",           // 11
-	"MW_C02",           // 12
-	"MW_C03",           // 13
-	"MW_D",             // 14
-	"END04A",           // 15
-	"END04B",           // 16
-	"END04C",           // 17
-	"END06",            // 18
-	"END01A",           // 19
-	"END01B",           // 20
-	"END01C",           // 21
-	"END01D",           // 22
-	"END01E",           // 23
-	"END01F",           // 24
-	"END03"             // 25
-};
+// Note: Expected subtitles font file name int the SUBTITLES.MIX file is "SUBTLS_E.FON" (SUBTITLES_FONT_FILENAME_EXTERNAL)
 
 /**
 * Subtitles Constructor
@@ -129,6 +95,8 @@ Subtitles::~Subtitles() {
 // Init is kept separated from constructor to allow not loading up resources if subtitles system is disabled
 //
 void Subtitles::init(void) {
+	const Common::String &SUBTITLES_FONT_FILENAME_EXTERNAL = "SUBTLS_E.FON";
+
 	if (_subtitlesSystemInactive) {
 		return;
 	}
@@ -137,11 +105,12 @@ void Subtitles::init(void) {
 	for (int i = 0; i < kMaxTextResourceEntries; i++) {
 		_vqaSubsTextResourceEntries[i] = new TextResource(_vm);
 		Common::String tmpConstructedFileName = "";
-		if (SUBTITLES_FILENAME_PREFIXES[i] == "WSTLGO" || SUBTITLES_FILENAME_PREFIXES[i] == "BRLOGO") {
-			tmpConstructedFileName = SUBTITLES_FILENAME_PREFIXES[i] + "_E"; // Only English versions of these exist
+		const Common::String &prefixInFilenamesTable = SUBTITLES_FILENAME_PREFIXES[i];
+		if (prefixInFilenamesTable == "WSTLGO" || prefixInFilenamesTable == "BRLOGO") {
+			tmpConstructedFileName = prefixInFilenamesTable + "_E"; // Only English versions of these exist
 		}
 		else {
-			tmpConstructedFileName = SUBTITLES_FILENAME_PREFIXES[i] + "_" + _vm->_languageCode;
+			tmpConstructedFileName = prefixInFilenamesTable + "_" + _vm->_languageCode;
 		}
 
 		if ( _vqaSubsTextResourceEntries[i]->open(tmpConstructedFileName)) {
@@ -184,11 +153,12 @@ void Subtitles::setSubtitlesSystemInactive(bool flag) {
 int Subtitles::getIdxForSubsTreName(const Common::String &treName) const {
 	Common::String tmpConstructedFileName = "";
 	for (int i = 0; i < kMaxTextResourceEntries; ++i) {
-		if (SUBTITLES_FILENAME_PREFIXES[i] == "WSTLGO" || SUBTITLES_FILENAME_PREFIXES[i] == "BRLOGO") {
-			tmpConstructedFileName = SUBTITLES_FILENAME_PREFIXES[i] + "_E"; // Only English versions of these exist
+		const Common::String &prefixInFilenamesTable = SUBTITLES_FILENAME_PREFIXES[i];
+		if (prefixInFilenamesTable == "WSTLGO" || prefixInFilenamesTable == "BRLOGO") {
+			tmpConstructedFileName = prefixInFilenamesTable + "_E"; // Only English versions of these exist
 		}
 		else {
-			tmpConstructedFileName = SUBTITLES_FILENAME_PREFIXES[i] + "_" + _vm->_languageCode;
+			tmpConstructedFileName = prefixInFilenamesTable + "_" + _vm->_languageCode;
 		}
 		if (tmpConstructedFileName == treName) {
 			return i;