Lint

Signed-off-by: Michael Carroll <[email protected]>

include/ignition/gazebo/detail/View.hh

@@ -17,6 +17,7 @@
 #ifndef IGNITION_GAZEBO_DETAIL_VIEW_HH_
 #define IGNITION_GAZEBO_DETAIL_VIEW_HH_
 
+#include <memory>
 #include <tuple>
 #include <unordered_map>
 #include <unordered_set>

src/Component_TEST.cc

@@ -178,7 +178,7 @@ class NoSerialize : public components::BaseComponent
  return 0;
  }
 
- public: std::unique_ptr<BaseComponent> Clone() const override 
+ public: std::unique_ptr<BaseComponent> Clone() const override
  {
  return nullptr;
  }

src/EntityComponentManager.cc

@@ -311,12 +311,12 @@ void EntityComponentManagerPrivate::Copy(
  this->descendantCache.clear();
  this->entityCount = _from.entityCount;
  this->removedComponents = _from.removedComponents;
- this->componentsMarkedAsRemoved= _from.componentsMarkedAsRemoved;
+ this->componentsMarkedAsRemoved = _from.componentsMarkedAsRemoved;
 
  for (const auto &[entity, comps] : _from.componentStorage)
  {
  this->componentStorage[entity].clear();
- for (const auto &comp: comps)
+ for (const auto &comp : comps)
  {
  this->componentStorage[entity].emplace_back(comp->Clone());
  }
@@ -325,8 +325,8 @@ void EntityComponentManagerPrivate::Copy(
  this->componentTypeIndexIterators.clear();
  this->componentTypeIndexDirty = true;
 
- // Not copying maps related to cloning since they are transient variables that 
- // are used as return values of some member functions.
+ // Not copying maps related to cloning since they are transient variables
+ // that are used as return values of some member functions.
 
  this->pinnedEntities = _from.pinnedEntities;
 }
@@ -2185,8 +2185,9 @@ void EntityComponentManager::ApplyDiff(const EntityComponentManager &_other,
  {
  this->dataPtr->CreateEntityImplementation(entity);
  this->RequestRemoveEntity(entity, false);
- // We want to set this entity as "removed", but CreateEntityImplementation 
- // set's is as "newlyCreated", so remove it from that list.
+ // We want to set this entity as "removed", but 
+ // CreateEntityImplementation sets it as "newlyCreated", 
+ // so remove it from that list.
  {
  std::lock_guard<std::mutex> lock(this->dataPtr->entityCreatedMutex);
  this->dataPtr->newlyCreatedEntities.erase(entity);

src/EntityComponentManagerDiff.hh

@@ -22,6 +22,8 @@
 #include "ignition/gazebo/Export.hh"
 #include "ignition/gazebo/Types.hh"
 
+#include <vector>
+
 namespace ignition
 {
  namespace gazebo

src/EntityComponentManager_TEST.cc

@@ -3173,7 +3173,7 @@ TEST_P(EntityComponentManagerFixture, ComputDiff)
  EXPECT_EQ(0u, diff.RemovedEntities().size());
  }
 
- // Now add another component to managerCopy. We should expect one more entity 
+ // Now add another component to managerCopy. We should expect one more entity
  // in RemovedEntities
  managerCopy.SetEntityCreateOffset(10);
  managerCopy.CreateEntity();

src/SimulationRunner.hh

@@ -520,8 +520,8 @@ namespace ignition
  /// \brief Manager of all components.
  private: EntityComponentManager entityCompMgr;
 
- /// \brief Copy of the EntityComponentManager immediately after the 
- /// initial entity creation/world load. 
+ /// \brief Copy of the EntityComponentManager immediately after the
+ /// initial entity creation/world load.
  private: EntityComponentManager initialEntityCompMgr;
 
  /// \brief Manager of all levels.