From dbd4b9d07d776c891099c5d0706aa15648292266 Mon Sep 17 00:00:00 2001 From: Robin Gareus Date: Sun, 15 Mar 2020 20:31:57 +0100 Subject: [PATCH] Fix cleanup report #7891 Direct call to Session::cleanup_sources() from ARDOUR_UI::cleanup() didn't zero initialize size. This centralizes initialization, so cleanup_trash_sources() does not need to explicitly clear it. --- libs/ardour/ardour/types.h | 1 + libs/ardour/session_state.cc | 3 --- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/libs/ardour/ardour/types.h b/libs/ardour/ardour/types.h index 6d9d0a9f47..b55e245282 100644 --- a/libs/ardour/ardour/types.h +++ b/libs/ardour/ardour/types.h @@ -662,6 +662,7 @@ enum ScreenSaverMode { }; struct CleanupReport { + CleanupReport () : space (0) {} std::vector paths; size_t space; }; diff --git a/libs/ardour/session_state.cc b/libs/ardour/session_state.cc index c84df9f2f8..6159a7dfae 100644 --- a/libs/ardour/session_state.cc +++ b/libs/ardour/session_state.cc @@ -3704,9 +3704,6 @@ Session::cleanup_trash_sources (CleanupReport& rep) vector::iterator i; string dead_dir; - rep.paths.clear (); - rep.space = 0; - for (i = session_dirs.begin(); i != session_dirs.end(); ++i) { dead_dir = Glib::build_filename ((*i).path, dead_dir_name);