[Scummvm-git-logs] scummvm master -> 9246adb9061923ccfa689dbbfaa229975720351d

[AndywinXp]

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:
9246adb906 SCUMM: COMI: Fix file being opened and not closed


Commit: 9246adb9061923ccfa689dbbfaa229975720351d
    https://github.com/scummvm/scummvm/commit/9246adb9061923ccfa689dbbfaa229975720351d
Author: AndywinXp (andywinxp at gmail.com)
Date: 2024-12-28T12:13:12+01:00

Commit Message:
SCUMM: COMI: Fix file being opened and not closed

This fixes #15633:
"SCUMM: COMI: Saving over an existing save loads that save instead"

...whoops.

This also raises the debug level of a related print which caused
noise on the debug log.

Changed paths:
    engines/scumm/saveload.cpp
    engines/scumm/script_v8.cpp


diff --git a/engines/scumm/saveload.cpp b/engines/scumm/saveload.cpp
index 3cabc0edc32..a27242947eb 100644
--- a/engines/scumm/saveload.cpp
+++ b/engines/scumm/saveload.cpp
@@ -311,6 +311,8 @@ void ScummEngine::copyHeapSaveGameToFile(int slot, const char *saveName) {
 		delete saveFile;
 	}
 
+	delete heapSaveFile;
+
 	if (saveFailed)
 		debug(1, "State save as '%s' FAILED", fileName.c_str());
 	else
diff --git a/engines/scumm/script_v8.cpp b/engines/scumm/script_v8.cpp
index 2e45d477331..49f7a14b2de 100644
--- a/engines/scumm/script_v8.cpp
+++ b/engines/scumm/script_v8.cpp
@@ -1149,7 +1149,7 @@ void ScummEngine_v8::o8_kernelSetFunctions() {
 		_saveTemporaryState = false;
 		break;
 	case 28:	// stampShotEnqueue
-		debug(0, "o8_kernelSetFunctions: stampShotEnqueue(%d, %d, %d, %d, %d, %d)", args[1], args[2], args[3], args[4], args[5], args[6]);
+		debug(8, "o8_kernelSetFunctions: stampShotEnqueue(%d, %d, %d, %d, %d, %d)", args[1], args[2], args[3], args[4], args[5], args[6]);
 		stampShotEnqueue(args[1], args[2], args[3], args[4], args[5], args[6]);
 		break;
 	case 29:	// setKeyScript