From 03a94d1264b0d18257e7e59d2958548a9852a61b Mon Sep 17 00:00:00 2001
From: Enno Rehling <enno.rehling@gmail.com>
Date: Mon, 14 Nov 2016 01:35:45 +0100
Subject: [PATCH] remove unused and broken backup functionality

---
 src/bind_eressea.c     |  2 +-
 src/gmtool.c           |  4 +---
 src/kernel/save.c      | 22 +---------------------
 src/kernel/save.h      |  4 +---
 src/kernel/save.test.c |  2 +-
 5 files changed, 5 insertions(+), 29 deletions(-)

diff --git a/src/bind_eressea.c b/src/bind_eressea.c
index d7683ec18..279ce8f4e 100755
--- a/src/bind_eressea.c
+++ b/src/bind_eressea.c
@@ -19,7 +19,7 @@ void eressea_free_game(void) {
 }
 
 int eressea_read_game(const char * filename) {
-    return readgame(filename, false);
+    return readgame(filename);
 }
 
 int eressea_write_game(const char * filename) {
diff --git a/src/gmtool.c b/src/gmtool.c
index 608ad3c71..05997ed6e 100644
--- a/src/gmtool.c
+++ b/src/gmtool.c
@@ -823,8 +823,7 @@ void loaddata(state *st)  {
 
     askstring(st->wnd_status->handle, "save as:", datafile, sizeof(datafile));
     if (strlen(datafile) > 0) {
-        create_backup(datafile);
-        readgame(datafile, false);
+        readgame(datafile);
         st->modified = 0;
     }
 }
@@ -834,7 +833,6 @@ void savedata(state *st)  {
 
     askstring(st->wnd_status->handle, "save as:", datafile, sizeof(datafile));
     if (strlen(datafile) > 0) {
-        create_backup(datafile);
         remove_empty_units();
         writegame(datafile);
         st->modified = 0;
diff --git a/src/kernel/save.c b/src/kernel/save.c
index 143912832..fa2beabc6 100644
--- a/src/kernel/save.c
+++ b/src/kernel/save.c
@@ -378,22 +378,6 @@ race_t typus2race(unsigned char typus)
     return NORACE;
 }
 
-void create_backup(char *file)
-{
-#ifdef HAVE_LINK
-    char bfile[MAX_PATH];
-    int c = 1;
-
-    if (access(file, R_OK) == 0)
-        return;
-    do {
-        sprintf(bfile, "%s.backup%d", file, c);
-        c++;
-    } while (access(bfile, R_OK) == 0);
-    link(file, bfile);
-#endif
-}
-
 static void read_alliances(struct gamedata *data)
 {
     storage *store = data->store;
@@ -1476,7 +1460,7 @@ static int cb_sb_maxlevel(spellbook_entry *sbe, void *cbdata) {
     return 0;
 }
 
-int readgame(const char *filename, bool backup)
+int readgame(const char *filename)
 {
     int n;
     char path[MAX_PATH];
@@ -1490,10 +1474,6 @@ int readgame(const char *filename, bool backup)
     log_debug("- reading game data from %s", filename);
     join_path(datapath(), filename, path, sizeof(path));
 
-    if (backup) {
-        create_backup(path);
-    }
-
     F = fopen(path, "rb");
     if (!F) {
         perror(path);
diff --git a/src/kernel/save.h b/src/kernel/save.h
index acaf74a3b..134fd8007 100644
--- a/src/kernel/save.h
+++ b/src/kernel/save.h
@@ -45,7 +45,7 @@ extern "C" {
 
     int readorders(const char *filename);
     int creategame(void);
-    int readgame(const char *filename, bool backup);
+    int readgame(const char *filename);
     int writegame(const char *filename);
 
     int current_turn(void);
@@ -65,8 +65,6 @@ extern "C" {
     void write_ship(struct gamedata *data, const struct ship *sh);
     struct ship *read_ship(struct gamedata *data);
 
-    void create_backup(char *file);
-
     int write_game(struct gamedata *data);
     int read_game(struct gamedata *data);
 
diff --git a/src/kernel/save.test.c b/src/kernel/save.test.c
index b4d79f4f8..6a2758a1d 100644
--- a/src/kernel/save.test.c
+++ b/src/kernel/save.test.c
@@ -38,7 +38,7 @@ static void test_readwrite_data(CuTest * tc)
     char path[MAX_PATH];
     test_setup();
     CuAssertIntEquals(tc, 0, writegame(filename));
-    CuAssertIntEquals(tc, 0, readgame(filename, false));
+    CuAssertIntEquals(tc, 0, readgame(filename));
     join_path(datapath(), filename, path, sizeof(path));
     CuAssertIntEquals(tc, 0, remove(path));
     test_cleanup();