Related to #3

Some people like to move the executable out of the repo without realizing that the asset directory needs to be moved too. Right now, this results in a cryptic error message about the renderer failing to initialize. Check if asset directory is present and print a useful error message if not.

The only check currently performed is for shader compilation. I should probably have at least a glGetError after all initialization to make sure resources were allocated properly.

The windows.c needs SPACE_SHOOTER_MIN_FRAME_TIME but the constants.h define it as SPACE_SHOOTER_MIN_FRAME_TIME_NS. I also tried to remove the _NS part but then the game just freeze. After looking through the startup code, I saw this:

        if (useSleep && SPACE_SHOOTER_MIN_FRAME_TIME - elapsedTime > 1.0f) {
            DWORD sleepMs = (DWORD) (SPACE_SHOOTER_MIN_FRAME_TIME - elapsedTime);
            Sleep(sleepMs);

The sleepMs was a very high number. If I replace the macro with something else like 1/60 then it would work just fine.

Audio layers currently just return on error. They should release any allocated resources.

As far as I understand, dead entities are only filtered before calling the drawing/rendering functions. While iterating over entities we don't seem to be skipping those with their ->dead[i] flag set. I am not sure if that's correct though.

In the function src/game/game.c/mainGame we call simWorld, which checks for collisions between player bullets and enemies (i.e it sets the ->dead[i] flag), later 'mainGame' also calls simPlayer which checks for collisions between the player and the enemies using the checkPlayerCollision function. But the checkPlayerCollision function doesn't skip "dead" enemies, which makes it possible for "dead" enemies to collide with the player.

main and WinMain in the top-level platform files currently return on error without doing any cleanup. Set up a goto chain to handle cleanup depending on which resources have been allocated (like in platform_loadFile).

If vsync isn't enabled, the game loop will currently run as fast as possible, which is pointless since game state updates are fixed at 60Hz. Setting the loop to max out at a high refresh rate (e.g. 300Hz) would let vsync do its thing but prevent overusing resources when it's not available.

Any plans for a Mac version?

Hi Tarek, thank you for making this project!

For anyone trying to compile this project on windows (Windows 10 Home for me), I ran into a slight problem. After running build.bat in a shell with cl setup, I got an error:

....
..\src\game\entities.c(135): error C2220: the following warning is treated as an error
..\src\game\entities.c(135): warning C4204: nonstandard extension used: non-constant aggregate initializer
...More of these warnings...

The way I solved this for now was to remove the /WX compile option from the build.bat script (basically replace line 8 with the line below):

@set shared_args=/std:c11 /W4 /wd5105 /wd4201 /wd4100 /D SOGL_MAJOR_VERSION=3 /D SOGL_MINOR_VERSION=3 /Fespace-shooter

/WX tells the compiler to treat all warnings as errors, so this probably isn't the best solution! But it's a workaround for now. Im not a C/C++ dev so not sure how to solve "non-constant aggregate initializer". If I figure it out I will post here.

A continuation of #2. A new update came to my qemu fork, and they added support for OpenGL 4.1. However, running with MESA_GL_VERSION_OVERRIDE=3.3 build/sp* (because the version string is for some reason, still at 2.1), fails with

Program failed to link!
Vertex shader failed to compile!
0:1(10): error: GLSL 3.30 is not supported. Supported versions are: 1.10, 1.20, 1.30, 1.00 ES, and 3.00 ES

Fragment shader failed to compile!
0:1(10): error: GLSL 3.30 is not supported. Supported versions are: 1.10, 1.20, 1.30, 1.00 ES, and 3.00 ES

It's currently based on number of ticks, which means it will poll at a different rate on how fast the game loop is running. Set it to a constant rate of 1Hz.

I could compile the project, but it fails to run with Unable to load OpenGL

Currently buffers are allocated on the first draw using glBufferData. Move the allocations to renderer_init and update them in game_draw using glBufferSubData.