[Scummvm-git-logs] scummvm master -> 20d9525c88ca1aeaf03db6de98c871390baa67b8

[sev-]

This automated email contains information about 1 new commit which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .

Summary:
20d9525c88 DIRECTOR: Load more fields from CastInfo and dump their context (noisily)


Commit: 20d9525c88ca1aeaf03db6de98c871390baa67b8
    https://github.com/scummvm/scummvm/commit/20d9525c88ca1aeaf03db6de98c871390baa67b8
Author: Eugene Sandulenko (sev at scummvm.org)
Date: 2025-02-21T12:59:29+01:00

Commit Message:
DIRECTOR: Load more fields from CastInfo and dump their context (noisily)

Changed paths:
    engines/director/cast.cpp


diff --git a/engines/director/cast.cpp b/engines/director/cast.cpp
index 0b9fc7ca07b..e581e47bd65 100644
--- a/engines/director/cast.cpp
+++ b/engines/director/cast.cpp
@@ -1366,7 +1366,49 @@ void Cast::loadCastInfo(Common::SeekableReadStreamEndian &stream, uint16 id) {
 	// adding tons of ifs, we use this switch()
 	switch (castInfo.strings.size()) {
 	default:
-		warning("Cast::loadCastInfo(): BUILDBOT: extra %d strings", castInfo.strings.size() - 8);
+		warning("Cast::loadCastInfo(): BUILDBOT: extra %d strings for castid %d", castInfo.strings.size() - 15, id);
+		// fallthrough
+	case 15:
+		if (castInfo.strings[14].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 14, id);
+			Common::hexdump(castInfo.strings[14].data, castInfo.strings[14].len);
+		}
+		// fallthrough
+	case 14:
+		if (castInfo.strings[13].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 13, id);
+			Common::hexdump(castInfo.strings[13].data, castInfo.strings[13].len);
+		}
+		// fallthrough
+	case 13:
+		if (castInfo.strings[12].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 12, id);
+			Common::hexdump(castInfo.strings[12].data, castInfo.strings[12].len);
+		}
+		// fallthrough
+	case 12:
+		if (castInfo.strings[11].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 11, id);
+			Common::hexdump(castInfo.strings[11].data, castInfo.strings[11].len);
+		}
+		// fallthrough
+	case 11:
+		if (castInfo.strings[10].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 11, id);
+			Common::hexdump(castInfo.strings[10].data, castInfo.strings[10].len);
+		}
+		// fallthrough
+	case 10:
+		if (castInfo.strings[9].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 10, id);
+			Common::hexdump(castInfo.strings[9].data, castInfo.strings[9].len);
+		}
+		// fallthrough
+	case 9:
+		if (castInfo.strings[8].len) {
+			warning("Cast::loadCastInfo(): BUILDBOT: string #%d for castid %d", 9, id);
+			Common::hexdump(castInfo.strings[8].data, castInfo.strings[8].len);
+		}
 		// fallthrough
 	case 8:
 		if (castInfo.strings[7].len) {