fix(GLFW): Actually change window attribute

Author: alexytomi

jre_lwjgl3glfw/src/main/java/org/lwjgl/glfw/GLFW.java

@@ -895,26 +895,31 @@ public static void glfwSetWindowMonitor(@NativeType("GLFWwindow *") long window,
     }
 
     public static int glfwGetWindowAttrib(@NativeType("GLFWwindow *") long window, int attrib) {
+        System.out.println("GLFW: Default window GL context is 3.3");
         // Turnip + Zink is the only way to get compliant GL 4.6
         boolean hasTurnipZink = java.util.Objects.requireNonNull(glGetString(GL_RENDERER)).contains("zink") &&
                 java.util.Objects.requireNonNull(glGetString(GL_RENDERER)).contains("Turnip");
         if (hasTurnipZink) {
-            if (attrib == GLFW_CONTEXT_VERSION_MAJOR) return 4;
-            if (attrib == GLFW_CONTEXT_VERSION_MINOR) return 6;
+            System.out.println("GLFW: Turnip+Zink detected, setting window GL context to 4.6");
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MAJOR, 4);
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MINOR, 6);
         }
         // It's been common consensus that virgl exports GL 4.3 on ES 3.2
         // I can't find any sources on this though...well besides the implied claim here
         // https://github.com/PojavLauncherTeam/PojavLauncher/issues/2697#issuecomment-1030837683
         boolean isVirgl = java.util.Objects.requireNonNull(glGetString(GL_RENDERER)).contains("virgl");
         if (isVirgl) {
-            if (attrib == GLFW_CONTEXT_VERSION_MAJOR) return 4;
-            if (attrib == GLFW_CONTEXT_VERSION_MINOR) return 3;
+            System.out.println("GLFW: virglrenderer detected, setting window GL context to 4.3");
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MAJOR, 4);
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MINOR, 6);
         }
-	// MobileGlues returns 4.0.0 as well as has compute shaders (from GL 4.0)
-	boolean isMobileGlues = java.util.Objects.requireNonNull(glGetString(GL_VERSION)).contains("MobileGlues");
-        if (isMobileGlues) { // In MobileGlues, GL_RENDERER returns GPU,( ANGLE, Vulkan,) ES while GL_VERSION returns
-            if (attrib == GLFW_CONTEXT_VERSION_MAJOR) return 4; // 4.0.0 MobileGlues <MobileGlues Release Version>
-            if (attrib == GLFW_CONTEXT_VERSION_MINOR) return 0;
+	      // MobileGlues returns 4.0.0 as well as has compute shaders (from GL 4.0)
+        boolean isMobileGlues = java.util.Objects.requireNonNull(glGetString(GL_VERSION)).contains("MobileGlues");
+        if (isMobileGlues) { // In MobileGlues, GL_RENDERER returns GPU,( ANGLE, Vulkan,) ES
+                            // while GL_VERSION returns 4.0.0 MobileGlues <MobileGlues Release Version>
+            System.out.println("GLFW: MobileGlues detected, setting window GL context to 4.0");
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MAJOR, 4);
+            internalGetWindow(window).windowAttribs.put(GLFW_CONTEXT_VERSION_MINOR, 6);
         }
         // We can assume GL 3.3 for everything else just to be safe stuff
         // doesn't ask for tesselation when we probably don't have any :(