Merge pull request #24 from BlackMATov/dev

Dev

CMakeLists.txt

@@ -39,5 +39,13 @@ set_target_properties(${PROJECT_NAME} PROPERTIES
     CXX_STANDARD_REQUIRED YES
     CXX_EXTENSIONS NO)
 
+target_compile_options(${PROJECT_NAME}
+    PRIVATE
+    $<$<CXX_COMPILER_ID:MSVC>:
+        /W4>
+    PRIVATE
+    $<$<OR:$<CXX_COMPILER_ID:GNU>,$<CXX_COMPILER_ID:Clang>,$<CXX_COMPILER_ID:AppleClang>>:
+        -Wall -Wextra -Wpedantic>)
+
 enable_testing()
 add_test(${PROJECT_NAME} ${PROJECT_NAME})

ecs.hpp

@@ -731,7 +731,7 @@ namespace ecs_hpp
             : owner_(owner) {}
 
             template < typename... Args >
-            T& assign(entity_id id, Args&&... args) {
+            T& assign(entity_id id, Args&&...) {
                 components_.insert(id);
                 return empty_value_;
             }