llvm 21 linting fixes

Author: apple1417

src/unrealsdk/commands.cpp

@@ -81,7 +81,7 @@ namespace impl {
 #ifndef UNREALSDK_IMPORTING
 
 bool is_command_valid(std::wstring_view line, bool direct_user_input) {
-    if (direct_user_input && commands.find(NEXT_LINE) != commands.end()) {
+    if (direct_user_input && commands.contains(NEXT_LINE)) {
         return true;
     }
     auto non_space = std::ranges::find_if_not(line, &std::iswspace);

src/unrealsdk/hook_manager.cpp

@@ -137,7 +137,7 @@ std::mutex log_all_calls_stream_mutex{};
 void log_all_calls(bool should_log) {
     // Only keep this file stream open while we need it
     if (should_log) {
-        const std::lock_guard<std::mutex> lock(log_all_calls_stream_mutex);
+        const std::scoped_lock lock(log_all_calls_stream_mutex);
         log_all_calls_stream.open(
             utils::get_this_dll().parent_path()
                 / config::get_str("unrealsdk.log_all_calls_file").value_or("unrealsdk.calls.tsv"),
@@ -147,7 +147,7 @@ void log_all_calls(bool should_log) {
     should_log_all_calls = should_log;
 
     if (!should_log) {
-        const std::lock_guard<std::mutex> lock(log_all_calls_stream_mutex);
+        const std::scoped_lock lock(log_all_calls_stream_mutex);
         log_all_calls_stream.close();
     }
 }
@@ -432,7 +432,7 @@ std::shared_ptr<Node> preprocess_hook(std::wstring_view source,
             func_name = func->get_path_name();
             auto obj_name = obj->get_path_name();
 
-            const std::lock_guard<std::mutex> lock(log_all_calls_stream_mutex);
+            const std::scoped_lock lock(log_all_calls_stream_mutex);
             log_all_calls_stream << source << L'\t' << func_name << L'\t' << obj_name << L'\n';
         }
     }

src/unrealsdk/logging.cpp

@@ -67,7 +67,7 @@ std::vector<log_callback> all_log_callbacks{};
         }
 
         {
-            const std::lock_guard<std::mutex> callback_lock(callback_mutex);
+            const std::scoped_lock callback_lock(callback_mutex);
             while (!pending_messages.empty()) {
                 auto msg = std::move(pending_messages.front());
                 pending_messages.pop();
@@ -289,7 +289,7 @@ void enqueue_log_msg(uint64_t unix_time_ms,
                      size_t location_size,
                      int line) {
     {
-        const std::lock_guard<std::mutex> lock(pending_messages_mutex);
+        const std::scoped_lock lock(pending_messages_mutex);
         pending_messages.emplace(unix_time_ms, level, msg, msg_size, location, location_size, line);
     }
     pending_messages_available.notify_all();
@@ -305,13 +305,13 @@ bool set_console_level(Level level) {
 }
 
 void add_callback(log_callback callback) {
-    const std::lock_guard<std::mutex> lock(callback_mutex);
+    const std::scoped_lock lock(callback_mutex);
 
     all_log_callbacks.push_back(callback);
 }
 
 void remove_callback(log_callback callback) {
-    const std::lock_guard<std::mutex> lock(callback_mutex);
+    const std::scoped_lock lock(callback_mutex);
 
     auto [begin, end] = std::ranges::remove(all_log_callbacks, callback);
     all_log_callbacks.erase(begin, end);

src/unrealsdk/unreal/structs/fframe.cpp

@@ -9,6 +9,7 @@ namespace unrealsdk::unreal {
 
 uint8_t* FFrame::extract_current_args(WrappedStruct& args) {
     auto args_addr = reinterpret_cast<uintptr_t>(args.base.get());
+    // NOLINTNEXTLINE(misc-const-correctness) - see llvm/llvm-project#157320
     uint8_t* original_code = this->Code;
 
     for (auto prop = reinterpret_cast<UProperty*>(args.type->Children());

src/unrealsdk/unreal/structs/fscriptdelegate.cpp

@@ -64,7 +64,7 @@ void FScriptDelegate::validate_signature(const std::optional<BoundFunction>& fun
     // Since delegates store the function name, make sure we can find this function again from just
     // it's name
     {
-        UFunction* func_from_find = nullptr;
+        const UFunction* func_from_find = nullptr;
         try {
             func_from_find = func->object->Class()->find_func_and_validate(func->func->Name());
         } catch (const std::invalid_argument&) {

src/unrealsdk/unrealsdk_main.cpp

@@ -45,7 +45,7 @@ std::unordered_set<void*> unreal_allocations{};
 #endif
 
 bool init(const std::function<std::unique_ptr<game::AbstractHook>(void)>& game_getter) {
-    const std::lock_guard<std::mutex> lock(init_mutex);
+    const std::scoped_lock lock(init_mutex);
 
     if (hook_instance != nullptr) {
         return false;
@@ -75,12 +75,12 @@ bool init(const std::function<std::unique_ptr<game::AbstractHook>(void)>& game_g
 }
 
 UNREALSDK_CAPI([[nodiscard]] bool, is_initialized) {
-    const std::lock_guard<std::mutex> lock(init_mutex);
+    const std::scoped_lock lock(init_mutex);
     return hook_instance != nullptr;
 }
 
 UNREALSDK_CAPI([[nodiscard]] bool, is_console_ready) {
-    return hook_instance && hook_instance->is_console_ready();
+    return is_initialized() && hook_instance->is_console_ready();
 }
 
 UNREALSDK_CAPI([[nodiscard]] const GObjects*, gobjects) {