Skip to content

Commit

Permalink
Suppress a new Jammy warnings
Browse files Browse the repository at this point in the history
Signed-off-by: Michael Carroll <[email protected]>
  • Loading branch information
mjcarroll committed May 24, 2022
1 parent 5832255 commit 45caad0
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion ogre/src/OgreMaterial.cc
Original file line number Diff line number Diff line change
Expand Up @@ -383,7 +383,7 @@ void OgreMaterial::UpdateShaderParams()
void OgreMaterial::UpdateShaderParams(ConstShaderParamsPtr _params,
Ogre::GpuProgramParametersSharedPtr _ogreParams)
{
for (const auto name_param : *_params)
for (const auto &name_param : *_params)
{
if (ShaderParam::PARAM_FLOAT == name_param.second.Type())
{
Expand Down
2 changes: 1 addition & 1 deletion ogre2/src/Ogre2Material.cc
Original file line number Diff line number Diff line change
Expand Up @@ -682,7 +682,7 @@ void Ogre2Material::UpdateShaderParams()
void Ogre2Material::UpdateShaderParams(ConstShaderParamsPtr _params,
Ogre::GpuProgramParametersSharedPtr _ogreParams)
{
for (const auto name_param : *_params)
for (const auto &name_param : *_params)
{
auto *constantDef =
Ogre::GpuProgramParameters::getAutoConstantDefinition(name_param.first);
Expand Down
2 changes: 1 addition & 1 deletion src/Heightmap_TEST.cc
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@

#include <ignition/common/Console.hh>
#include <ignition/common/geospatial/ImageHeightmap.hh>
#include <ignition/utilities/ExtraTestMacros.hh>
#include <ignition/utils/ExtraTestMacros.hh>

#include "test_config.h" // NOLINT(build/include)
#include "ignition/rendering/RenderEngine.hh"
Expand Down
2 changes: 1 addition & 1 deletion src/ShaderParams_TEST.cc
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ TEST(ShaderParams, ConstRangeForLoopDoesNotDirty)
params["some_parameter"] = 4.0f;
params.ClearDirty();

for (const auto name_param : params)
for (const auto &name_param : params)
{
EXPECT_EQ(std::string("some_parameter"), name_param.first);
}
Expand Down

0 comments on commit 45caad0

Please sign in to comment.