Opened 8 months ago

Closed 8 months ago

#14621 closed defect (fixed)

SCUMM: VAR_VIDEOMODE warning when loading savegames for HE version 70

Reported by: eriktorbjorn Owned by: AndywinXp
Priority: normal Component: Engine: SCUMM
Version: Keywords:
Cc: Game: Putt-Putt Joins the Parade

Description

I made a savegame for Putt-Putt Joins the Parade. (Very early in the game, so I see no need to attach it.) When I loaded it, I got the following warning:

WARNING: Loading savegame with a different render mode setting. Glitches might occur!

The game is version 6, HE version 70. That should mean it runs on an instance of ScummEngine_v70he. That means setupScummVars() will set up VAR_VIDEOMODE to be variable number 49. (In later HE versions, I think it's not used.)

However, resetScummVars() will only initialize VAR(VAR_VIDEOMODE) for games where the HE version is less than 70. So when saveLoadWithSerializer() calls setVideoModeVarToCurrentConfig(), it's likely to change the value of VAR(VAR_VIDEOMODE) thus triggering the warning.

I'm guessing the error is that the game gets a VAR_VIDEOMODE variable, because it doesn't seem possible to change the rendering mode for the game. But I don't dare to change this because a) I'm not sure, and b) if I'm right, are there other variables that the game also shouldn't have but that it inherits from earlier SCUMM versions? How serious would that be?

Change History (2)

comment:1 by AndywinXp, 8 months ago

Thanks, I'll check what the original should actually do and fix it

comment:2 by AndywinXp, 8 months ago

Owner: set to AndywinXp
Resolution: fixed
Status: newclosed

Done :)

Note: See TracTickets for help on using tickets.