ardour/livetrax
13
0
Fork 0
Code

update semantics of PBD::remove_directory()

Browse Source 
Remove the directory recursively including itself. The function
is used in two places only:

* LV2Plugin::add_state() -- no change, remove tmp. state
* Session::save_as()  -- on error, remove target

In both cases removing the folder itself is correct.
This commit is contained in:
Robin Gareus 2015-09-27 12:44:54 +02:00
parent cf761d4f67
commit e344d379d2
3 changed files with 4 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
libs/pbd/file_utils.cc
View File

@ -476,6 +476,7 @@ void
remove_directory (const std::string& dir)
{
remove_directory_internal (dir, 0, 0, false);
g_rmdir (dir.c_str());
}
string

4
libs/pbd/pbd/file_utils.h
View File

@ -235,9 +235,9 @@ LIBPBD_API int clear_directory (const std::string& dir, size_t* size = 0,
/**
* Remove all the contents of a directory recursively.
* @note dir will not be removed
* including the dir itself (`rm -rf $dir`)
*
* @param dir The directory to remove files from.
* @param dir The directory to remove recursively
*/
LIBPBD_API void remove_directory (const std::string& dir);

9
libs/pbd/test/filesystem_test.cc
View File

@ -335,14 +335,7 @@ FilesystemTest::testRemoveDirectory ()
PBD::remove_directory (output_dir_path);
// doesn't actually remove directory though...just contents
CPPUNIT_ASSERT (Glib::file_test (output_dir_path, Glib::FILE_TEST_IS_DIR));
files_in_output_dir.clear ();
PBD::get_paths (files_in_output_dir, output_dir_path, false, true);
CPPUNIT_ASSERT (files_in_output_dir.size () == 0);
CPPUNIT_ASSERT (!Glib::file_test (output_dir_path, Glib::FILE_TEST_EXISTS));
}
void