Browse Source

gpu_thread: Remove unused dma_pusher class member variable from ThreadManager

The pusher instance is only ever used in the constructor of the
ThreadManager for creating the thread that the ThreadManager instance
contains. Aside from that, the member is unused, so it can be removed.
nce_cpp
Lioncash 7 years ago
parent
commit
3b04b916bb
  1. 4
      src/video_core/gpu_thread.cpp
  2. 3
      src/video_core/gpu_thread.h

4
src/video_core/gpu_thread.cpp

@ -52,8 +52,8 @@ static void RunThread(VideoCore::RendererBase& renderer, Tegra::DmaPusher& dma_p
} }
ThreadManager::ThreadManager(VideoCore::RendererBase& renderer, Tegra::DmaPusher& dma_pusher) ThreadManager::ThreadManager(VideoCore::RendererBase& renderer, Tegra::DmaPusher& dma_pusher)
: renderer{renderer}, dma_pusher{dma_pusher}, thread{RunThread, std::ref(renderer),
std::ref(dma_pusher), std::ref(state)} {}
: renderer{renderer}, thread{RunThread, std::ref(renderer), std::ref(dma_pusher),
std::ref(state)} {}
ThreadManager::~ThreadManager() { ThreadManager::~ThreadManager() {
// Notify GPU thread that a shutdown is pending // Notify GPU thread that a shutdown is pending

3
src/video_core/gpu_thread.h

@ -4,10 +4,8 @@
#pragma once #pragma once
#include <array>
#include <atomic> #include <atomic>
#include <condition_variable> #include <condition_variable>
#include <memory>
#include <mutex> #include <mutex>
#include <optional> #include <optional>
#include <thread> #include <thread>
@ -177,7 +175,6 @@ private:
private: private:
SynchState state; SynchState state;
VideoCore::RendererBase& renderer; VideoCore::RendererBase& renderer;
Tegra::DmaPusher& dma_pusher;
std::thread thread; std::thread thread;
std::thread::id thread_id; std::thread::id thread_id;
}; };

Loading…
Cancel
Save