.gitmodules

@@ -13,6 +13,3 @@
 [submodule "external/ImFileDialog"]
 	path = external/ImFileDialog
 	url = https://github.com/dfranx/ImFileDialog.git
-[submodule "external/svector"]
-	path = external/svector
-	url = https://github.com/martinus/svector.git

CMakeLists.txt

@@ -35,8 +35,6 @@ else ()
     message(STATUS "ccache not found")
 endif ()
 
-add_subdirectory(external/svector)
-
 option(BUILD_SHARED_LIBS "Build shared libraries" OFF)
 option(GLFW_BUILD_EXAMPLES "Build the GLFW example programs" OFF)
 option(GLFW_BUILD_TESTS "Build the GLFW test programs" OFF)
@@ -106,7 +104,7 @@ add_executable(BillySheet ${SOURCES})
 
 target_include_directories(BillySheet PRIVATE include include/imgui external/ImFileDialog)
 
-set_target_properties(BillySheet spdlog svector PROPERTIES
+set_target_properties(BillySheet spdlog PROPERTIES
                       CXX_STANDARD 17
                       CXX_STANDARD_REQUIRED ON
                       CXX_EXTENSIONS OFF
@@ -114,7 +112,7 @@ set_target_properties(BillySheet spdlog svector PROPERTIES
                       # UNITY_BUILD ON
 )
 
-set_target_properties(spdlog svector PROPERTIES UNITY_BUILD ON)
+set_target_properties(spdlog PROPERTIES UNITY_BUILD ON)
 
 set_target_properties(glfw PROPERTIES
                       C_STANDARD 11
@@ -124,35 +122,27 @@ set_target_properties(glfw PROPERTIES
                       UNITY_BUILD ON
 )
 
+set_target_properties(spdlog PROPERTIES UNITY_BUILD ON)
 
 target_compile_definitions(BillySheet PRIVATE
                            $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>
                            $<$<CONFIG:Debug>:SPDLOG_ACTIVE_LEVEL=SPDLOG_LEVEL_DEBUG>
                            $<$<CONFIG:Release>:SPDLOG_ACTIVE_LEVEL=SPDLOG_LEVEL_ERROR>)
 
-target_compile_definitions(spdlog PRIVATE $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>)
-target_compile_definitions(svector INTERFACE $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>)
-target_compile_definitions(glfw PRIVATE $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>)
+target_compile_definitions(spdlog PRIVATE
+                           $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>)
+
+target_compile_definitions(glfw PRIVATE
+                           $<$<CONFIG:Debug>:_GLIBCXX_DEBUG>)
 
 target_compile_options(spdlog PRIVATE ${COMPILE_FLAGS})
-target_compile_options(svector INTERFACE ${COMPILE_FLAGS})
 target_compile_options(glfw PRIVATE ${COMPILE_FLAGS})
 target_compile_options(BillySheet PRIVATE ${COMPILE_FLAGS})
 
 target_link_options(spdlog PRIVATE ${LINKER_OPTIONS})
-target_link_options(svector INTERFACE ${LINKER_OPTIONS})
 target_link_options(glfw PRIVATE ${LINKER_OPTIONS})
 target_link_options(BillySheet PRIVATE ${LINKER_OPTIONS})
 
 target_link_libraries(spdlog PRIVATE ${LINKER_FLAGS})
-target_link_libraries(svector INTERFACE ${LINKER_FLAGS})
 target_link_libraries(glfw PRIVATE ${LINKER_FLAGS})
-target_link_libraries(BillySheet glfw
-                      spdlog::spdlog_header_only
-                      svector::svector
-                      OpenGL::OpenGL
-                      GLEW::GLEW
-                      nlohmann_json::nlohmann_json
-                      ${Boost_LIBRARIES}
-                      ${LINKER_FLAGS}
-)
+target_link_libraries(BillySheet glfw spdlog::spdlog_header_only OpenGL::OpenGL GLEW::GLEW nlohmann_json::nlohmann_json ${Boost_LIBRARIES} ${LINKER_FLAGS})

external/svector

@@ -1 +0,0 @@
-Subproject commit 74817cfbdbaceb415508de1ff10a0589bf8f25d7

include/billy_objects.hpp

@@ -10,7 +10,6 @@
 #include <functional>
 #include <variant>
 #include <string_view>
-#include <ankerl/svector.h>
 #include "characteristic/characteristic.hpp"
 
 // helper type for the visitor
@@ -46,7 +45,7 @@ namespace character {
     class BillyObjects final {
     public:
         using billyObject = std::variant<weapons, equipments, tools>;
-        using container = ankerl::svector<billyObject, 3>;
+        using container = std::array<billyObject, 3>;
 
         static constexpr std::string_view sword{ "Sword" };
         static constexpr std::string_view lance{ "Lance" };
@@ -69,7 +68,9 @@ namespace character {
 
         void pop_object(CharacterSheet &sheet) noexcept;
 
-        [[nodiscard]] const container &get_objects() const noexcept { return objects; }
+        [[nodiscard]] std::pair<const container &, std::int8_t> get_objects() const noexcept {
+            return { objects, end_object };
+        }
 
 //        void insert_weapon(weapons weapon, CharacterSheet &sheet) noexcept;
 
@@ -77,13 +78,15 @@ namespace character {
         container objects;
         std::plus<std::uint32_t> plus;
         std::minus<std::uint32_t> minus;
+        std::uint8_t end_object{ 0 };
 
         static void change_carac_weapon(const weapons &arg,
                                         CharacterSheet &sheet,
                                         characteristic::Characteristic &localHabilete,
                                         characteristic::Characteristic &localAdresse,
                                         characteristic::Characteristic &localEndurance,
-                const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &operation) noexcept;
+                                        const std::function<std::uint32_t(std::uint32_t,
+                                                                          std::uint32_t)> &operation) noexcept;
 
         static void change_carac_equipment(const equipments &arg,
                                            CharacterSheet &sheet,
@@ -92,7 +95,8 @@ namespace character {
                                            characteristic::Characteristic &localEndurance,
                                            characteristic::Characteristic &localChance,
                                            const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &primary,
-                const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &complement) noexcept;
+                                           const std::function<std::uint32_t(std::uint32_t,
+                                                                             std::uint32_t)> &complement) noexcept;
 
         static void change_carac_tools(const tools &arg,
                                        CharacterSheet &sheet,
@@ -100,7 +104,8 @@ namespace character {
                                        characteristic::Characteristic &localAdresse,
                                        characteristic::Characteristic &localEndurance,
                                        characteristic::Characteristic &localChance,
-                const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &operation) noexcept;
+                                       const std::function<std::uint32_t(std::uint32_t,
+                                                                         std::uint32_t)> &operation) noexcept;
     };
 }
 

include/character_sheet.hpp

@@ -45,8 +45,6 @@ namespace character {
 
         std::uint32_t damage{ 0 };
 
-        std::uint32_t critique{ 0 };
-
         std::uint32_t glory{ 0 };
 
         std::uint32_t money{ 0 };

src/billy_objects.cpp

@@ -9,8 +9,9 @@ namespace character {
     using characteristic::Characteristic;
 
     void BillyObjects::push_object(const billyObject &object, CharacterSheet &sheet) noexcept {
-        if (objects.size() < 3) {
-            objects.emplace_back(object);
+        if (end_object < 3) {
+            objects.at(end_object) = object;
+            ++end_object;
 
             auto &local_habilete = static_cast<Characteristic &>(sheet.habilete);
             auto &local_adresse = static_cast<Characteristic &>(sheet.adresse);
@@ -48,9 +49,9 @@ namespace character {
     }
 
     void BillyObjects::pop_object(CharacterSheet &sheet) noexcept {
-        if (!objects.empty()) {
-            const billyObject obj = objects.back();
-            objects.pop_back();
+        if (end_object > 0) {
+            const auto &obj = objects.at(end_object - 1);
+            --end_object;
 
             auto &local_habilete = static_cast<Characteristic &>(sheet.habilete);
             auto &local_adresse = static_cast<Characteristic &>(sheet.adresse);
@@ -93,7 +94,8 @@ namespace character {
                                           Characteristic &localAdresse,
                                           Characteristic &localEndurance,
                                           Characteristic &localChance,
-            const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &operation) noexcept {
+                                          const std::function<std::uint32_t(std::uint32_t,
+                                                                            std::uint32_t)> &operation) noexcept {
         switch (arg) {
             case tools::Fourche:
                 localHabilete.materiel = operation(localHabilete.materiel, 1);
@@ -118,7 +120,8 @@ namespace character {
                                               Characteristic &localEndurance,
                                               Characteristic &localChance,
                                               const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &primary,
-            const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &complement) noexcept {
+                                              const std::function<std::uint32_t(std::uint32_t,
+                                                                                std::uint32_t)> &complement) noexcept {
         switch (arg) {
             case equipments::Chainmail:
                 localHabilete.materiel = complement(localHabilete.materiel, 1);
@@ -144,7 +147,8 @@ namespace character {
                                            Characteristic &localHabilete,
                                            Characteristic &localAdresse,
                                            Characteristic &localEndurance,
-            const std::function<std::uint32_t(std::uint32_t, std::uint32_t)> &operation) noexcept {
+                                           const std::function<std::uint32_t(std::uint32_t,
+                                                                             std::uint32_t)> &operation) noexcept {
         switch (arg) {
             case weapons::Sword:
                 localHabilete.materiel = operation(localHabilete.materiel, 4);

src/gui/gui.cpp

@@ -58,10 +58,9 @@ void gui::Gui::render_gui(const Controller &controller) {
             for (std::size_t i = 0; i < GuiData::classes.size(); ++i) {
                 const bool is_selected = (data.billy.get_current_class() == static_cast<character::classe>(i));
 
-                if (is_selected) {
+                if (is_selected)
                     ImGui::SetItemDefaultFocus();
             }
-            }
             ImGui::EndCombo();
         }