[dbus] Don't clear metadata when reloading config file

src/dbus.cpp

@@ -143,7 +143,7 @@ namespace dbusmgr {
 bool dbus_manager::get_media_player_metadata(metadata& meta, std::string name) {
     if (name == "") name = m_active_player;
     if (name == "") return false;
-    meta.clear();
+    meta = {};
     dbus_get_player_property(*this, meta, name.c_str(), "Metadata");
     dbus_get_player_property(*this, meta, name.c_str(), "PlaybackStatus");
     meta.valid = (meta.artists.size() || !meta.title.empty());

src/dbus_info.h

@@ -26,14 +26,6 @@ struct metadata {
     bool valid = false;
     bool got_song_data = false;
     bool got_playback_data = false;
-
-    void clear() {
-        artists.clear();
-        title.clear();
-        album.clear();
-        artUrl.clear();
-        valid = false;
-    }
 };
 
 struct mutexed_metadata {

src/overlay_params.cpp

@@ -529,11 +529,6 @@ parse_overlay_config(struct overlay_params *params,
 
 #ifdef HAVE_DBUS
    if (params->enabled[OVERLAY_PARAM_ENABLED_media_player]) {
-      // lock mutexes for config file change notifier thread
-      {
-         std::lock_guard<std::mutex> lk(main_metadata.mtx);
-         main_metadata.meta.clear();
-      }
       dbusmgr::dbus_mgr.init(params->media_player_name);
    } else {
       dbusmgr::dbus_mgr.deinit();