From 940cc843ed99cacf58472566e7ef766432550d4f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?L=C3=A9o=20Lam?= Date: Thu, 3 Aug 2017 16:42:18 +0800 Subject: Config: Fix Movie config loading/saving * Add missing Language setting loading/saving. This was added after the original OnionConfig PR, which is why support for it was missing. * Change MovieConfigLoader to reuse ConfigInfos. Less duplication. * Extract MovieConfigLoader::Save into SaveToDTM. The DTM should use the current config and not just the movie layer. This makes more sense than just saving the movie layer, which may not always exist, and also fixes a crash that would happen when creating a new recording because the movie layer wouldn't exist in that case. (Plus, having to get the loader from the layer and call ChangeDTM on it manually is not very pretty.) --- Source/Core/Common/Config/Layer.cpp | 5 ----- 1 file changed, 5 deletions(-) (limited to 'Source/Core/Common/Config/Layer.cpp') diff --git a/Source/Core/Common/Config/Layer.cpp b/Source/Core/Common/Config/Layer.cpp index 61f90538a0..b613c4366b 100644 --- a/Source/Core/Common/Config/Layer.cpp +++ b/Source/Core/Common/Config/Layer.cpp @@ -109,11 +109,6 @@ const LayerMap& Layer::GetLayerMap() const return m_sections; } -ConfigLayerLoader* Layer::GetLoader() const -{ - return m_loader.get(); -} - bool Layer::IsDirty() const { return std::any_of(m_sections.begin(), m_sections.end(), [](const auto& system) { -- cgit v1.2.3