runtime: controlport: Use headers with rpcserver class definitions.

Instead of just declaring the rpcserver_base and rpcserver_thrift classes, just include the headers that actually define them. With only declarations, MSVC fails to build rpcserver_selector.cc because it cannot verify that rpcserver_thrift inherits from rpcserver_base. Signed-off-by: Ryan Volz <ryan.volz@gmail.com>
ryanvolz · Sep 19, 2023 · 9825804 · 9825804
1 parent f490e1a
commit 9825804
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/gnuradio-runtime/include/gnuradio/rpcserver_booter_base.h b/gnuradio-runtime/include/gnuradio/rpcserver_booter_base.h
@@ -11,11 +11,10 @@
 #ifndef RPCSERVER_BOOTER_BASE
 #define RPCSERVER_BOOTER_BASE
 
+#include <gnuradio/rpcserver_base.h>
 #include <string>
 #include <vector>
 
-class rpcserver_base;
-
 class rpcserver_booter_base
 {
 public:

diff --git a/gnuradio-runtime/include/gnuradio/rpcserver_booter_thrift.h b/gnuradio-runtime/include/gnuradio/rpcserver_booter_thrift.h
@@ -13,10 +13,9 @@
 
 #include "thrift/ControlPort.h"
 #include <gnuradio/rpcserver_booter_base.h>
+#include <gnuradio/rpcserver_thrift.h>
 #include <gnuradio/thrift_server_template.h>
 
-class rpcserver_base;
-class rpcserver_thrift;
 
 class rpcserver_booter_thrift
     : public virtual rpcserver_booter_base,