Opened 5 years ago
Closed 5 years ago
#11174 closed defect (fixed)
SCI: Hoyle 5 Bridge - save hand doesn't work
Reported by: | misterhands | Owned by: | bluegr |
---|---|---|---|
Priority: | normal | Component: | Engine: SCI |
Version: | Keywords: | ||
Cc: | Game: |
Description
Hoyle 5 Bridge Win
and
Hoyle Bridge Win
The save/restore hand feature in Hoyle 5 Bridge and Hoyle Bridge does not work properly. The save dialog is shown instead of the restore dialog when restoring a hand. Also after selecting a saved hand to restore (in the save dialog),
an error message is shown. Please see the pic.
Attachments (1)
Change History (12)
by , 5 years ago
Attachment: | scummvm00000.png added |
---|
comment:1 by , 5 years ago
comment:2 by , 5 years ago
You need to have "pre-saved hand" selected in the options.
Then on a new game the restore dialog is supposed to show up,
but it's the save dialog that does.
comment:3 by , 5 years ago
The first problem is that Hoyle5 has a custom SRestore:doit. It takes a second parameter that identifies the card game. Only bridge and poker use save/restore. Poker isn't supported yet since it requires penguin16.dll which hasn't been reverse engineered.
Bridge calls save and restore from script 700 like this:
Save doit: "default save game" 700
Restore doit: "default save game" 700
Poker does the same from script 1190 and passes 1100 for the second parameter.
GuestAdditions::promptSaveRestoreDefault uses the parameter count to differentiate between save and load calls. It treats them both as Save because there's always more than one parameter.
I patched GuestAdditions::promptSaveRestoreDefault to use Restore mode to test if that took care of the error message, but it didn't, I also got the error about the file being invalid.
comment:8 by , 5 years ago
Most of this is now fixed.
- You can now enable the original save/load screens. You'll probably have to remove the game from ScummVM and re-add it for ScummVM to show that option in settings.
- The save files are no longer corrupted so they can now be restored. Any that you've already saved won't work, you'll still get the message if you try to restore those.
When using the ScummVM save/restore screens, you will still get the Save screen when you should get the Restore screen. That's not fixed yet. But it will still restore a game correctly if you select an existing one.
comment:9 by , 5 years ago
Yes it works as you described. Using the original save/load screen, it works perfectly.
Thanks a lot!
comment:11 by , 5 years ago
Owner: | set to |
---|---|
Resolution: | → fixed |
Status: | new → closed |
I couldn't find how to restore a saved hand. Could you tell me how to access this dialog?