Resolves #197.

Checklist

• [x] I have read the coding conventions
• [x] I have added a changelog entry about what my pull request accomplishes

Description

Added level selection to the load game menu. Changes savegames to remove start info and only have current info. Adds an initial version to savegames. ...

Latest test build: Tomb1Main-2.7-170-g97d060c_beta_v6.zip ...

Testing Scenarios

No saves

• [x] Starting Gym should start with Compass
• [x] Starting the game should start with Pistols and Compass
• [x] Starting the game, picking up a Medpack, finishing the level should start the next level with Pistols, Compass and a Medpack
• [x] Starting the game, picking up Golden Idol or other key item, finishing the level should remove such key items
• [x] Starting the game, picking up Shotgun, finishing the level should keep the Shotgun
• [x] Starting the game, playing all the way until Natla's Mines, starting Natla's Mines should remove all weapons
• [x] Starting the game, picking up a Medpack, dying, restarting should start with Pistols and Compass and reset all the stats to 0
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, dying, restarting should start with Pistols, Compass and one Medpack, with stats reset to stats from starting level 2
• [x] Starting the game with the no healing mod, getting hit, dying, restarting should reset Lara's health to back to full
• [x] Starting the game with the no healing mod, getting hit (Lara's health is X), finishing the level, getting hit again (Lara's health is Y), dying, restarting should reset Lara's health back to value X
• [x] Starting the game in new game plus mode should start with all weapons.

Legacy saves

Here we use TombATI to save the game and 2.8 to load the game.

• [x] Starting the game, saving, loading should start with Pistols, Compass and non-zero time stats
• [x] Starting the game, picking up a Medpack, saving, loading should start with Pistols, Compass, Medpack and non-zero stats
• [x] Starting the game, playing all the way until Lost Valley, picking up Shotgun, saving, reloading should keep Shotgun
• [x] Starting the game, playing all the way until Natla's Mines, starting Natla's Mines, saving, reloading should keep weapons removed
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, reloading should start with Pistols, Compass and two Medpacks, with stats being kept from just before the save
• [x] Playing all the way till Tihocan, killing Pierre, saving, reloading should respawn Pierre's drops
• [x] The restart level passport page should be skipped since legacy saves do not support the feature.
• [x] Select level should show an unsupported message if it's enabled in the settings, not shown at all otherwise.
• [x] No healing mod can work however it wants, we don't care about it for legacy saves.

2.6/2.7 saves

Here we use 2.7 build to save the game and 2.8 to load the game.

• [x] Starting the game, saving, loading should start with Pistols, Compass and non-zero time stats
• [x] Starting the game, picking up a Medpack, saving, loading should start with Pistols, Compass, Medpack and non-zero stats
• [x] Starting the game, playing all the way until Lost Valley, picking up Shotgun, saving, reloading should keep Shotgun
• [x] Starting the game, playing all the way until Natla's Mines, starting Natla's Mines, saving, reloading should keep weapons removed
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, reloading should start with Pistols, Compass and two Medpacks, with stats being kept from just before the save
• [x] Playing all the way till Tihocan, killing Pierre, saving, reloading should respawn Pierre's drops
• [x] The restart level passport page should be available upon death.
• [x] Starting the game, picking up a Medpack, saving, loading, dying, restarting should start with Pistols and Compass and reset all the stats to 0
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, loading, dying, restarting should start with Pistols, Compass and one Medpack, with stats reset to stats from starting level 2
• [x] Select level should show an unsupported message even if a new save is made from a loaded 2.6/2.7 save.
• [x] Starting the game with the no healing mod, getting hit, saving, loading, should keep Lara's health
• [x] Starting the game with the no healing mod, getting hit (Lara's health is X), finishing the level, getting hit again (Lara's health is Y), saving, reloading should reset Lara's health back to value Y

2.8+ saves

Here we both save and load with the 2.8 build.

• [x] Starting the game, saving, loading should start with Pistols, Compass and non-zero time stats
• [x] Starting the game, picking up a Medpack, saving, loading should start with Pistols, Compass, Medpack and non-zero stats
• [x] Starting the game, playing all the way until Lost Valley, picking up Shotgun, saving, reloading should keep Shotgun
• [x] Starting the game, playing all the way until Natla's Mines, starting Natla's Mines, saving, reloading should keep weapons removed
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, reloading should start with Pistols, Compass and two Medpacks, with stats being kept from just before the save
• [x] Playing all the way till Tihocan, killing Pierre, saving, reloading should respawn Pierre's drops
• [x] The restart level passport page should be available upon death for any 2.8+ save (not started from a TombATI save).
• [x] Starting the game, picking up a Medpack, saving, loading, dying, restarting should start with Pistols and Compass and reset all the stats to 0
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, loading, dying, restarting should start with Pistols, Compass and one Medpack, with stats reset to stats from starting level 2
• [x] Select level should be available for unlocked levels if it's enabled in the settings, not shown at all otherwise. Lara should start with the items she began the level with.
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, restarting this save from Level 1 should start with Pistols and Compass with zero stats
• [x] Starting the game, picking up a Medpack, finishing the level, picking up another Medpack, saving, restarting this save from Level 2 should start with Pistols and Compass, one Medpack and stats from starting the Level 1
• [x] Starting the game with the no healing mod, getting hit, saving, loading, should keep Lara's health
• [x] Starting the game with the no healing mod, getting hit (Lara's health is X), finishing the level, getting hit again (Lara's health is Y), saving, reloading should reset Lara's health back to value Y
• [x] Select level for Natla's Mines should remove weapons.
• [x] Selecting level from a new game plus save should put the game in new game plus mode.

Resolves #362.

Prepared to get slammed, but as far as I know we have to manually define lists for objects that are killable and pickupable. Not sure if I missed any.

For killables, I had to remove some enemies like the Mummy from Qualopec who can't be killed. But Atlantis has an issue where Lara can kill more than there are killables, but idk what I missed.

For pickups, the different Scions behave differently. I excluded O_SCION_ITEM3 because that is the one you have to shoot.

Checklist

• [X] I have read the coding conventions
• [X] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

For now this branch only demonstrates we can build GUI apps with .NET Core with our Docker toolchain. The .exe is just 7 KB, but needs .NET Runtime 3.5 which hopefully majority of the players have installed on their computers.

Resolves #135.

Checklist

• [x] I have read the coding conventions
• [x] I have added a changelog entry about what my pull request accomplishes

Description

Added a cheat (tab key) to increase the game speed. ...

Resolves #277

The new format uses zlib-compressed BSON. The game always saves in the new format, which uses a new filename convention. In case a given save slot has files in both formats available, the game prioritizes the new format upon loading. Loading old saves is still possible. In theory saving in the legacy format is also possible, but I advise against bringing it back.

Other things:

• made the format resilient. No more crashes when trying to load saves in the new format made for a mod or a custom level, instead there is a log message and the game exits to title.
• removed GAMEINFO.savegame_buffer which was frankly quite ugly. I still need info which save slot to load from upon game start, but it's a lot better as it decouples the game from the savegame structure by a great deal already.
• refactored JSON routines.
• refactored save and loading to cater for the new multi-format philosophy.
• added a Python script that can inspect the new savegame files; it is located in scripts/inspect_save.

One thing I'd like to investigate before we merge, provided everything is OK, is whether these snippets, copied from the legacy loader,

    size_t idx = (size_t)arm->frame_base - (size_t)g_AnimFrames;

are dependent on the size of the g_AnimFrames structure, or retrieve the index.

Would it be possible to add player death count to the level statistics? I think it would be interesting to see this information.

I'm not sure if this should be added to #362 or be its own.

Checklist

• [x] I have read the coding conventions
• [X] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

Added 2 additional custom control schemes (default + 3 custom schemes total). Added customizable keybinds. Refactored option_control.c to support this feature and prepare for #564.

• [X] Refactor option_control.c.
• [X] Add custom keybinds for drawing each weapon, small medipack, large medipack, quick save, quick load, FPS, and bilinear filter.
• [X] Add 2 additional custom control schemes for a total of 3 custom schemes.
• [X] Remove the two column control menu and change to a scrolling list to fit all the new keybinds.
• [x] Lock controls when navigating the controls option to prevent medipack usage, toggling the FPS, toggling bilinear filter, etc.
• [x] Bind action and select to the same key.

Future PRs:

• Add the ability to unbind non essential keys (such as camera controls, FPS, etc).
• Add a reset to defaults button for all keys in the current scheme.
• Don't fallback to default keys (example, if user rebinds alt, the game shouldn't do the custom key and also still jump)
• Test remapping a controller.

...

Checklist

• [x] I have read the coding conventions
• [x] I have added a changelog entry about what my pull request accomplishes

Description

... I'm aware this is not done and does not solve legacy saving/loading. I just wanted to ask you to look over this to confirm this is the correct direction for the data types.

GAME_STATS is now in RESUME_INFO. GAME_INFO now has RESUME_INFO *start and RESUME_INFO *end.

Checklist

• [X] I have read the coding conventions
• [X] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

Adds a vertical center gradient for text boxes that is used for all main menu options to add PS1 style purple backgrounds.

Video: https://streamable.com/q7sl3y ...

When you start the level and fall into the stream, it sucks you down the waterfall as is expected. After you've picked up the three cogs in the level and make your way back to put them in, this happened:

TombATI:

https://user-images.githubusercontent.com/4934209/137548483-d09b6e1b-6e82-4d6e-8d83-ac0a3a5f127d.mp4

TR1Main:

https://user-images.githubusercontent.com/4934209/137548586-585ea41d-9256-4480-9f4c-4352a2d18885.mp4

Tested the full scenario of selecting the cogs in both games and then falling into the stream. If the cogs are actually related or not I don't know of course. In TR1Main, if you save whilst you're swimming in the non-working stream, and load, it'll suddenly suck you down again, as if it was reset.

Hi! I've followed the download and extract of Tomb1Main as per the instructions (download TOMBATI first, then extract and replace files with TOMB1Main. However, whereas TOMBATI opens before I replace it, it does not after I have replaced it with Tomb1Main files. Can you help please?

The problem is that the save crystal is located right above pickup and Lara can't pick it up, she gets stucked in the hitbox of the save crystal. Is there any way to fix this issue like fixing the hitbox of the save crystal or moving it nearby that place so it won't interfere?

Checklist

• [x] I have read the coding conventions
• [x] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

For improving the porting, it would be worth to migrate to SDL_GL functions made available into SDL2. They provide the same functionalities of wgl_ext without the need of platform-specific calls to functions that will work only on Windows. After these changes, wgl_ext.c and wgl_ext.h are not used anymore and they can be safely deleted.

The source src\gfx\gl\gl_core_3_3.c could be also simplified by using SDL_GL_GetProcAddress(), but at the moment the code does not make direct use of platform-specific functions, so I left that file as it is.

fixed rare bug when Lara was dying without playing the crash animation when her resulting health = 0

As it is illustrated on this video: https://www.youtube.com/watch?v=mteF9NCoBG8

Checklist

• [X] I have read the coding conventions
• [X] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Checklist

• [x] I have read the coding conventions
• [ ] I have added a changelog entry about what my pull request accomplishes, or it is an internal change

Description

While compiling my UNIX port of Tomb1Main, I got some error messages like this one:

In file included from src/game/camera.c:7:
src/game/sound.h:19:5: error: unknown type name ‘size_t’
   19 |     size_t num_samples, const char **sample_pointers, size_t *sizes);
      |     ^~~~~~
src/game/sound.h:1:1: note: ‘size_t’ is defined in header ‘<stddef.h>’; did you forget to '#include <stddef.h>’?
  +++ |+#include <stddef.h>

I would like to suggest to add an #include <stddef.h> into global/types.h and clean up the sources by redundant inclusions of stddef.h, stdint.h and stdbool.h, if they already include that file.

I'm wondering if it would be possible to add a reply to this question to "Q&A" section of README.md.

It would be useful to write few instructions for knowing how to make them working, if the engine is able to run (or it may be able to do it in the future if it does not yet). I think that it would be very useful to be able to run them, for testing/evaluating the engine without having the full game.

Closing game is always crashing the second time that function is called:

void S_Input_Shutdown(void) { if (m_Controller) { SDL_GameControllerClose(m_Controller); } }

No idea what exactly that function does, but I managed to fix it somehow by adding a new line to prevent a second pass that always fixes it for me:

void S_Input_Shutdown(void) { if (m_Controller) { SDL_GameControllerClose(m_Controller); m_Controller = 0; } }

Note that I got only one joypad (see the logs) Also added some new logs entries to trace it better: ---

Tomb1Main.crashing_at_second_SDL_GameControllerClose()_pass.log

Tomb1Main.no_crashing_with_m_Controller=0_after_first_SDL_GameControllerClose()_pass.log

Edit: Forgot to mention that it crashes on both branchs. Also crashing with default Tomb1Main.json5 file.