formatting, require virtual base destructor to compile

ouster-lidar · Nov 8, 2023 · e1a9da5 · e1a9da5
1 parent 9007d1e
commit e1a9da5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/src/os_driver_nodelet.cpp b/src/os_driver_nodelet.cpp
@@ -32,7 +32,7 @@ namespace ouster_ros {
 class OusterDriver : public OusterSensor {
    public:
     OusterDriver() : tf_bcast(getName()) {}
-    virtual ~OusterDriver() {
+    ~OusterDriver() override {
         NODELET_DEBUG("OusterDriver::~OusterDriver() called");
         halt();
     }

diff --git a/src/os_sensor_nodelet.h b/src/os_sensor_nodelet.h
@@ -30,10 +30,8 @@ namespace ouster_ros {
 
 class OusterSensor : public OusterSensorNodeletBase {
    public:
-    ~OusterSensor();
+    ~OusterSensor() override;
 
-    void halt();
-
    private:
     virtual void onInit() override;
 
@@ -48,6 +46,8 @@ class OusterSensor : public OusterSensorNodeletBase {
 
     virtual void on_imu_packet_msg(const uint8_t* raw_imu_packet);
 
+    void halt();
+
    private:
     std::string get_sensor_hostname();
 
@@ -80,7 +80,6 @@ class OusterSensor : public OusterSensorNodeletBase {
 
     std::string load_config_file(const std::string& config_file);
 
-   private:
     // fill in values that could not be parsed from metadata
     void populate_metadata_defaults(sensor::sensor_info& info,
                                     sensor::lidar_mode specified_lidar_mode);