summaryrefslogtreecommitdiff
path: root/src/video/video_driver.cpp
diff options
context:
space:
mode:
authorPatric Stout <truebrain@openttd.org>2021-02-24 15:04:41 +0100
committerPatric Stout <github@truebrain.nl>2021-03-08 19:18:55 +0100
commit4610aa7ae3db6cf561e156ba011de3c7b516876d (patch)
treec2e998494f5ed9f1a037b3a493f8919d8287ae57 /src/video/video_driver.cpp
parent56911a86ea2fad84383729ece63c837dc012f14d (diff)
downloadopenttd-4610aa7ae3db6cf561e156ba011de3c7b516876d.tar.xz
Remove: [Video] no longer draw in a thread
Drawing in a thread is a bit odd, and often leads to surprising issues. For example, OpenGL would only allow it if you move the full context to the thread. Which is not always easily done on all OSes. In general, the advise is to handle system events and drawing from the main thread, and do everything else in other threads. So, let's be more like other games. Additionally, putting the drawing routine in a thread was only done for a few targets. Upcoming commit will move the GameLoop in a thread, which will work for all targets.
Diffstat (limited to 'src/video/video_driver.cpp')
-rw-r--r--src/video/video_driver.cpp8
1 files changed, 3 insertions, 5 deletions
diff --git a/src/video/video_driver.cpp b/src/video/video_driver.cpp
index cb81f857d..49337c03a 100644
--- a/src/video/video_driver.cpp
+++ b/src/video/video_driver.cpp
@@ -19,7 +19,7 @@
bool _video_hw_accel; ///< Whether to consider hardware accelerated video drivers.
-bool VideoDriver::Tick()
+void VideoDriver::Tick()
{
auto cur_ticks = std::chrono::steady_clock::now();
@@ -66,12 +66,10 @@ bool VideoDriver::Tick()
::InputLoop();
UpdateWindows();
- this->CheckPaletteAnim();
- return true;
+ this->CheckPaletteAnim();
+ this->Paint();
}
-
- return false;
}
void VideoDriver::SleepTillNextTick()