diff options
Diffstat (limited to 'loader')
-rw-r--r-- | loader/ground-traits.cpp | 7 | ||||
-rw-r--r-- | loader/vobj.cpp | 1 | ||||
-rw-r--r-- | loader/wall-atlas.cpp | 1 | ||||
-rw-r--r-- | loader/wall-cell.cpp | 3 | ||||
-rw-r--r-- | loader/wall-cell.hpp | 2 | ||||
-rw-r--r-- | loader/wall-traits.cpp | 2 | ||||
-rw-r--r-- | loader/wall-traits.hpp | 1 |
7 files changed, 8 insertions, 9 deletions
diff --git a/loader/ground-traits.cpp b/loader/ground-traits.cpp index 0ec42f0e..e9a82b24 100644 --- a/loader/ground-traits.cpp +++ b/loader/ground-traits.cpp @@ -31,17 +31,18 @@ void ground_traits::ensure_atlases_loaded(Storage& st) fm_assert(st.name_map.empty()); st.cell_array = ground_cell::load_atlases_from_json().vec; + st.name_map.reserve(st.cell_array.size()); fm_assert(!st.cell_array.empty()); fm_assert(st.name_map.empty()); constexpr bool add_invalid = true; if constexpr(add_invalid) + { for (auto& x : st.cell_array) fm_soft_assert(x.name != loader.INVALID); - - if constexpr(true) // NOLINT(*-simplify-boolean-expr) - st.cell_array.push_back(make_invalid_atlas(st)); // add invalid atlas + st.cell_array.push_back(make_invalid_atlas(st)); + } st.name_map.reserve(st.cell_array.size()); diff --git a/loader/vobj.cpp b/loader/vobj.cpp index d437df69..010cc642 100644 --- a/loader/vobj.cpp +++ b/loader/vobj.cpp @@ -36,6 +36,7 @@ void adl_serializer<vobj>::to_json(json& j, const vobj& val) void adl_serializer<vobj>::from_json(const json& j, vobj& val) { + val = {}; val.name = j["name"]; if (j.contains("description")) val.descr = j["description"]; diff --git a/loader/wall-atlas.cpp b/loader/wall-atlas.cpp index b07c73f2..67e4fdf8 100644 --- a/loader/wall-atlas.cpp +++ b/loader/wall-atlas.cpp @@ -1,5 +1,4 @@ #include "loader/impl.hpp" -#include "compat/vector-wrapper.hpp" #include "src/wall-atlas.hpp" #include "loader/wall-cell.hpp" #include "loader/wall-traits.hpp" diff --git a/loader/wall-cell.cpp b/loader/wall-cell.cpp index 6003ad77..92034659 100644 --- a/loader/wall-cell.cpp +++ b/loader/wall-cell.cpp @@ -11,8 +11,7 @@ using nlohmann::json; [[maybe_unused]] static void from_json(const json& j, wall_cell& val) { - val = {}; - val.name = j["name"]; + val = { .atlas = {}, .name = j["name"], }; fm_soft_assert(loader.check_atlas_name(val.name)); } diff --git a/loader/wall-cell.hpp b/loader/wall-cell.hpp index 507a09c2..e8cd15d5 100644 --- a/loader/wall-cell.hpp +++ b/loader/wall-cell.hpp @@ -9,8 +9,8 @@ class wall_atlas; struct wall_cell { - String name; std::shared_ptr<wall_atlas> atlas; + String name; static vector_wrapper<const wall_cell> load_atlases_from_json(); }; diff --git a/loader/wall-traits.cpp b/loader/wall-traits.cpp index 9fbe4286..8f1c2e23 100644 --- a/loader/wall-traits.cpp +++ b/loader/wall-traits.cpp @@ -59,7 +59,7 @@ auto wall_traits::make_invalid_atlas(Storage& st) -> const Cell& {{ {.val = 0}, {}, }}, {1u}, }, name, loader.make_error_texture(frame_size)); - st.invalid_atlas = Pointer<wall_cell>{InPlaceInit, wall_cell{ .name = name, .atlas = std::move(a) } }; + st.invalid_atlas = Pointer<wall_cell>{InPlaceInit, wall_cell{ .atlas = std::move(a), .name = name, } }; return *st.invalid_atlas; } diff --git a/loader/wall-traits.hpp b/loader/wall-traits.hpp index 50576e2c..72492d98 100644 --- a/loader/wall-traits.hpp +++ b/loader/wall-traits.hpp @@ -22,7 +22,6 @@ template<> struct atlas_loader_traits<wall_atlas> static const Cell& make_invalid_atlas(Storage& st); static std::shared_ptr<Atlas> make_atlas(StringView name, const Cell& c); - static const Cell& make_placeholder_cell() noexcept; }; } // namespace floormat::loader_detail |