fix: remove redundant version.h includes per code review

pasta · pasta · commit 7a9f56255a42 · 2025-08-06T03:23:33.000-05:00
- Remove redundant version.h from src/net.h (already included via protocol.h)
- Remove unused version.h from src/rpc/rawtransaction.cpp
- Remove unused version.h from src/test/util/setup_common.h

Addresses reviewer feedback from CodeRabbit identifying unnecessary header inflation.
diff --git a/src/net.h b/src/net.h
@@ -35,7 +35,6 @@
 #include <util/system.h>
 #include <util/threadinterrupt.h>
 #include <util/wpipe.h>
-#include <version.h>
 #include <consensus/params.h>
 
 #include <atomic>
diff --git a/src/rpc/rawtransaction.cpp b/src/rpc/rawtransaction.cpp
@@ -46,7 +46,6 @@
 #include <util/translation.h>
 #include <validation.h>
 #include <validationinterface.h>
-#include <version.h>
 #include <util/irange.h>
 
 #include <chainlock/chainlock.h>
diff --git a/src/test/util/setup_common.h b/src/test/util/setup_common.h
@@ -19,7 +19,6 @@
 #include <util/string.h>
 #include <util/time.h>
 #include <util/vector.h>
-#include <version.h>
 
 #include <functional>
 #include <stdexcept>
