From e344d379d274f91850e8ca7c0f450a82b3c04c86 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sun, 27 Sep 2015 12:44:54 +0200 Subject: update semantics of PBD::remove_directory() 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. --- libs/pbd/test/filesystem_test.cc | 9 +-------- 1 file changed, 1 insertion(+), 8 deletions(-) (limited to 'libs/pbd/test') diff --git a/libs/pbd/test/filesystem_test.cc b/libs/pbd/test/filesystem_test.cc index 1eaba54968..ccd4deab65 100644 --- a/libs/pbd/test/filesystem_test.cc +++ b/libs/pbd/test/filesystem_test.cc @@ -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 -- cgit v1.2.3