Fix build

This removes the forward declaration and just includes the `gz/rendering/GlobalIlluminationCiVct.hh`. Since CiVctCascadePrivate.hh is not going to be installed, and it's only used by CiVctCascadePrivate.cc, I don't think there's much gain from forward declarations. Signed-off-by: Addisu Z. Taddese <[email protected]>
gazebosim · Oct 11, 2023 · 1acf47c · 1acf47c
1 parent 39863ff
commit 1acf47c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 14 deletions.
diff --git a/src/gui/plugins/global_illumination_civct/CiVctCascadePrivate.cc b/src/gui/plugins/global_illumination_civct/CiVctCascadePrivate.cc
@@ -21,8 +21,6 @@
 
 #include "GlobalIlluminationCiVct.hh"
 
-#include "gz/rendering/GlobalIlluminationCiVct.hh"
-
 using namespace gz;
 using namespace sim;
 

diff --git a/src/gui/plugins/global_illumination_civct/CiVctCascadePrivate.hh b/src/gui/plugins/global_illumination_civct/CiVctCascadePrivate.hh
@@ -26,18 +26,8 @@
 
 #include "Tsa.hh"
 
-namespace gz
-{
-  namespace rendering
-  {
-    inline namespace GZ_SIM_VERSION_NAMESPACE
-    {
-      /// Forward declare the only ptr we need
-      class CiVctCascade;
-      typedef std::shared_ptr<CiVctCascade> CiVctCascadePtr;
-    }  // namespace GZ_SIM_GAZEBO_VERSION_NAMESPACE
-  }  // namespace rendering
-}  // namespace gz
+
+#include "gz/rendering/GlobalIlluminationCiVct.hh"
 
 namespace gz
 {