The option to limit the number of savestates with autoincrement doesn't appear to be working.

First and foremost consider this:

• Only RetroArch bugs should be filed here. Not core bugs or game bugs
• This is not a forum or a help section, this is strictly developer oriented

Description

It's as said in the title, savestates go over the limit. And i also noticed that sometimes the 'wrong' savestate seems to be deleted, when i try to load and nothing happens except a message 'savestate was loaded, like what retroarch thinks the index is and what it really is is was different.

I don't have a theory, but i have auto-increment on, a 10 savestates limit, and autosavestate (on exit) turned off and autosavestate load turned on (but since there is none, nothing should happen).

Expected behavior

Well first the limit to work. I'm pretty sure i tried both with the index starting at -1 (Auto) and from 0.

I retried it now with puae and a limit of 2 savestates (to be faster) and it never cycled but continued on.

with -1 (auto) as the start slot it said 'Saved state to slot #0' (which btw, isn't it wrong? I thought slot 0 was always skipped but maybe it's because i have autoincrement and cores start with 0 instead of -1) then continued for 1, 2, 3, 4 and never rolled back to 0.

with it starting at 0 it said 'Saved state to slot #1' and continued on through 1-4 without a rollover.

Actual behavior

The limit isn't working. There is something screwy sometimes where savestates 'disappear' or at least can't be loaded after heavy savestate-load cycles with this configuration which i'm not sure is the cause.

Steps to reproduce the bug

1. [First step]
2. [Second step]
3. [and so on...]

Bisect Results

[Try to bisect and tell us when this started happening]

Version/Commit

You can find this information under Information/System Information

• RetroArch: 2329a36f54

Environment information

• OS: [The operating system you're running]
• Compiler: [In case you are running local builds]

Start by reproducing the report with RetroArch, the puae core, autoincrement enabled, a limit of 2, and start slots of -1 and 0. Compare the saved-state messages and loading behavior; done means the configured limit cycles through the expected slots without states disappearing or becoming unloadable.