Added glfwIsInitialized function #2065

Frostie314159 · 2022-03-30T20:43:46Z

This adds a glfwIsInitialized function, that just accesses the internal initialized paramter of the GLFWlibrary object.

This makes it more direct and convenient to check if glfw is already initialized.

As requested in glfw#2036

wintertime · 2022-04-02T07:23:49Z

There is more than one way to help users with the underlying problem:

I'd guess, having this function would be the simplest addition to code around mutliple libraries in one application not knowing if they still need to initialize GLFW. It would just be a higher burden on users to alw 97E2 ays check.
If glfwInit would just error out with something like GLFW_ERROR_ALREADY_INITIALIZED (instead of silently ignoring the call) that would solve the problem without having to add a new function. Users would then just check this and avoid calling glfwTerminate if some other part already initialized GLFW. I'm just not sure if that meets compatibility requirements. User code would be about as complicated as 1.
The most convenient way for users without making the implementation much more complicated than now would be to make initialized a counter. On glfwInit check the counter then only call the initialization code if it was 0, but always increase it by 1. On glfwTerminate error (or maybe ignore) if the counter is 0, decrease it by 1 and call termination code when it reaches 0 after it was decresed. That would be compatible with current user code and enable very easy use (just call glfwInit and glfwTerminate irregardless of other users).

Added glfwIsInitialized function

a262671

Frostie314159 requested a review from elmindreda as a code owner March 30, 2022 20:43

wintertime mentioned this pull request Apr 2, 2022

Improve init and termination with counting #2070

Open

tlf30 mentioned this pull request Sep 15, 2022

Check if GLFW has been initialized #2036

Open

Frostie314159 closed this Apr 10, 2024

Provide feedback