[Scummvm-git-logs] scummvm master -> 9337e704bf9216bf51ce762c339d38163efb61b5

[djsrv]

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:
9337e704bf DIRECTOR: Improve movie load debugging


Commit: 9337e704bf9216bf51ce762c339d38163efb61b5
    https://github.com/scummvm/scummvm/commit/9337e704bf9216bf51ce762c339d38163efb61b5
Author: djsrv (dservilla at gmail.com)
Date: 2020-07-16T18:15:20-04:00

Commit Message:
DIRECTOR: Improve movie load debugging

Changed paths:
    engines/director/movie.cpp
    engines/director/resource.cpp
    engines/director/stage.cpp


diff --git a/engines/director/movie.cpp b/engines/director/movie.cpp
index a76a7ea01a..a65e0ac1c6 100644
--- a/engines/director/movie.cpp
+++ b/engines/director/movie.cpp
@@ -256,7 +256,7 @@ void Movie::loadSharedCastsFrom(Common::String filename) {
 	sharedCast->setFileName(filename);
 
 	debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
-	debug(0, "@@@@ Loading Shared cast '%s'", filename.c_str());
+	debug(0, "@@@@   Loading shared cast '%s'", filename.c_str());
 	debug(0, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 
 	_sharedCast = new Cast(this, true);
diff --git a/engines/director/resource.cpp b/engines/director/resource.cpp
index c4015a4a65..5f31ccf869 100644
--- a/engines/director/resource.cpp
+++ b/engines/director/resource.cpp
@@ -52,7 +52,9 @@ Archive *DirectorEngine::createArchive() {
 }
 
 Common::Error Stage::loadInitialMovie() {
-	debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\nObtaining movie name\n");
+	debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
+	debug(0, "@@@@   Loading initial movie");
+	debug(0, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 	Common::String movie = (_vm->getGameGID() == GID_TESTALL) ? getNextMovieFromQueue().movie : _vm->getEXEName();
 
 	if (g_director->getPlatform() == Common::kPlatformWindows)
diff --git a/engines/director/stage.cpp b/engines/director/stage.cpp
index 068095144e..37ba1dcbde 100644
--- a/engines/director/stage.cpp
+++ b/engines/director/stage.cpp
@@ -380,7 +380,7 @@ bool Stage::step() {
 	// finish last movie
 	if (_currentMovie && _currentMovie->getScore()->_playState == kPlayStopped) {
 		debugC(3, kDebugEvents, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
-		debugC(3, kDebugEvents, "@@@@   Movie name '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
+		debugC(3, kDebugEvents, "@@@@   Finishing movie '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
 		debugC(3, kDebugEvents, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 
 		_currentMovie->getScore()->stopPlay();
@@ -420,7 +420,10 @@ bool Stage::step() {
 
 		_currentMovie = new Movie(this);
 		_currentMovie->setArchive(mov);
-		debug(0, "Switching to movie '%s'", _currentMovie->getMacName().c_str());
+
+		debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
+		debug(0, "@@@@   Switching to movie '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
+		debug(0, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 
 		g_lingo->resetLingo();
 		if (sharedCast && sharedCast->_castArchive
@@ -436,15 +439,13 @@ bool Stage::step() {
 
 	// play current movie
 	if (_currentMovie) {
-		if (_newMovieStarted || debugChannelSet(3, kDebugEvents)) {
-			debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
-			debug(0, "@@@@   Movie name '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
-			debug(0, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
-		}
-
 		switch (_currentMovie->getScore()->_playState) {
 		case kPlayNotStarted:
 			{
+				debug(0, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
+				debug(0, "@@@@   Loading movie '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
+				debug(0, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
+
 				bool goodMovie = _currentMovie->loadArchive();
 
 				// If we came in a loop, then skip as requested
@@ -467,6 +468,9 @@ bool Stage::step() {
 			}
 			// fall through
 		case kPlayStarted:
+			debugC(3, kDebugEvents, "\n@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@");
+			debugC(3, kDebugEvents, "@@@@   Stepping movie '%s' in '%s'", _currentMovie->getMacName().c_str(), _currentPath.c_str());
+			debugC(3, kDebugEvents, "@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@\n");
 			_currentMovie->getScore()->step();
 			return true;
 		default: