Update window.cpp

Fixed a problem that'd cause setting vsync to behave strangely and overlays to crash.

main.cpp

@@ -2,11 +2,15 @@
 #include "include/rewindow/types/window.h"
 int main() {
     auto* window = new(RWindow);
-    window->init(RenderingAPI::OPENGL, "name",100,100,false);
+    window->init(RenderingAPI::OPENGL, "name",1152,864, false);
     window->setFlag(RWindowFlags::RESIZABLE, false);
+    int i;
     while (true) {
+        if (i <= 10)
         window->pollEvents();
-
+        window->glSwapBuffers();
+        i++;
+        std::cout << i << std::endl;
         if (window->keyDown(SCANCODE::A)) {
             std::cout << "A" << std::endl;
             std::cout << (int64_t) window->getEvent(SCANCODE::A).empty() << std::endl;

src/linux/window.cpp

@@ -1,29 +1,29 @@
 #include <iostream>
 #include "rewindow/types/window.h"
 
+bool vsync = false;
 Window window;
 XEvent xev;
 Display* display = XOpenDisplay(nullptr);
 int defaultScreen = DefaultScreen(display);
-//Visual* visual = DefaultVisual(display,defaultScreen);
 XVisualInfo* visual;
-//int depth = DefaultDepth(display, defaultScreen);
 XSetWindowAttributes xSetWindowAttributes;
 XWindowAttributes windowAttributes;
 Atom wmDeleteWindow;
 XSizeHints hints;
 GLXContext glContext;
+PFNGLXSWAPINTERVALEXTPROC _glXSwapIntervalEXT = nullptr; //It causes issues if it's named just "glXSwapIntervalEXT".
 
-void RWindow::init(RenderingAPI api, const char* title, int width, int height, bool vsync) {
+void RWindow::init(RenderingAPI api, const char* title, int width, int height, bool sync) {
     if (api == RenderingAPI::OPENGL) {
-    xSetWindowAttributes.border_pixel = BlackPixel(display, defaultScreen);
-    xSetWindowAttributes.background_pixel = BlackPixel(display, defaultScreen);
-    xSetWindowAttributes.override_redirect = True;
-    xSetWindowAttributes.event_mask = ExposureMask;
-        setVsyncEnabled(vsync);
-        GLint glAttributes[] = {GLX_RGBA, GLX_DEPTH_SIZE, 24, GLX_DOUBLEBUFFER, None};
+        xSetWindowAttributes.border_pixel = BlackPixel(display, defaultScreen);
+        xSetWindowAttributes.background_pixel = BlackPixel(display, defaultScreen);
+        xSetWindowAttributes.override_redirect = True;
+        xSetWindowAttributes.event_mask = ExposureMask;
+        GLint glAttributes[] = {GLX_CONTEXT_MAJOR_VERSION_ARB, 1, GLX_CONTEXT_MINOR_VERSION_ARB, 4, GL_CONTEXT_COMPATIBILITY_PROFILE_BIT, GLX_RGBA, GLX_DEPTH_SIZE, 24, GLX_DOUBLEBUFFER, None};
         visual = glXChooseVisual(display, defaultScreen, glAttributes);
         glContext = glXCreateContext(display, visual, nullptr, GL_TRUE);
+        vsync = sync;
         xSetWindowAttributes.colormap = XCreateColormap(display, RootWindow(display, defaultScreen), visual->visual,
                                                         AllocNone);
 
@@ -40,6 +40,7 @@ void RWindow::init(RenderingAPI api, const char* title, int width, int height, b
         wmDeleteWindow = XInternAtom(display, "WM_DELETE_WINDOW", False);
         XSetWMProtocols(display, window, &wmDeleteWindow, 1);
         glXMakeCurrent(display, window, glContext);
+        setVsyncEnabled(vsync);
     } else {exit(0);}
 }
 
@@ -200,9 +201,8 @@ void RWindow::glSwapBuffers() {
 }
 
 void RWindow::setVsyncEnabled(bool b) {
-    PFNGLXSWAPINTERVALEXTPROC glXSwapIntervalEXT = nullptr;
-    glXSwapIntervalEXT = (PFNGLXSWAPINTERVALEXTPROC)glXGetProcAddressARB((const GLubyte*)"glXSwapIntervalEXT");
-    glXSwapIntervalEXT(display, None, b);
-
+    vsync = b;
+    _glXSwapIntervalEXT = (PFNGLXSWAPINTERVALEXTPROC)glXGetProcAddressARB((const GLubyte*)"glXSwapIntervalEXT");
+    _glXSwapIntervalEXT(display, window, vsync);
 }