Ticket #8740: renameSavefile.patch

backends/saves/savefile.cpp

@@ -32 +32 @@
 
 namespace Common {
 
+bool SaveFileManager::renameSavefile(const char *oldFilename, const char *newFilename) {
+
+        enum {
+                NOTHING_DONE = 0,
+                OPEN_LOAD = 1,
+                OPEN_SAVE = 2,
+                BUFFER_READY = 4,               // for flexibility: we may want not to assert as we do when failing to allocate the buffer
+                COPIED = 8,
+                CAN_COPY = OPEN_SAVE | BUFFER_READY,
+        };
+
+        uint32 status = NOTHING_DONE;
+        InSaveFile *inFile = 0;
+        OutSaveFile *outFile = 0;
+        uint32 size = 0;
+        void *buffer = 0;
+        bool success = false;
+
+        inFile = openForLoading(oldFilename);
+        status |= (inFile) ? OPEN_LOAD : NOTHING_DONE;
+
+        if (status & OPEN_LOAD) {
+                size = inFile->size();
+                buffer = malloc(size);
+                assert(buffer);
+                status |= BUFFER_READY;
+
+                outFile = openForSaving(newFilename);
+                status |= (outFile) ? OPEN_SAVE : NOTHING_DONE;
+
+                if (status & CAN_COPY) {
+                        inFile->read(buffer, size);
+                        if (!inFile->ioFailed()) {
+                                outFile->write(buffer, size);
+                                status |= (outFile->ioFailed()) ? NOTHING_DONE : COPIED;
+                        }
+                }
+        }
+
+        if (status & BUFFER_READY) {
+                free(buffer);
+        }
+
+        if (status & OPEN_SAVE) {
+                outFile->finalize();
+                delete outFile;
+        }
+
+        if (status & OPEN_LOAD) {
+                delete inFile;
+        }
+
+        if (status & COPIED) {
+                success = removeSavefile(oldFilename);
+        }
+
+        return success;
+}
+
 const char *SaveFileManager::getSavePath() const {
 
 #if defined(PALMOS_MODE) || defined(__PSP__)

common/savefile.h

@@ -132 +132 @@
         virtual bool removeSavefile(const char *filename) = 0;
 
         /**
+         * Renames the given savefile.
+         * @param oldFilename Old filename.
+         * @param newFilename New filename.
+         * @return true if no error ocurred. false otherwise.
+         */
+        virtual bool renameSavefile(const char *oldFilename, const char *newFilename);
+
+        /**
          * Request a list of available savegames with a given regex.
          * @param regex Regular expression to match. Wildcards like * or ? are available.
          * returns a list of strings for all present file names.