Cleanup: remove BLI_cleanup_file
authorCampbell Barton <ideasman42@gmail.com>
Fri, 6 Mar 2020 13:13:03 +0000 (00:13 +1100)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 6 Mar 2020 13:19:01 +0000 (00:19 +1100)
In practice, references to files rarely have trailing slashes
(by accident). Remove this function in favor of BLI_cleanup_path.

source/blender/blenkernel/intern/bpath.c
source/blender/blenlib/BLI_path_util.h
source/blender/blenlib/intern/path_util.c
source/blender/freestyle/intern/application/Controller.cpp
source/blender/freestyle/intern/system/StringUtils.cpp

index 26e8bbde0098828d63e4a7d409be35212e8d325b..38f0e516bebfc729b6d23be60b8ee2ea0bff6e58 100644 (file)
@@ -807,13 +807,13 @@ bool BKE_bpath_relocate_visitor(void *pathbase_v, char *path_dst, const char *pa
   }
 
   /* Make referenced file absolute. This would be a side-effect of
-   * BLI_cleanup_file, but we do it explicitly so we know if it changed. */
+   * BLI_cleanup_path, but we do it explicitly so we know if it changed. */
   BLI_strncpy(filepath, path_src, FILE_MAX);
   if (BLI_path_abs(filepath, base_old)) {
     /* Path was relative and is now absolute. Remap.
      * Important BLI_cleanup_dir runs before the path is made relative
      * because it wont work for paths that start with "//../" */
-    BLI_cleanup_file(base_new, filepath);
+    BLI_cleanup_path(base_new, filepath);
     BLI_path_rel(filepath, base_new);
     BLI_strncpy(path_dst, filepath, FILE_MAX);
     return true;
index ec911c11a8cc1a3295c20d26132a1db13005a62d..d4e3ebdb2e136a093bf87008d112b99c67f5672a 100644 (file)
@@ -81,12 +81,9 @@ int BLI_stringdec(const char *string, char *head, char *start, unsigned short *n
 void BLI_stringenc(
     char *string, const char *head, const char *tail, unsigned short numlen, int pic);
 
-/* removes trailing slash */
-void BLI_cleanup_file(const char *relabase, char *path) ATTR_NONNULL(2);
-/* same as above but adds a trailing slash */
-void BLI_cleanup_dir(const char *relabase, char *dir) ATTR_NONNULL(2);
-/* doesn't touch trailing slash */
 void BLI_cleanup_path(const char *relabase, char *path) ATTR_NONNULL(2);
+/* Same as above but adds a trailing slash. */
+void BLI_cleanup_dir(const char *relabase, char *dir) ATTR_NONNULL(2);
 
 bool BLI_filename_make_safe(char *fname) ATTR_NONNULL(1);
 bool BLI_path_make_safe(char *path) ATTR_NONNULL(1);
index e3c455fe4170c078995c3a5c12dd942800273ceb..48148fc11344117aa7c36792c30c4ea5f15e87f1 100644 (file)
@@ -269,15 +269,6 @@ void BLI_cleanup_dir(const char *relabase, char *dir)
   BLI_add_slash(dir);
 }
 
-/**
- * Cleanup filepath ensuring no trailing slash.
- */
-void BLI_cleanup_file(const char *relabase, char *path)
-{
-  BLI_cleanup_path(relabase, path);
-  BLI_del_slash(path);
-}
-
 /**
  * Make given name safe to be used in paths.
  *
index ef05d0a3a8fb48df5d1fd445b786cf24d1b92a38..c852cda6c8c9f62b7cd1cd15a895c7262923087e 100644 (file)
@@ -347,7 +347,7 @@ int Controller::LoadMesh(Render *re, ViewLayer *view_layer, Depsgraph *depsgraph
   soc string basename((const char *)qfi.fileName().toAscii().data());
   char cleaned[FILE_MAX];
   BLI_strncpy(cleaned, iFileName, FILE_MAX);
-  BLI_cleanup_file(NULL, cleaned);
+  BLI_cleanup_path(NULL, cleaned);
   string basename = string(cleaned);
 #endif
 
index 5e56dba17bff14f22f51751c12a2522a4a0b2765..23f454c0a0457264942884ade3214fa95eb69281 100644 (file)
@@ -46,7 +46,7 @@ void getPathName(const string &path, const string &base, vector<string> &pathnam
     dir = path.substr(pos, sep - pos);
 
     BLI_strncpy(cleaned, dir.c_str(), FILE_MAX);
-    BLI_cleanup_file(NULL, cleaned);
+    BLI_cleanup_path(NULL, cleaned);
     res = string(cleaned);
 
     if (!base.empty()) {