Feature: Persistant rotation of numbered auto/netsave after restart (#9397)

It was always starting from 0 on openttd restart.
Now the most recent auto/netsave number will be used as a base to generate the next filename.

1 files changed, 3 insertions, 14 deletions

diff --git a/src/saveload/saveload.cpp b/src/saveload/saveload.cpp
index d771bf526..c8a1a0d29 100644
--- a/src/saveload/saveload.cpp
+++ b/src/saveload/saveload.cpp
@@ -3328,26 +3328,15 @@ SaveOrLoadResult SaveOrLoad(const std::string &filename, SaveLoadOperation fop,
  * @param counter A reference to the counter variable to be used for rotating the file name.
  * @param netsave Indicates if this is a regular autosave or a netsave.
  */
-void DoAutoOrNetsave(int &counter, bool netsave)
+void DoAutoOrNetsave(FiosNumberedSaveName &counter)
 {
 	char buf[MAX_PATH];
 
 	if (_settings_client.gui.keep_all_autosave) {
 		GenerateDefaultSaveName(buf, lastof(buf));
-		if (!netsave) {
-			strecat(buf, ".sav", lastof(buf));
-		} else {
-			strecat(buf, "-netsave.sav", lastof(buf));
-		}
+		strecat(buf, counter.Extension().c_str(), lastof(buf));
 	} else {
-		/* Generate a savegame name and number according to _settings_client.gui.max_num_autosaves. */
-		if (!netsave) {
-			seprintf(buf, lastof(buf), "autosave%d.sav", counter);
-		} else {
-			seprintf(buf, lastof(buf), "netsave%d.sav", counter);
-		}
-
-		if (++counter >= _settings_client.gui.max_num_autosaves) counter = 0;
+		strecpy(buf, counter.Filename().c_str(), lastof(buf));
 	}
 
 	Debug(sl, 2, "Autosaving to '{}'", buf);