From 9d8ee2a1d045dea59179fd2403cb2d778f34e7ab Mon Sep 17 00:00:00 2001
From: Vaxry <43317083+vaxerski@users.noreply.github.com>
Date: Wed, 21 Feb 2024 16:56:25 +0000
Subject: [PATCH] core: Nvidia and stuff fixes (#44)

* nvidia and stuff fixes

* no no null confures

* drop stupid roundtrip
---
 src/core/LockSurface.cpp | 35 +++++------------------------------
 1 file changed, 5 insertions(+), 30 deletions(-)

diff --git a/src/core/LockSurface.cpp b/src/core/LockSurface.cpp
index 683e529..d160292 100644
--- a/src/core/LockSurface.cpp
+++ b/src/core/LockSurface.cpp
@@ -47,13 +47,8 @@ CSessionLockSurface::CSessionLockSurface(COutput* output) : output(output) {
     if (fractional) {
         wp_fractional_scale_v1_add_listener(fractional, &fsListener, this);
         viewport = wp_viewporter_get_viewport(g_pHyprlock->getViewporter(), surface);
-        wl_display_roundtrip(g_pHyprlock->getDisplay());
-    } else {
+    } else
         Debug::log(LOG, "No fractional-scale support! Oops, won't be able to scale!");
-    }
-
-    configure(output->size, 0);
-    g_pRenderer->renderLock(*this);
 
     lockSurface = ext_session_lock_v1_get_lock_surface(g_pHyprlock->getSessionLock(), surface, output->output);
 
@@ -63,8 +58,6 @@ CSessionLockSurface::CSessionLockSurface(COutput* output) : output(output) {
     }
 
     ext_session_lock_surface_v1_add_listener(lockSurface, &lockListener, this);
-    wl_display_roundtrip(g_pHyprlock->getDisplay());
-    wl_display_flush(g_pHyprlock->getDisplay());
 }
 
 void CSessionLockSurface::configure(const Vector2D& size_, uint32_t serial_) {
@@ -76,8 +69,7 @@ void CSessionLockSurface::configure(const Vector2D& size_, uint32_t serial_) {
 
     Debug::log(LOG, "Configuring surface for logical {} and pixel {}", logicalSize, size);
 
-    if (serial != 0)
-        ext_session_lock_surface_v1_ack_configure(lockSurface, serial);
+    ext_session_lock_surface_v1_ack_configure(lockSurface, serial);
 
     if (fractional)
         wp_viewport_set_destination(viewport, logicalSize.x, logicalSize.y);
@@ -95,7 +87,7 @@ void CSessionLockSurface::configure(const Vector2D& size_, uint32_t serial_) {
         exit(1);
     }
 
-    if (serial == 0)
+    if (!eglSurface)
         eglSurface = g_pEGL->eglCreatePlatformWindowSurfaceEXT(g_pEGL->eglDisplay, g_pEGL->eglConfig, eglWindow, nullptr);
 
     if (!eglSurface) {
@@ -105,18 +97,7 @@ void CSessionLockSurface::configure(const Vector2D& size_, uint32_t serial_) {
 
     readyForFrame = true;
 
-    if (serial != 0)
-        render();
-
-    if (fractional)
-        wp_viewport_set_destination(viewport, logicalSize.x, logicalSize.y);
-
-    wl_surface_set_buffer_scale(surface, 1);
-    wl_surface_damage_buffer(surface, 0, 0, 0xFFFF, 0xFFFF);
-
-    wl_surface_commit(surface);
-    wl_display_roundtrip(g_pHyprlock->getDisplay());
-    wl_display_flush(g_pHyprlock->getDisplay());
+    render();
 }
 
 static void handleDone(void* data, wl_callback* wl_callback, uint32_t callback_data) {
@@ -138,15 +119,9 @@ void CSessionLockSurface::render() {
     const auto FEEDBACK = g_pRenderer->renderLock(*this);
     frameCallback       = wl_surface_frame(surface);
     wl_callback_add_listener(frameCallback, &callbackListener, this);
+
     eglSwapBuffers(g_pEGL->eglDisplay, eglSurface);
 
-    if (fractional)
-        wp_viewport_set_destination(viewport, logicalSize.x, logicalSize.y);
-
-    wl_surface_damage_buffer(surface, 0, 0, 0xFFFF, 0xFFFF);
-    wl_surface_set_buffer_scale(surface, 1);
-    wl_surface_commit(surface);
-
     needsFrame = FEEDBACK.needsFrame;
 }