HYPNO: The game save is not loaded in the correct save position.

[windlepoons]

When I save a save game after a puzzle and load it, I am before the puzzle and not after.

Example 'SP' puzzle attached.

Marvel Comics Spider-Man: The Sinister Six (German version)
ScummVM 2.6.0pre99-gbdc149bd5e8 (Jun 16 2022 06:47:23)
OS: Siduction (Debian/sid)

[neuromancer]

Hey, thanks for the bug report. Just to clarify, you save the game after completing the puzzle or before completing it?

[windlepoons]

Replying to neuromancer:

Hey, thanks for the bug report. Just to clarify, you save the game after completing the puzzle or before completing it?

Before the puzzle, and after the completing puzzles (= new scene, then saving).

[neuromancer]

I think it should be solved here: ​https://github.com/scummvm/scummvm/commit/4cd06a5022db66b5b6e0106300b6b692798878d8

However, that is in master, and according to the information you provided, you are using a binary from 2.6.0 release candidate. This is fine, but we should confirm the fix in master before.

So I need you to compile latest master branch and re-test. The saved games *should* be compatible between 2.6.0 and master.

When you confirm the issue, I will proceed to fix it for the rest of the hardcoded puzzles, because I think it affects others as well. Once this is confirmed, I will cherry pick that fix into 2.6.0 for the upcoming release. Let me know if you need help with instructions on how to compile it (or you can wait for tomorrow's daily build)

[windlepoons]

Hint for face puzzle

[windlepoons]

I compiled the master branch and now it loads correctly after *this* puzzle, other puzzles do not.

I attached another save games.

[neuromancer]

Great, please test ​https://github.com/scummvm/scummvm/commit/48e32b28b7403336f99d55e85b3be9352411865f

That should fix the loading for other puzzles. Please test if you can :)

[windlepoons]

Replying to neuromancer:

Great, please test ​https://github.com/scummvm/scummvm/commit/48e32b28b7403336f99d55e85b3be9352411865f

That should fix the loading for other puzzles. Please test if you can :)

The bank puzzle works, but not the roof puzzle (attached). I hope I don't have to test all the puzzles.

[neuromancer]

Thanks for the testing so far. No need to test every puzzle, most of the puzzles are implemented as "scenes" from game data. These only allow to save when a scene is complete, that is why the roof puzzle cannot be saved in the middle, until the segment is complete (I think it is after you get the video from Chip's office.

The issue here are hardcoded puzzles, which we needed to re-done like the color code, the SP panel and the bank matrix one.

[neuromancer]

This was ported to the 2.6 branch. If you have time, please test ​https://github.com/scummvm/scummvm/commit/363cfcce07b21cc9aea2a071431976756987b7ce so we can close the issue.

[neuromancer]

This should be fixed as much as possible already.