From 71ad1c743e3777a2d0237a63b87befd65a641f3c Mon Sep 17 00:00:00 2001 From: BlackMATov Date: Thu, 23 May 2019 21:29:41 +0700 Subject: [PATCH] rename component_factory to factory --- headers/enduro2d/high/_all.hpp | 4 +- headers/enduro2d/high/_high.hpp | 3 +- headers/enduro2d/high/components/actor.hpp | 8 +- headers/enduro2d/high/components/camera.hpp | 8 +- .../high/components/flipbook_player.hpp | 8 +- .../high/components/flipbook_source.hpp | 8 +- .../high/components/model_renderer.hpp | 8 +- headers/enduro2d/high/components/renderer.hpp | 8 +- headers/enduro2d/high/components/scene.hpp | 8 +- .../high/components/sprite_renderer.hpp | 8 +- .../high/{component.hpp => factory.hpp} | 75 +++++++++---------- .../high/{component.inl => factory.inl} | 37 +++++---- sources/enduro2d/high/assets/prefab_asset.cpp | 12 +-- sources/enduro2d/high/components/actor.cpp | 10 +-- sources/enduro2d/high/components/camera.cpp | 10 +-- .../high/components/flipbook_player.cpp | 10 +-- .../high/components/flipbook_source.cpp | 10 +-- .../high/components/model_renderer.cpp | 10 +-- sources/enduro2d/high/components/renderer.cpp | 10 +-- sources/enduro2d/high/components/scene.cpp | 10 +-- .../high/components/sprite_renderer.cpp | 10 +-- .../high/{component.cpp => factory.cpp} | 14 ++-- sources/enduro2d/high/starter.cpp | 6 +- 23 files changed, 144 insertions(+), 151 deletions(-) rename headers/enduro2d/high/{component.hpp => factory.hpp} (57%) rename headers/enduro2d/high/{component.inl => factory.inl} (65%) rename sources/enduro2d/high/{component.cpp => factory.cpp} (75%) diff --git a/headers/enduro2d/high/_all.hpp b/headers/enduro2d/high/_all.hpp index 7937f82b..3eb67d45 100644 --- a/headers/enduro2d/high/_all.hpp +++ b/headers/enduro2d/high/_all.hpp @@ -40,8 +40,8 @@ #include "asset.hpp" #include "asset.inl" #include "atlas.hpp" -#include "component.hpp" -#include "component.inl" +#include "factory.hpp" +#include "factory.inl" #include "flipbook.hpp" #include "gobject.hpp" #include "library.hpp" diff --git a/headers/enduro2d/high/_high.hpp b/headers/enduro2d/high/_high.hpp index eff52ff0..8438da4b 100644 --- a/headers/enduro2d/high/_high.hpp +++ b/headers/enduro2d/high/_high.hpp @@ -52,13 +52,12 @@ namespace e2d class content_asset; class asset; + class factory; class library; class asset_cache; class asset_group; class asset_dependencies; - class component_factory; - class atlas; class flipbook; class gobject; diff --git a/headers/enduro2d/high/components/actor.hpp b/headers/enduro2d/high/components/actor.hpp index 9c278fb0..ac86107c 100644 --- a/headers/enduro2d/high/components/actor.hpp +++ b/headers/enduro2d/high/components/actor.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../node.hpp" namespace e2d @@ -27,17 +27,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( actor& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/camera.hpp b/headers/enduro2d/high/components/camera.hpp index ab36829f..a2549d8b 100644 --- a/headers/enduro2d/high/components/camera.hpp +++ b/headers/enduro2d/high/components/camera.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" namespace e2d { @@ -36,17 +36,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( camera& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/flipbook_player.hpp b/headers/enduro2d/high/components/flipbook_player.hpp index 241fdff9..a8e64882 100644 --- a/headers/enduro2d/high/components/flipbook_player.hpp +++ b/headers/enduro2d/high/components/flipbook_player.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../assets/flipbook_asset.hpp" namespace e2d @@ -53,17 +53,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( flipbook_player& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/flipbook_source.hpp b/headers/enduro2d/high/components/flipbook_source.hpp index ee0460a8..2ca0e546 100644 --- a/headers/enduro2d/high/components/flipbook_source.hpp +++ b/headers/enduro2d/high/components/flipbook_source.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../assets/flipbook_asset.hpp" namespace e2d @@ -25,17 +25,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( flipbook_source& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/model_renderer.hpp b/headers/enduro2d/high/components/model_renderer.hpp index a92dc56a..51f33729 100644 --- a/headers/enduro2d/high/components/model_renderer.hpp +++ b/headers/enduro2d/high/components/model_renderer.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../assets/model_asset.hpp" namespace e2d @@ -25,17 +25,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( model_renderer& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/renderer.hpp b/headers/enduro2d/high/components/renderer.hpp index 3c0134bd..8c7931ba 100644 --- a/headers/enduro2d/high/components/renderer.hpp +++ b/headers/enduro2d/high/components/renderer.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../assets/material_asset.hpp" namespace e2d @@ -38,17 +38,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( renderer& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/scene.hpp b/headers/enduro2d/high/components/scene.hpp index 8a3e3128..294ebbd2 100644 --- a/headers/enduro2d/high/components/scene.hpp +++ b/headers/enduro2d/high/components/scene.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" namespace e2d { @@ -23,17 +23,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( scene& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/components/sprite_renderer.hpp b/headers/enduro2d/high/components/sprite_renderer.hpp index 8e6571d6..e234ed12 100644 --- a/headers/enduro2d/high/components/sprite_renderer.hpp +++ b/headers/enduro2d/high/components/sprite_renderer.hpp @@ -8,7 +8,7 @@ #include "../_high.hpp" -#include "../component.hpp" +#include "../factory.hpp" #include "../assets/sprite_asset.hpp" namespace e2d @@ -33,17 +33,17 @@ namespace e2d }; template <> - class component_loader { + class factory_loader final : factory_loader<> { public: static const char* schema_source; bool operator()( sprite_renderer& component, - const component_loader<>::fill_context& ctx) const; + const fill_context& ctx) const; bool operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const collect_context& ctx) const; }; } diff --git a/headers/enduro2d/high/component.hpp b/headers/enduro2d/high/factory.hpp similarity index 57% rename from headers/enduro2d/high/component.hpp rename to headers/enduro2d/high/factory.hpp index fab6fa95..a634dc53 100644 --- a/headers/enduro2d/high/component.hpp +++ b/headers/enduro2d/high/factory.hpp @@ -4,8 +4,8 @@ * Copyright (C) 2018-2019, by Matvey Cherevko (blackmatov@gmail.com) ******************************************************************************/ -#ifndef E2D_INCLUDE_GUARD_DEE612552A524342A523868F34FF88BA -#define E2D_INCLUDE_GUARD_DEE612552A524342A523868F34FF88BA +#ifndef E2D_INCLUDE_GUARD_9C9B1BA27AAC42CBAA07890458076B81 +#define E2D_INCLUDE_GUARD_9C9B1BA27AAC42CBAA07890458076B81 #pragma once #include "_high.hpp" @@ -13,25 +13,25 @@ namespace e2d { // - // bad_component_factory_operation + // bad_factory_operation // - class bad_component_factory_operation final : public exception { + class bad_factory_operation final : public exception { public: const char* what() const noexcept final { - return "bad component factory operation"; + return "bad factory operation"; } }; // - // component_loader + // factory_loader // template < typename Component = void > - class component_loader; + class factory_loader; template <> - class component_loader<> { + class factory_loader<> { public: struct fill_context { const str parent_address; @@ -57,71 +57,66 @@ namespace e2d : parent_address(std::move(nparent_address)) , root(nroot) {} }; - public: }; - + // - // component_creator_base + // factory_creator // - class component_creator_base; - using component_creator_base_iptr = intrusive_ptr; + class factory_creator; + using factory_creator_iptr = intrusive_ptr; - class component_creator_base + class factory_creator : private noncopyable - , public ref_counter { + , public ref_counter { public: - component_creator_base() = default; - virtual ~component_creator_base() noexcept = default; + factory_creator() = default; + virtual ~factory_creator() noexcept = default; virtual bool validate_json( const rapidjson::Value& root) const = 0; virtual bool fill_prototype( ecs::prototype& prototype, - const component_loader<>::fill_context& ctx) const = 0; + const factory_loader<>::fill_context& ctx) const = 0; virtual bool collect_dependencies( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const = 0; + const factory_loader<>::collect_context& ctx) const = 0; }; - // - // component_creator - // - template < typename Component > - class component_creator : public component_creator_base { + class typed_factory_creator final : public factory_creator { public: - component_creator(); - ~component_creator() noexcept override = default; + typed_factory_creator(); + ~typed_factory_creator() noexcept final = default; bool validate_json( - const rapidjson::Value& root) const override; + const rapidjson::Value& root) const final; bool fill_prototype( ecs::prototype& prototype, - const component_loader<>::fill_context& ctx) const override; + const factory_loader<>::fill_context& ctx) const final; bool collect_dependencies( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const override; + const factory_loader<>::collect_context& ctx) const final; private: - component_loader loader_; + factory_loader loader_; std::unique_ptr schema_; }; // - // component_factory + // factory // - class component_factory : public module { + class factory : public module { public: - component_factory() = default; - ~component_factory() noexcept final = default; + factory() = default; + ~factory() noexcept final = default; template < typename Component > - component_factory& register_component(str_hash type); + factory& register_component(str_hash type); bool validate_json( str_hash type, @@ -130,19 +125,19 @@ namespace e2d bool fill_prototype( str_hash type, ecs::prototype& prototype, - const component_loader<>::fill_context& ctx) const; + const factory_loader<>::fill_context& ctx) const; bool collect_dependencies( str_hash type, asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const; + const factory_loader<>::collect_context& ctx) const; private: - component_creator_base_iptr find_creator(str_hash type) const; + factory_creator_iptr find_creator(str_hash type) const; private: mutable std::mutex mutex_; - hash_map creators_; + hash_map creators_; }; } -#include "component.inl" +#include "factory.inl" #endif diff --git a/headers/enduro2d/high/component.inl b/headers/enduro2d/high/factory.inl similarity index 65% rename from headers/enduro2d/high/component.inl rename to headers/enduro2d/high/factory.inl index 560f0776..6decca06 100644 --- a/headers/enduro2d/high/component.inl +++ b/headers/enduro2d/high/factory.inl @@ -4,32 +4,32 @@ * Copyright (C) 2018-2019, by Matvey Cherevko (blackmatov@gmail.com) ******************************************************************************/ -#ifndef E2D_INCLUDE_GUARD_85A0D62C0D794C4C9DDFB1BAED3C0A18 -#define E2D_INCLUDE_GUARD_85A0D62C0D794C4C9DDFB1BAED3C0A18 +#ifndef E2D_INCLUDE_GUARD_6CDF0C8E49C9459EB7762C8AF7F7ED85 +#define E2D_INCLUDE_GUARD_6CDF0C8E49C9459EB7762C8AF7F7ED85 #pragma once #include "_high.hpp" -#include "component.hpp" +#include "factory.hpp" namespace e2d { // - // component_creator + // typed_factory_creator // template < typename Component > - component_creator::component_creator() { + typed_factory_creator::typed_factory_creator() { rapidjson::Document doc; if ( doc.Parse(loader_.schema_source).HasParseError() ) { - the().error("COMPONENT: Failed to parse component loader schema"); - throw bad_component_factory_operation(); + the().error("FACTORY: Failed to parse factory loader schema"); + throw bad_factory_operation(); } json_utils::add_common_schema_definitions(doc); schema_ = std::make_unique(doc); } template < typename Component > - bool component_creator::validate_json( + bool typed_factory_creator::validate_json( const rapidjson::Value& root) const { rapidjson::SchemaValidator validator(*schema_); @@ -38,10 +38,10 @@ namespace e2d } rapidjson::StringBuffer sb; if ( !validator.GetInvalidDocumentPointer().StringifyUriFragment(sb) ) { - the().error("COMPONENT: Failed to validate component json"); + the().error("FACTORY: Failed to validate component json"); return false; } - the().error("COMPONENT: Failed to validate component json:\n" + the().error("FACTORY: Failed to validate component json:\n" "--> Invalid schema keyword: %0\n" "--> Invalid document pointer: %1", validator.GetInvalidSchemaKeyword(), @@ -50,9 +50,9 @@ namespace e2d } template < typename Component > - bool component_creator::fill_prototype( + bool typed_factory_creator::fill_prototype( ecs::prototype& prototype, - const component_loader<>::fill_context& ctx) const + const factory_loader<>::fill_context& ctx) const { Component component; prototype.apply_to_component(component); @@ -66,27 +66,26 @@ namespace e2d } template < typename Component > - bool component_creator::collect_dependencies( + bool typed_factory_creator::collect_dependencies( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const factory_loader<>::collect_context& ctx) const { return loader_(dependencies, ctx); } // - // component_factory + // factory // template < typename Component > - component_factory& component_factory::register_component(str_hash type) { + factory& factory::register_component(str_hash type) { std::lock_guard guard(mutex_); if ( creators_.count(type) > 0 ) { - throw bad_component_factory_operation(); + throw bad_factory_operation(); } - component_creator_base_iptr creator(new component_creator()); + factory_creator_iptr creator(new typed_factory_creator()); creators_.emplace(type, std::move(creator)); return *this; } } - #endif diff --git a/sources/enduro2d/high/assets/prefab_asset.cpp b/sources/enduro2d/high/assets/prefab_asset.cpp index f6de16d3..f4fdebb7 100644 --- a/sources/enduro2d/high/assets/prefab_asset.cpp +++ b/sources/enduro2d/high/assets/prefab_asset.cpp @@ -6,7 +6,7 @@ #include -#include +#include #include namespace @@ -83,7 +83,7 @@ namespace ++component_root ) { { - bool success = the().validate_json( + bool success = the().validate_json( component_root->name.GetString(), component_root->value); if ( !success ) { @@ -91,10 +91,10 @@ namespace } } { - component_loader<>::collect_context ctx( + factory_loader<>::collect_context ctx( parent_address, component_root->value); - bool success = the().collect_dependencies( + bool success = the().collect_dependencies( component_root->name.GetString(), dependencies, ctx); @@ -145,11 +145,11 @@ namespace component_root != components_root.MemberEnd(); ++component_root ) { - component_loader<>::fill_context ctx( + factory_loader<>::fill_context ctx( parent_address, component_root->value, dependencies); - the().fill_prototype( + the().fill_prototype( component_root->name.GetString(), content.prototype(), ctx); diff --git a/sources/enduro2d/high/components/actor.cpp b/sources/enduro2d/high/components/actor.cpp index 8dc5ff00..7f1b5125 100644 --- a/sources/enduro2d/high/components/actor.cpp +++ b/sources/enduro2d/high/components/actor.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -19,9 +19,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( actor& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( !component.node() ) { component.node(node::create()); @@ -57,9 +57,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { E2D_UNUSED(dependencies, ctx); return true; diff --git a/sources/enduro2d/high/components/camera.cpp b/sources/enduro2d/high/components/camera.cpp index 58f0c423..4214ef1f 100644 --- a/sources/enduro2d/high/components/camera.cpp +++ b/sources/enduro2d/high/components/camera.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -20,9 +20,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( camera& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("depth") ) { auto depth = component.depth(); @@ -67,9 +67,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { E2D_UNUSED(dependencies, ctx); return true; diff --git a/sources/enduro2d/high/components/flipbook_player.cpp b/sources/enduro2d/high/components/flipbook_player.cpp index 337a77d5..9991d384 100644 --- a/sources/enduro2d/high/components/flipbook_player.cpp +++ b/sources/enduro2d/high/components/flipbook_player.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -22,9 +22,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( flipbook_player& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("time") ) { auto time = component.time(); @@ -83,9 +83,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { E2D_UNUSED(dependencies, ctx); return true; diff --git a/sources/enduro2d/high/components/flipbook_source.cpp b/sources/enduro2d/high/components/flipbook_source.cpp index 08a339f1..13c4436d 100644 --- a/sources/enduro2d/high/components/flipbook_source.cpp +++ b/sources/enduro2d/high/components/flipbook_source.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -17,9 +17,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( flipbook_source& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("flipbook") ) { auto flipbook = ctx.dependencies.find_asset( @@ -38,9 +38,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { if ( ctx.root.HasMember("flipbook") ) { dependencies.add_dependency( diff --git a/sources/enduro2d/high/components/model_renderer.cpp b/sources/enduro2d/high/components/model_renderer.cpp index 1a7a11f2..c792e556 100644 --- a/sources/enduro2d/high/components/model_renderer.cpp +++ b/sources/enduro2d/high/components/model_renderer.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -17,9 +17,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( model_renderer& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("model") ) { auto model = ctx.dependencies.find_asset( @@ -38,9 +38,9 @@ namespace e2d return true;; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { if ( ctx.root.HasMember("model") ) { dependencies.add_dependency( diff --git a/sources/enduro2d/high/components/renderer.cpp b/sources/enduro2d/high/components/renderer.cpp index f1ac9d07..47420c6c 100644 --- a/sources/enduro2d/high/components/renderer.cpp +++ b/sources/enduro2d/high/components/renderer.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -21,9 +21,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( renderer& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("enabled") ) { auto enabled = component.enabled(); @@ -60,9 +60,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { if ( ctx.root.HasMember("materials") ) { const rapidjson::Value& materials_root = ctx.root["materials"]; diff --git a/sources/enduro2d/high/components/scene.cpp b/sources/enduro2d/high/components/scene.cpp index 50847f6f..1ade833f 100644 --- a/sources/enduro2d/high/components/scene.cpp +++ b/sources/enduro2d/high/components/scene.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -17,9 +17,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( scene& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("depth") ) { auto depth = component.depth(); @@ -33,9 +33,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { E2D_UNUSED(dependencies, ctx); return true; diff --git a/sources/enduro2d/high/components/sprite_renderer.cpp b/sources/enduro2d/high/components/sprite_renderer.cpp index cdbc7ef9..da620bec 100644 --- a/sources/enduro2d/high/components/sprite_renderer.cpp +++ b/sources/enduro2d/high/components/sprite_renderer.cpp @@ -8,7 +8,7 @@ namespace e2d { - const char* component_loader::schema_source = R"json({ + const char* factory_loader::schema_source = R"json({ "type" : "object", "required" : [], "additionalProperties" : false, @@ -19,9 +19,9 @@ namespace e2d } })json"; - bool component_loader::operator()( + bool factory_loader::operator()( sprite_renderer& component, - const component_loader<>::fill_context& ctx) const + const fill_context& ctx) const { if ( ctx.root.HasMember("tint") ) { auto tint = component.tint(); @@ -58,9 +58,9 @@ namespace e2d return true; } - bool component_loader::operator()( + bool factory_loader::operator()( asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const collect_context& ctx) const { if ( ctx.root.HasMember("sprite") ) { dependencies.add_dependency( diff --git a/sources/enduro2d/high/component.cpp b/sources/enduro2d/high/factory.cpp similarity index 75% rename from sources/enduro2d/high/component.cpp rename to sources/enduro2d/high/factory.cpp index da4d4e2d..12425921 100644 --- a/sources/enduro2d/high/component.cpp +++ b/sources/enduro2d/high/factory.cpp @@ -4,11 +4,11 @@ * Copyright (C) 2018-2019, by Matvey Cherevko (blackmatov@gmail.com) ******************************************************************************/ -#include +#include namespace e2d { - bool component_factory::validate_json( + bool factory::validate_json( str_hash type, const rapidjson::Value& root) const { @@ -18,10 +18,10 @@ namespace e2d : false; } - bool component_factory::fill_prototype( + bool factory::fill_prototype( str_hash type, ecs::prototype& prototype, - const component_loader<>::fill_context& ctx) const + const factory_loader<>::fill_context& ctx) const { auto creator = find_creator(type); return creator @@ -29,10 +29,10 @@ namespace e2d : false; } - bool component_factory::collect_dependencies( + bool factory::collect_dependencies( str_hash type, asset_dependencies& dependencies, - const component_loader<>::collect_context& ctx) const + const factory_loader<>::collect_context& ctx) const { auto creator = find_creator(type); return creator @@ -40,7 +40,7 @@ namespace e2d : false; } - component_creator_base_iptr component_factory::find_creator(str_hash type) const { + factory_creator_iptr factory::find_creator(str_hash type) const { std::lock_guard guard(mutex_); const auto iter = creators_.find(type); return iter != creators_.end() diff --git a/sources/enduro2d/high/starter.cpp b/sources/enduro2d/high/starter.cpp index 59b068fe..a923837f 100644 --- a/sources/enduro2d/high/starter.cpp +++ b/sources/enduro2d/high/starter.cpp @@ -7,8 +7,8 @@ #include #include +#include #include -#include #include #include @@ -125,7 +125,7 @@ namespace e2d starter::starter(int argc, char *argv[], const parameters& params) { safe_module_initialize(argc, argv, params.engine_params()); - safe_module_initialize() + safe_module_initialize() .register_component("actor") .register_component("camera") .register_component("flipbook_player") @@ -141,7 +141,7 @@ namespace e2d starter::~starter() noexcept { modules::shutdown(); modules::shutdown(); - modules::shutdown(); + modules::shutdown(); modules::shutdown(); }