#8416 closed patch
GOB: Possible inter_renewTimeInVars() fix
Reported by: | eriktorbjorn | Owned by: | sev- |
---|---|---|---|
Priority: | normal | Component: | Engine: Gob |
Version: | Keywords: | ||
Cc: | Game: |
Description
This patch is pure guesswork, but...
I noticed there was some code to get current year, month and date that had been commented out. This patch re-enables that code and modifies it to work like the one in the SCUMM engine, so it should be portable enough.
It also moves the code to inter_renewTimeInVars() and changes that function to fill out the remaining time variables with the current time rather than the amount of time that has passed since ScummVM was started.
Does anyone know what variable 7 is supposed to contain? It's currently always set to 0.
Ticket imported from: #1181639. Ticket imported from: patches/521.
Attachments (1)
Change History (7)
by , 20 years ago
Attachment: | gobtime.diff added |
---|
comment:1 by , 20 years ago
Owner: | set to |
---|
comment:2 by , 20 years ago
Setting variable 7 to 0 matches the disassembly. I have no idea what the meaning of it is... maybe the game scripts set/use it somewhere.
comment:3 by , 20 years ago
one other comment: because we don't know what var7 is, it might not be a good idea to set it to 0 in renewTimeVars(). Maybe only set it to 0 in game_playTot() where renewTimeVars() is called?
comment:4 by , 20 years ago
Status: | new → closed |
---|
comment:6 by , 6 years ago
Component: | → Engine: Gob |
---|
Patch against an April 12 CVS snapshot