diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2015-03-11 13:24:11 -0500 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2015-03-11 13:24:47 -0500 |
commit | 59880b7fa37780780f5b614324afe0ef4283681e (patch) | |
tree | bd134b32e7f0bdeef2182a50b43d76cc0d9fce9f /libs/ardour/filesystem_paths.cc | |
parent | f9ce27fd6538cc90be0f6908cb321d546d335d20 (diff) |
efficiency is overrated - don't cache the return value of user_config_directory_name() since it can be called with different arguments
Diffstat (limited to 'libs/ardour/filesystem_paths.cc')
-rw-r--r-- | libs/ardour/filesystem_paths.cc | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/libs/ardour/filesystem_paths.cc b/libs/ardour/filesystem_paths.cc index 50ef0be314..9005bd01a3 100644 --- a/libs/ardour/filesystem_paths.cc +++ b/libs/ardour/filesystem_paths.cc @@ -50,7 +50,7 @@ user_config_directory_name (int version = -1) } const string config_dir_name = string_compose ("%1%2", X_(PROGRAM_NAME), version); - + #if defined (__APPLE__) || defined (PLATFORM_WINDOWS) return config_dir_name; #else @@ -61,9 +61,7 @@ user_config_directory_name (int version = -1) std::string user_config_directory (int version) { - static std::string p; - - if (!p.empty()) return p; + std::string p; #ifdef __APPLE__ |