Fix BLI_rename returned success w/ missing source path
authorCampbell Barton <ideasman42@gmail.com>
Wed, 3 Feb 2016 04:57:24 +0000 (15:57 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 3 Feb 2016 04:57:24 +0000 (15:57 +1100)
source/blender/blenlib/intern/fileops.c

index 401bdd445e5bc099e13b440b56e67f29ebc5699f..db4b3bcf20ce91bf9e4bdbace65f0e6f79807fe4 100644 (file)
@@ -720,6 +720,8 @@ int   BLI_access(const char *filename, int mode)
 /**
  * Deletes the specified file or directory (depending on dir), optionally
  * doing recursive delete of directory contents.
+ *
+ * \return zero on success (matching 'remove' behavior).
  */
 int BLI_delete(const char *file, bool dir, bool recursive)
 {
@@ -1043,9 +1045,14 @@ bool BLI_dir_create_recursive(const char *dirname)
        return ret;
 }
 
+/**
+ * \return zero on success (matching 'rename' behavior).
+ */
 int BLI_rename(const char *from, const char *to)
 {
-       if (!BLI_exists(from)) return 0;
+       if (!BLI_exists(from)) {
+               return 1;
+       }
        
        if (BLI_exists(to))
                if (BLI_delete(to, false, false)) return 1;