Skip to content
Permalink
Browse files

Truly use emplace_back in CommandLineConfigLayerLoader constructor

  • Loading branch information...
CookiePLMonster committed Aug 11, 2019
1 parent dd8bc7b commit 80e8037e0b5e1894140cf4d7f8a1881df0428fe9
Showing with 4 additions and 4 deletions.
  1. +4 −4 Source/Core/UICommon/CommandLineParse.cpp
@@ -26,11 +26,10 @@ class CommandLineConfigLayerLoader final : public Config::ConfigLayerLoader
: ConfigLayerLoader(Config::LayerType::CommandLine)
{
if (!video_backend.empty())
m_values.emplace_back(std::make_tuple(Config::MAIN_GFX_BACKEND.location, video_backend));
m_values.emplace_back(Config::MAIN_GFX_BACKEND.location, video_backend);

if (!audio_backend.empty())
m_values.emplace_back(
std::make_tuple(Config::MAIN_DSP_HLE.location, ValueToString(audio_backend == "HLE")));
m_values.emplace_back(Config::MAIN_DSP_HLE.location, ValueToString(audio_backend == "HLE"));

// Arguments are in the format of <System>.<Section>.<Key>=Value
for (const auto& arg : args)
@@ -45,7 +44,8 @@ class CommandLineConfigLayerLoader final : public Config::ConfigLayerLoader
if (system)
{
m_values.emplace_back(
std::make_tuple(Config::ConfigLocation{*system, section, key}, value));
Config::ConfigLocation{std::move(*system), std::move(section), std::move(key)},
std::move(value));
}
}
}

0 comments on commit 80e8037

Please sign in to comment.
You can’t perform that action at this time.