More sanitizer fixes

CMakeLists.txt

@@ -129,6 +129,7 @@ if (CREATURELIB_TESTS)
     # Add a definition for the test library
     target_compile_definitions(CreatureLibTests PRIVATE TESTS_BUILD)
     if (SANITIZER_TESTS AND NOT WINDOWS)
+        target_compile_options(CreatureLib PRIVATE -fsanitize=address,undefined,leak,integer,nullability -fno-sanitize-recover=all)
         target_compile_options(CreatureLibTests PRIVATE -fsanitize=address,undefined,leak,integer,nullability -fno-sanitize-recover=all)
         target_link_options(CreatureLibTests PRIVATE -fsanitize=address,undefined,leak,integer,nullability -fno-sanitize-recover=all)
     endif ()

src/Battling/Flow/ChoiceQueue.cpp

@@ -22,6 +22,9 @@ bool CreatureLib::Battling::ChoiceQueue::MoveCreatureChoiceNext(CreatureLib::Bat
     for (size_t i = choiceIndex - 1; i >= _current && i != SIZE_MAX; i--) {
         // Place everything in one spot later.
         _queue[i + 1] = _queue[i];
+        if (i == 0) {
+            break;
+        }
     }
     // Set the creature choice up next.
     _queue[_current] = creatureChoice;

src/Battling/Models/CreateCreature.cpp

@@ -40,7 +40,7 @@ Creature* CreateCreature::Create() {
     }
     auto identifier = this->_identifier;
     if (identifier == 0) {
-        identifier = rand.Get();
+        identifier = rand.GetUnsigned();
     }
     auto gender = this->_gender;
     if (gender == static_cast<Library::Gender>(-1)) {

src/Battling/Models/Creature.hpp

@@ -34,7 +34,7 @@ namespace CreatureLib::Battling {
         Library::Gender _gender;
         u8 _coloring;
 
-        u32 _currentHealth = -1;
+        u32 _currentHealth = 0;
 
         f32 _weight;
         f32 _height;