5f17068...
by
Karsten Knese <email address hidden>
Avoid range-loop-construct in TopicManager (#2983)
Clang (on OSX) issues a warning at the TopicManager.
```
error: loop variable 'liter' of type 'const boost::shared_ptr<gazebo::transport::Node>' creates a copy from type 'const boost::shared_ptr<gazebo::transport::Node>' for (const auto liter : iter2.second) ^
note: use reference type 'const boost::shared_ptr<gazebo::transport::Node> &' to prevent copying for (const auto liter : iter2.second) ^~~~~~~~~~~~~~~~~~ &
```
As proposed, using it as a reference silences the warning.
* Remove modifications to the shadow technique done in the
DepthCamera, this is not necessary and breaks lighting for all
other cameras that will render after it in the same frame
* Add call to setShadowsEnabled for the normals viewport as well
Co-authored-by: Steve Peters <email address hidden>