use new has connections function

Signed-off-by: Ian Chen <[email protected]>
gazebosim · Jun 8, 2022 · 6d86600 · 6d86600
1 parent 62de3a2
commit 6d86600
Showing 1 changed file with 1 addition and 49 deletions.
diff --git a/src/systems/sensors/Sensors.cc b/src/systems/sensors/Sensors.cc
@@ -189,11 +189,6 @@ class gz::sim::systems::SensorsPrivate
   /// \param[in] _ecm Entity component manager
   public: void UpdateBatteryState(const EntityComponentManager &_ecm);
 
-  /// \brief Check if sensor has subscribers
-  /// \param[in] _sensor Sensor to check
-  /// \return True if the sensor has subscribers, false otherwise
-  public: bool HasConnections(sensors::RenderingSensor *_sensor) const;
-
   /// \brief Use to optionally set the background color.
   public: std::optional<math::Color> backgroundColor;
 
@@ -332,7 +327,7 @@ void SensorsPrivate::RunOnce()
     {
       sensors::Sensor *s = this->sensorManager.Sensor(id);
       auto rs = dynamic_cast<sensors::RenderingSensor *>(s);
-      if (rs->IsActive() && !this->HasConnections(rs))
+      if (rs->IsActive() && !rs->HasConnections())
       {
         rs->SetActive(false);
         tmpDisabledSensors.insert(rs);
@@ -841,49 +836,6 @@ std::string Sensors::CreateSensor(const Entity &_entity,
   return sensor->Name();
 }
 
-//////////////////////////////////////////////////
-bool SensorsPrivate::HasConnections(sensors::RenderingSensor *_sensor) const
-{
-  if (!_sensor)
-    return true;
-
-  // \todo(iche033) Remove this function once a virtual
-  // sensors::RenderingSensor::HasConnections function is available
-  {
-    auto s = dynamic_cast<sensors::RgbdCameraSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  {
-    auto s = dynamic_cast<sensors::DepthCameraSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  {
-    auto s = dynamic_cast<sensors::GpuLidarSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  {
-    auto s = dynamic_cast<sensors::SegmentationCameraSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  {
-    auto s = dynamic_cast<sensors::ThermalCameraSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  {
-    auto s = dynamic_cast<sensors::CameraSensor *>(_sensor);
-    if (s)
-      return s->HasConnections();
-  }
-  gzwarn << "Unable to check connection count for sensor: " << _sensor->Name()
-          << ". Unknown sensor type." << std::endl;
-  return true;
-}
-
 IGNITION_ADD_PLUGIN(Sensors, System,
   Sensors::ISystemConfigure,
   Sensors::ISystemUpdate,