fix: addressing cppcheck warnings

Author: SuperFola

include/Ark/Utils/Logger.hpp

@@ -74,7 +74,7 @@ namespace Ark::internal
          * @param args
          */
         template <typename... Args>
-        void debug(const Logger::MessageAndLocation data, Args&&... args)
+        void debug(const Logger::MessageAndLocation& data, Args&&... args)
         {
             if (shouldDebug())
                 fmt::println(

src/arkreactor/VM/VM.cpp

@@ -370,10 +370,7 @@ namespace Ark
         }
 
         if (ctx == nullptr)
-        {
-            m_execution_contexts.push_back(std::make_unique<ExecutionContext>());
-            ctx = m_execution_contexts.back().get();
-        }
+            ctx = m_execution_contexts.emplace_back(std::make_unique<ExecutionContext>()).get();
 
         assert(!ctx->primary && "The new context shouldn't be marked as primary!");
         assert(ctx != m_execution_contexts.front().get() && "The new context isn't really new!");

src/arkreactor/VM/Value/Dict.cpp

@@ -32,9 +32,8 @@ namespace Ark::internal
     {
         std::vector<Value> keys;
         keys.reserve(m_dict.size());
+        std::ranges::copy(std::ranges::views::keys(m_dict), std::back_inserter(keys));
 
-        for (auto&& key : std::ranges::views::keys(m_dict))
-            keys.push_back(key);
         return keys;
     }