diff --git a/graphics/glfw/src/main/java/de/staropensource/sosengine/graphics/glfw/classes/GlfwWindow.java b/graphics/glfw/src/main/java/de/staropensource/sosengine/graphics/glfw/classes/GlfwWindow.java index 49f731ff..f3b0e354 100644 --- a/graphics/glfw/src/main/java/de/staropensource/sosengine/graphics/glfw/classes/GlfwWindow.java +++ b/graphics/glfw/src/main/java/de/staropensource/sosengine/graphics/glfw/classes/GlfwWindow.java @@ -57,6 +57,7 @@ public abstract class GlfwWindow extends Window { */ private long identifierLong; + // ------------------------------------------------ [ Window initialization ] ------------------------------------------------ // /** * Creates a new window. * @@ -100,104 +101,6 @@ public abstract class GlfwWindow extends Window { */ protected abstract void initializeGlfwWindow() throws Throwable; - /** - * Updates the window state. - * - * @since v1-alpha2 - */ - @Override - public void updateState() { - // Ensure running on the main thread - if (!Miscellaneous.onMainThread()) - throw new NotOnMainThreadException(); - - // Own context - ownContext(); - - // Set swap interval based on isDisallowTearing setting - glfwSwapInterval(Miscellaneous.getIntegerizedBoolean(GraphicsSubsystemConfiguration.getInstance().isDisallowTearing())); - - // Integer arrays, used for 'size' and 'position' - int[] width = new int[0]; - int[] height = new int[0]; - - // Set window size - glfwGetWindowSize(identifierLong, width, height); - super.setSize(new Vec2i(width[0], height[0])); - - // Set window position - glfwGetWindowPos(identifierLong, width, height); - super.setPosition(new Vec2i(width[0], height[0])); - - // Set window mode - if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_ICONIFIED))) - super.setWindowMode(WindowMode.MINIMIZED); - else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_MAXIMIZED))) - super.setWindowMode(WindowMode.MAXIMIZED); - else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_VISIBLE))) - super.setWindowMode(WindowMode.WINDOWED); - else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_VISIBLE))) - super.setWindowMode(WindowMode.HIDDEN); - - // Set booleans - super.setResizable(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_RESIZABLE))); - super.setOnTop(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_FLOATING))); - super.setTransparent(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_TRANSPARENT_FRAMEBUFFER))); - - // Make Graphics API update it's state - updateGlfwState(); - } - - /** - * Updates the state of the GLFW window. - * - * @throws Throwable throwable - * @since v1-alpha2 - */ - protected abstract void updateGlfwState(); - - /** {@inheritDoc} */ - @Override - public void render() throws NotOnMainThreadException { - // Ensure running on the main thread - if (!Miscellaneous.onMainThread()) - throw new NotOnMainThreadException(); - - glfwSwapBuffers(identifierLong); - glfwPollEvents(); - } - - /** {@inheritDoc} */ - @Override - public boolean isClosureRequested() { - return glfwWindowShouldClose(identifierLong); - } - - /** {@inheritDoc} */ - @Override - public boolean isFocused() { - return Tristate.toBoolean(Miscellaneous.getTristatedInteger(glfwGetWindowAttrib(identifierLong, GLFW_FOCUSED))); - } - - /** {@inheritDoc} */ - public void focus() { - glfwFocusWindow(identifierLong); - } - - /** {@inheritDoc} */ - public void requestAttention() { - glfwRequestWindowAttention(identifierLong); - } - - /** - * Updates the OpenGL context. - * - * @since v1-alpha2 - */ - public void ownContext() { - glfwMakeContextCurrent(identifierLong); - } - /** * (Re-)Creates the GLFW window. * @@ -236,12 +139,16 @@ public abstract class GlfwWindow extends Window { identifierLong = identifier; setIdentifier(String.valueOf(identifier)); + // Own context + ownContext(); + + // Set swap interval based on isDisallowTearing setting + glfwSwapInterval(Miscellaneous.getIntegerizedBoolean(GraphicsSubsystemConfiguration.getInstance().isDisallowTearing())); + // Update the window state - setTitle(getTitle()); setSize(getSize()); setMinimumSize(getMinimumSize()); setMaximumSize(getMaximumSize()); - setPosition(getPosition()); setWindowMode(getWindowMode()); } @@ -252,6 +159,116 @@ public abstract class GlfwWindow extends Window { */ public abstract void setWindowHints(); + // ------------------------------------------------ [ State updates ] ------------------------------------------------ // + /** + * Updates the window state. + * + * @since v1-alpha2 + */ + @Override + public void updateState() { + // Ensure running on the main thread + if (!Miscellaneous.onMainThread()) + throw new NotOnMainThreadException(); + + // Own context + ownContext(); + + // Set swap interval based on isDisallowTearing setting + glfwSwapInterval(Miscellaneous.getIntegerizedBoolean(GraphicsSubsystemConfiguration.getInstance().isDisallowTearing())); + + /* + // Initialize IntBuffers, used for 'size' and 'position' + IntBuffer width = MemoryUtil.memAllocInt(1); + IntBuffer height = MemoryUtil.memAllocInt(1); + + // Update window size + glfwGetFramebufferSize(identifierLong, width, height); + super.setSize(new Vec2i(width.get(0), height.get(0))); + + // Clear IntBuffers + width.clear(); + height.clear(); + + // Update window position + glfwGetWindowPos(identifierLong, width, height); + super.setPosition(new Vec2i(width.get(0), height.get(0))); + */ + + // Update window mode + if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_ICONIFIED))) + super.setWindowMode(WindowMode.MINIMIZED); + else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_MAXIMIZED))) + super.setWindowMode(WindowMode.MAXIMIZED); + else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_VISIBLE))) + super.setWindowMode(WindowMode.WINDOWED); + else if (Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_VISIBLE))) + super.setWindowMode(WindowMode.HIDDEN); + + // Update booleans + super.setResizable(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_RESIZABLE))); + super.setOnTop(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_FLOATING))); + super.setTransparent(Miscellaneous.getBooleanizedInteger(glfwGetWindowAttrib(identifierLong, GLFW_TRANSPARENT_FRAMEBUFFER))); + + // Make Graphics API update it's state + updateGlfwState(); + } + + /** + * Updates the state of the GLFW window. + * + * @throws Throwable throwable + * @since v1-alpha2 + */ + protected abstract void updateGlfwState(); + + // ------------------------------------------------ [ Rendering ] ------------------------------------------------ // + /** {@inheritDoc} */ + @Override + public void render() throws NotOnMainThreadException { + // Ensure running on the main thread + if (!Miscellaneous.onMainThread()) + throw new NotOnMainThreadException(); + + glfwSwapBuffers(identifierLong); + glfwPollEvents(); + } + + // ------------------------------------------------ [ GLFW handling ] ------------------------------------------------ // + /** + * Updates the OpenGL context. + * + * @since v1-alpha2 + */ + public void ownContext() { + glfwMakeContextCurrent(identifierLong); + } + + // ------------------------------------------------ [ Information/Action methods ] ------------------------------------------------ // + /** {@inheritDoc} */ + @Override + public boolean isClosureRequested() { + return glfwWindowShouldClose(identifierLong); + } + + /** {@inheritDoc} */ + @Override + public boolean isFocused() { + return Tristate.toBoolean(Miscellaneous.getTristatedInteger(glfwGetWindowAttrib(identifierLong, GLFW_FOCUSED))); + } + + /** {@inheritDoc} */ + public void focus() { + glfwFocusWindow(identifierLong); + } + + /** {@inheritDoc} */ + public void requestAttention() { + glfwRequestWindowAttention(identifierLong); + } + + // ------------------------------------------------ [ Setter overrides ] ------------------------------------------------ // + /** {@inheritDoc} */ @Override public void setName(@NotNull String name) { diff --git a/graphics/opengl/src/main/java/de/staropensource/sosengine/graphics/opengl/classes/OpenGlWindow.java b/graphics/opengl/src/main/java/de/staropensource/sosengine/graphics/opengl/classes/OpenGlWindow.java index 9c34ee80..aa341d45 100644 --- a/graphics/opengl/src/main/java/de/staropensource/sosengine/graphics/opengl/classes/OpenGlWindow.java +++ b/graphics/opengl/src/main/java/de/staropensource/sosengine/graphics/opengl/classes/OpenGlWindow.java @@ -38,6 +38,7 @@ import static org.lwjgl.glfw.GLFW.*; */ @SuppressWarnings({ "unused" }) public class OpenGlWindow extends GlfwWindow { + // ------------------------------------------------ [ Window initialization ] ------------------------------------------------ // /** * Creates a new window. * @@ -83,6 +84,7 @@ public class OpenGlWindow extends GlfwWindow { glfwWindowHint(GLFW_CONTEXT_CREATION_API, GLFW_EGL_CONTEXT_API); } + // ------------------------------------------------ [ Information/Action methods ] ------------------------------------------------ // /** {@inheritDoc} */ @NotNull @Override