diff --git a/gz-waves/include/gz/waves/MeshTools.hh b/gz-waves/include/gz/waves/MeshTools.hh index 4a5ab42..b124a8c 100644 --- a/gz-waves/include/gz/waves/MeshTools.hh +++ b/gz-waves/include/gz/waves/MeshTools.hh @@ -25,7 +25,6 @@ #include "gz/waves/CGALTypes.hh" -#include #include namespace gz diff --git a/gz-waves/include/gz/waves/OceanTile.hh b/gz-waves/include/gz/waves/OceanTile.hh index d9dead5..b61ab21 100644 --- a/gz-waves/include/gz/waves/OceanTile.hh +++ b/gz-waves/include/gz/waves/OceanTile.hh @@ -20,8 +20,6 @@ #include #include -#include -#include #include #include "gz/waves/CGALTypes.hh" diff --git a/gz-waves/include/gz/waves/Wavefield.hh b/gz-waves/include/gz/waves/Wavefield.hh index 117220d..05df8d6 100644 --- a/gz-waves/include/gz/waves/Wavefield.hh +++ b/gz-waves/include/gz/waves/Wavefield.hh @@ -24,8 +24,6 @@ #include #include -#include - namespace gz { namespace waves diff --git a/gz-waves/src/CGAL_TEST.cc b/gz-waves/src/CGAL_TEST.cc index ac728ed..d14e375 100644 --- a/gz-waves/src/CGAL_TEST.cc +++ b/gz-waves/src/CGAL_TEST.cc @@ -50,7 +50,7 @@ #include "gz/waves/Wavefield.hh" #include "gz/waves/WaveParameters.hh" -#include +#include #include #include #include diff --git a/gz-waves/src/Grid.cc b/gz-waves/src/Grid.cc index 81f23c6..9670d2e 100644 --- a/gz-waves/src/Grid.cc +++ b/gz-waves/src/Grid.cc @@ -25,7 +25,7 @@ #include #include -#include +#include #include "gz/waves/CGALTypes.hh" #include "gz/waves/Geometry.hh" diff --git a/gz-waves/src/LinearRandomFFTWaveSimulation.cc b/gz-waves/src/LinearRandomFFTWaveSimulation.cc index f969513..84ff3c4 100644 --- a/gz-waves/src/LinearRandomFFTWaveSimulation.cc +++ b/gz-waves/src/LinearRandomFFTWaveSimulation.cc @@ -25,7 +25,7 @@ #include #include -#include +#include #include "gz/waves/Algorithm.hh" #include "gz/waves/Types.hh" diff --git a/gz-waves/src/LinearRandomFFTWaveSimulationRef.cc b/gz-waves/src/LinearRandomFFTWaveSimulationRef.cc index e9e80ee..82a0f95 100644 --- a/gz-waves/src/LinearRandomFFTWaveSimulationRef.cc +++ b/gz-waves/src/LinearRandomFFTWaveSimulationRef.cc @@ -37,7 +37,7 @@ #include #include -#include +#include #include "gz/waves/Types.hh" #include "LinearRandomFFTWaveSimulationRefImpl.hh" diff --git a/gz-waves/src/LinearRandomWaveSimulation.cc b/gz-waves/src/LinearRandomWaveSimulation.cc index a47828b..7ccb4a1 100644 --- a/gz-waves/src/LinearRandomWaveSimulation.cc +++ b/gz-waves/src/LinearRandomWaveSimulation.cc @@ -21,7 +21,7 @@ #include #include -#include +#include #include "gz/waves/WaveSpectrum.hh" @@ -491,7 +491,7 @@ void LinearRandomWaveSimulation::SetWindVelocity(double ux, double uy) } ////////////////////////////////////////////////// -void LinearRandomWaveSimulation::SetSteepness(double value) +void LinearRandomWaveSimulation::SetSteepness(double /*value*/) // NOLINT { /// \todo(srmainwaring) IMPLEMENT } diff --git a/gz-waves/src/LinearRegularWaveSimulation.cc b/gz-waves/src/LinearRegularWaveSimulation.cc index 5319bff..aac202f 100644 --- a/gz-waves/src/LinearRegularWaveSimulation.cc +++ b/gz-waves/src/LinearRegularWaveSimulation.cc @@ -578,7 +578,7 @@ void LinearRegularWaveSimulation::SetWindVelocity(double /*ux*/, double /*uy*/) } ////////////////////////////////////////////////// -void LinearRegularWaveSimulation::SetSteepness(double value) +void LinearRegularWaveSimulation::SetSteepness(double /*value*/) // NOLINT { /// \todo(srmainwaring) IMPLEMENT } diff --git a/gz-waves/src/MeshTools.cc b/gz-waves/src/MeshTools.cc index 8dd080c..d61786a 100644 --- a/gz-waves/src/MeshTools.cc +++ b/gz-waves/src/MeshTools.cc @@ -20,8 +20,7 @@ #include #include -#include -#include +#include #include "gz/waves/CGALTypes.hh" diff --git a/gz-waves/src/MeshTools_TEST.cc b/gz-waves/src/MeshTools_TEST.cc index 4938d16..3d7a499 100644 --- a/gz-waves/src/MeshTools_TEST.cc +++ b/gz-waves/src/MeshTools_TEST.cc @@ -18,7 +18,7 @@ #include #include -#include +#include #include #include #include diff --git a/gz-waves/src/OceanTile.cc b/gz-waves/src/OceanTile.cc index 930c4c0..b117c76 100644 --- a/gz-waves/src/OceanTile.cc +++ b/gz-waves/src/OceanTile.cc @@ -22,7 +22,7 @@ #include #include -#include +#include #include #include diff --git a/gz-waves/src/Physics.cc b/gz-waves/src/Physics.cc index f5290bf..8b9aa5d 100644 --- a/gz-waves/src/Physics.cc +++ b/gz-waves/src/Physics.cc @@ -26,9 +26,9 @@ #include #include +#include #include #include -#include #include diff --git a/gz-waves/src/Physics_TEST.cc b/gz-waves/src/Physics_TEST.cc index 7ab3d99..241817b 100644 --- a/gz-waves/src/Physics_TEST.cc +++ b/gz-waves/src/Physics_TEST.cc @@ -20,7 +20,7 @@ #include #include -#include +#include #include #include #include diff --git a/gz-waves/src/TrochoidIrregularWaveSimulation.cc b/gz-waves/src/TrochoidIrregularWaveSimulation.cc index a8e37db..b25da02 100644 --- a/gz-waves/src/TrochoidIrregularWaveSimulation.cc +++ b/gz-waves/src/TrochoidIrregularWaveSimulation.cc @@ -184,8 +184,8 @@ void TrochoidIrregularWaveSimulation::Impl::Elevation( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::Pressure( - double x, double y, double z, - double &pressure) + double /*x*/, double /*y*/, double /*z*/, + double &/*pressure*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "Pressure: Not implemented!\n"; @@ -193,10 +193,10 @@ void TrochoidIrregularWaveSimulation::Impl::Pressure( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::Pressure( - const Eigen::Ref& x, - const Eigen::Ref& y, - const Eigen::Ref& z, - Eigen::Ref pressure) + const Eigen::Ref& /*x*/, + const Eigen::Ref& /*y*/, + const Eigen::Ref& /*z*/, + Eigen::Ref /*pressure*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "Pressure: Not implemented!\n"; @@ -374,8 +374,8 @@ void TrochoidIrregularWaveSimulation::Impl::DisplacementDerivAt( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::ElevationAt( - Index ix, Index iy, - double &h) + Index /*ix*/, Index /*iy*/, + double &/*h*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "ElevationAt: Not implemented!\n"; @@ -383,8 +383,8 @@ void TrochoidIrregularWaveSimulation::Impl::ElevationAt( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::DisplacementAt( - Index ix, Index iy, - double& sx, double& sy) + Index /*ix*/, Index /*iy*/, + double& /*sx*/, double& /*sy*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "DisplacementAt: Not implemented!\n"; @@ -392,8 +392,8 @@ void TrochoidIrregularWaveSimulation::Impl::DisplacementAt( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::PressureAt( - Index ix, Index iy, Index iz, - double &pressure) + Index /*ix*/, Index /*iy*/, Index /*iz*/, + double &/*pressure*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "PressureAt: Not implemented!\n"; @@ -401,8 +401,8 @@ void TrochoidIrregularWaveSimulation::Impl::PressureAt( ////////////////////////////////////////////////// void TrochoidIrregularWaveSimulation::Impl::PressureAt( - Index iz, - Eigen::Ref pressure) + Index /*iz*/, + Eigen::Ref /*pressure*/) { /// \todo(srmainwaring) IMPLEMENT gzerr << "PressureAt: Not implemented!\n"; diff --git a/gz-waves/src/Utilities.cc b/gz-waves/src/Utilities.cc index 68b6a41..c032d46 100644 --- a/gz-waves/src/Utilities.cc +++ b/gz-waves/src/Utilities.cc @@ -19,7 +19,7 @@ #include #include -#include +#include #include #include diff --git a/gz-waves/src/WaveParameters.cc b/gz-waves/src/WaveParameters.cc index e22ee90..2e84990 100644 --- a/gz-waves/src/WaveParameters.cc +++ b/gz-waves/src/WaveParameters.cc @@ -20,7 +20,7 @@ #include #include -#include +#include #include #include #include diff --git a/gz-waves/src/Wavefield.cc b/gz-waves/src/Wavefield.cc index cd1d5d7..72a30f9 100644 --- a/gz-waves/src/Wavefield.cc +++ b/gz-waves/src/Wavefield.cc @@ -21,7 +21,8 @@ #include #include -#include +#include +#include #include "gz/waves/CGALTypes.hh" #include "gz/waves/OceanTile.hh" @@ -30,7 +31,6 @@ #include "gz/waves/Utilities.hh" #include "gz/waves/WaveParameters.hh" - namespace gz { namespace waves diff --git a/gz-waves/src/WavefieldSampler.cc b/gz-waves/src/WavefieldSampler.cc index 2a830bf..6335b4e 100644 --- a/gz-waves/src/WavefieldSampler.cc +++ b/gz-waves/src/WavefieldSampler.cc @@ -22,7 +22,7 @@ #include #include -#include +#include #include #include #include diff --git a/gz-waves/src/systems/dynamic/CMakeLists.txt b/gz-waves/src/systems/dynamic/CMakeLists.txt index fb4fedd..d0074f0 100644 --- a/gz-waves/src/systems/dynamic/CMakeLists.txt +++ b/gz-waves/src/systems/dynamic/CMakeLists.txt @@ -13,4 +13,6 @@ gz_add_system(dynamic-geometry gz-rendering${GZ_RENDERING_VER}-ogre2 gz-sim${GZ_SIM_VER}::gz-sim${GZ_SIM_VER} GzOGRE2::GzOGRE2 + PRIVATE_COMPILE_DEFS + OGRE_IGNORE_UNKNOWN_DEBUG ) diff --git a/gz-waves/src/systems/waves/CMakeLists.txt b/gz-waves/src/systems/waves/CMakeLists.txt index 6d8941c..5735c67 100644 --- a/gz-waves/src/systems/waves/CMakeLists.txt +++ b/gz-waves/src/systems/waves/CMakeLists.txt @@ -65,6 +65,13 @@ target_link_libraries(${rendering_ogre2_target} gz-plugin${GZ_PLUGIN_VER}::register ) +# Suppress an Ogre warning about adding _DEBUG to precompiler definitions. +# TODO: set conditional on the cmake build type. +target_compile_definitions(${rendering_ogre2_target} + PUBLIC + OGRE_IGNORE_UNKNOWN_DEBUG +) + ################################################# gz_add_system(waves-visual SOURCES diff --git a/gz-waves/src/systems/waves/Ogre2OceanGeometry.cc b/gz-waves/src/systems/waves/Ogre2OceanGeometry.cc index 28b6965..010d049 100644 --- a/gz-waves/src/systems/waves/Ogre2OceanGeometry.cc +++ b/gz-waves/src/systems/waves/Ogre2OceanGeometry.cc @@ -15,7 +15,7 @@ #include "Ogre2OceanGeometry.hh" -#include +#include #include #include diff --git a/gz-waves/src/systems/waves/Ogre2OceanVisual.cc b/gz-waves/src/systems/waves/Ogre2OceanVisual.cc index 44d4c15..0ab4e28 100644 --- a/gz-waves/src/systems/waves/Ogre2OceanVisual.cc +++ b/gz-waves/src/systems/waves/Ogre2OceanVisual.cc @@ -15,7 +15,7 @@ #include "Ogre2OceanVisual.hh" -#include +#include #include #include #include diff --git a/gz-waves/src/systems/waves/RenderEngineExtensionManager.cc b/gz-waves/src/systems/waves/RenderEngineExtensionManager.cc index 4e75539..98d2693 100644 --- a/gz-waves/src/systems/waves/RenderEngineExtensionManager.cc +++ b/gz-waves/src/systems/waves/RenderEngineExtensionManager.cc @@ -39,6 +39,7 @@ #include #include #include +#include #include "RenderEngineExtensionManager.hh" #include "RenderEngineExtension.hh" @@ -460,7 +461,7 @@ bool RenderEngineExtensionManagerPrivate::LoadExtensionPlugin( // Add default install folder. systemPaths.AddPluginPaths(std::string(GZ_RENDERING_PLUGIN_PATH)); - systemPaths.AddPluginPaths({GZ_RENDERING_ENGINE_INSTALL_DIR}); + systemPaths.AddPluginPaths(gz::rendering::getEngineInstallDir()); // Add any preset plugin paths. for (const auto &path : this->pluginPaths) diff --git a/gz-waves/test/plots/CMakeLists.txt b/gz-waves/test/plots/CMakeLists.txt index 3d54d0b..058a957 100644 --- a/gz-waves/test/plots/CMakeLists.txt +++ b/gz-waves/test/plots/CMakeLists.txt @@ -17,7 +17,9 @@ target_link_libraries(PLOT_GnuPlotExample target_include_directories(PLOT_GnuPlotExample PUBLIC ${gnuplot-iostream_INCLUDE_DIRS} ) - +target_compile_options(PLOT_GnuPlotExample + PRIVATE "-Wno-unused-result" +) add_executable(PLOT_LinearRandomFFTWaves PLOT_LinearRandomFFTWaves.cc) target_link_libraries(PLOT_LinearRandomFFTWaves @@ -27,7 +29,9 @@ target_link_libraries(PLOT_LinearRandomFFTWaves target_include_directories(PLOT_LinearRandomFFTWaves PUBLIC ${gnuplot-iostream_INCLUDE_DIRS} ) - +target_compile_options(PLOT_LinearRandomFFTWaves + PRIVATE "-Wno-unused-result" +) add_executable(PLOT_LinearRandomWaves PLOT_LinearRandomWaves.cc) target_link_libraries(PLOT_LinearRandomWaves @@ -37,7 +41,9 @@ target_link_libraries(PLOT_LinearRandomWaves target_include_directories(PLOT_LinearRandomWaves PUBLIC ${gnuplot-iostream_INCLUDE_DIRS} ) - +target_compile_options(PLOT_LinearRandomWaves + PRIVATE "-Wno-unused-result" +) add_executable(PLOT_LinearRegularWaves PLOT_LinearRegularWaves.cc) target_link_libraries(PLOT_LinearRegularWaves @@ -47,7 +53,9 @@ target_link_libraries(PLOT_LinearRegularWaves target_include_directories(PLOT_LinearRegularWaves PUBLIC ${gnuplot-iostream_INCLUDE_DIRS} ) - +target_compile_options(PLOT_LinearRegularWaves + PRIVATE "-Wno-unused-result" +) add_executable(PLOT_WaveSpectrum PLOT_WaveSpectrum.cc) target_link_libraries(PLOT_WaveSpectrum @@ -57,6 +65,9 @@ target_link_libraries(PLOT_WaveSpectrum target_include_directories(PLOT_WaveSpectrum PUBLIC ${gnuplot-iostream_INCLUDE_DIRS} ) +target_compile_options(PLOT_WaveSpectrum + PRIVATE "-Wno-unused-result" +) #============================================================================ # Install targets