Skip to content
Permalink
Browse files

Core: Use move semantics for Renderer::SaveScreenshot

  • Loading branch information...
CookiePLMonster committed Aug 26, 2019
1 parent ff8f978 commit 37ef5a54c906ae697c76e2724f85f0beab4a3435
Showing with 3 additions and 3 deletions.
  1. +2 −2 Source/Core/VideoCommon/RenderBase.cpp
  2. +1 −1 Source/Core/VideoCommon/RenderBase.h
@@ -369,12 +369,12 @@ Renderer::ConvertStereoRectangle(const MathUtil::Rectangle<int>& rc) const
return std::make_tuple(left_rc, right_rc);
}

void Renderer::SaveScreenshot(const std::string& filename, bool wait_for_completion)
void Renderer::SaveScreenshot(std::string filename, bool wait_for_completion)
{
// We must not hold the lock while waiting for the screenshot to complete.
{
std::lock_guard<std::mutex> lk(m_screenshot_lock);
m_screenshot_name = filename;
m_screenshot_name = std::move(filename);
m_screenshot_request.Set();
}

@@ -196,7 +196,7 @@ class Renderer
float EFBToScaledYf(float y) const;

// Random utilities
void SaveScreenshot(const std::string& filename, bool wait_for_completion);
void SaveScreenshot(std::string filename, bool wait_for_completion);
void DrawDebugText();

// ImGui initialization depends on being able to create textures and pipelines, so do it last.

0 comments on commit 37ef5a5

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