From b8384c0c9199c00b06a03265ce75d684e2c1310f Mon Sep 17 00:00:00 2001
From: fearlessTobi <thm.frey@gmail.com>
Date: Sat, 7 Jul 2018 14:11:49 +0200
Subject: [PATCH] Port #3505 from CItra
---
src/yuzu/bootmanager.cpp | 13 +++++++------
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/src/yuzu/bootmanager.cpp b/src/yuzu/bootmanager.cpp
index 5c17cd0d9c..833085559b 100644
--- a/src/yuzu/bootmanager.cpp
+++ b/src/yuzu/bootmanager.cpp
@@ -127,13 +127,14 @@ void GRenderWindow::moveContext() {
}
void GRenderWindow::SwapBuffers() {
-#if !defined(QT_NO_DEBUG)
- // Qt debug runtime prints a bogus warning on the console if you haven't called makeCurrent
- // since the last time you called swapBuffers. This presumably means something if you're using
- // QGLWidget the "regular" way, but in our multi-threaded use case is harmless since we never
- // call doneCurrent in this thread.
+ // In our multi-threaded QGLWidget use case we shouldn't need to call `makeCurrent`,
+ // since we never call `doneCurrent` in this thread.
+ // However:
+ // - The Qt debug runtime prints a bogus warning on the console if `makeCurrent` wasn't called
+ // since the last time `swapBuffers` was executed;
+ // - On macOS, if `makeCurrent` isn't called explicitely, resizing the buffer breaks.
child->makeCurrent();
-#endif
+
child->swapBuffers();
}
--
GitLab