Landstalker, possibly general .srm file bug?

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Remove any Landstalker .srm files from your "saves" folder
2. Load up the game
3. Get to profile screen

What is the expected output? What do you see instead?

The game doesn't produce an .srm file, and leaves you with four corrupt 
profiles on startup. I'm not sure what happens if you already have an .srm file 
generated by the emulator, since I didn't have one.

What version of the emulator are you using (official, SVN revision,...)?

r593

Please provide any additional information below (Video settings, Console
region,...)

All save-related settings are at their defaults. I tried another game, Phantasy 
Star IV, and it created an .srm file. I'm not sure if there are any other games 
that have this problem, but there probably are.

Original issue reported on code.google.com by SixaxisS...@gmail.com on 30 May 2011 at 6:44

[GoogleCodeExporter]

First, .srm files would only be created if the SRAM is actually modified by the 
game itself (and if you have Auto-SRAM feature enabled off course) so saying 
"the game doesn't produce an .srm file" without saying "when" does not tell 
much.

Please try with official version (1.5.0) and see if it works better, it could 
be the change I made to get SOnic Megamix working indeed broke something.

Original comment by ekeeke31@gmail.com on 30 May 2011 at 12:58

[GoogleCodeExporter]

Ah, I was under the impression that they were automatically made upon game 
startup, since that seemed to be the case with the other emulators I've tried 
so far, my mistake. 

Anyway, yeah, the game works fine with the official version, just tested it. 

Original comment by SixaxisS...@gmail.com on 30 May 2011 at 1:11

[GoogleCodeExporter]

Found the bug, it was introduced in r582 and should be fixed as soon as I can 
commit code again.

Result was that SRAM was disabled in ALL games except those using bankshifting 
(Phantasy Star 4, Beyond Oasis / Legend of Thor), this should teach me to test 
changes I make more seriously :)

Original comment by ekeeke31@gmail.com on 30 May 2011 at 1:17

• Changed state: Accepted
• Added labels: Type-Defect, Priority-Medium, CoreEmulation

[GoogleCodeExporter]

Fixed in r594

Original comment by ekeeke31@gmail.com on 30 May 2011 at 7:36

• Changed state: Verified