fix: Remove unnecessary lambda captures

Remove const/constexpr values from lambda capture lists since they
don't need explicit capture (compiler handles them implicitly).

Fixes -Wunused-lambda-capture warnings in Clang.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: mhaynie

.config/git-hooks/git-hooks.json

@@ -0,0 +1,6 @@
+{
+  "$schema": "lib/git-hooks.schema.json",
+  "hooks": {
+    "pre-commit": ["lib:block-master-commit"]
+  }
+}

test/io_pipe_test.cpp

@@ -423,7 +423,7 @@ TEST_CASE("concurrent pipe operations", "[io][pipe]")
 
         // Start writer threads
         for (int t = 0; t < num_threads; ++t) {
-            writers.emplace_back([&pipe, &completed_writes, t, messages_per_thread]() {
+            writers.emplace_back([&pipe, &completed_writes, t]() {
                 for (int i = 0; i < messages_per_thread; ++i) {
                     std::string message = "Thread" + std::to_string(t) + "Msg" + std::to_string(i) + "\n";
                     auto bytes_written = pipe->in->write_async(message.data(), message.size()).get();
@@ -435,7 +435,7 @@ TEST_CASE("concurrent pipe operations", "[io][pipe]")
 
         // Reader thread
         std::vector<std::string> received_messages;
-        std::thread reader([&pipe, &received_messages, num_threads, messages_per_thread]() {
+        std::thread reader([&pipe, &received_messages]() {
             std::string accumulated_data;
             char buffer[1024];
 

test/math_random_test.cpp

@@ -260,7 +260,7 @@ TEST_CASE("get_random thread safety", "[math][random]")
 		std::vector<std::vector<int>> thread_results(num_threads);
 
 		for (int t = 0; t < num_threads; ++t) {
-			threads.emplace_back([&thread_results, t, values_per_thread]() {
+			threads.emplace_back([&thread_results, t]() {
 				for (int i = 0; i < values_per_thread; ++i) {
 					thread_results[t].push_back(mh::get_random(1, 1000));
 				}