Browse Source

clang-tidy: partially fix hicpp-use-override

sbg
Daniel Agar 5 years ago
parent
commit
c284198bec
  1. 2
      src/drivers/camera_trigger/camera_trigger.cpp
  2. 2
      src/drivers/gps/gps.cpp
  3. 10
      src/lib/cdev/test/cdevtest_example.cpp
  4. 6
      src/lib/drivers/led/led.cpp
  5. 2
      src/lib/rc/rc_tests/RCTest.cpp
  6. 2
      src/modules/airspeed_selector/airspeed_selector_main.cpp
  7. 4
      src/modules/commander/commander_tests/state_machine_helper_test.cpp
  8. 2
      src/modules/load_mon/load_mon.cpp
  9. 2
      src/modules/local_position_estimator/local_position_estimator_main.cpp
  10. 428
      src/modules/mavlink/mavlink_messages.cpp
  11. 2
      src/modules/mc_pos_control/mc_pos_control_main.cpp
  12. 8
      src/modules/simulator/gpssim/gpssim.cpp
  13. 8
      src/modules/simulator/ledsim/led.cpp

2
src/drivers/camera_trigger/camera_trigger.cpp

@ -103,7 +103,7 @@ public: @@ -103,7 +103,7 @@ public:
/**
* Destructor, also kills task.
*/
~CameraTrigger();
~CameraTrigger() override;
/**
* Run intervalometer update

2
src/drivers/gps/gps.cpp

@ -97,7 +97,7 @@ public: @@ -97,7 +97,7 @@ public:
GPS(const char *path, gps_driver_mode_t mode, GPSHelper::Interface interface, bool fake_gps, bool enable_sat_info,
Instance instance, unsigned configured_baudrate);
virtual ~GPS();
~GPS() override;
/** @see ModuleBase */
static int task_spawn(int argc, char *argv[]);

10
src/lib/cdev/test/cdevtest_example.cpp

@ -106,12 +106,12 @@ public: @@ -106,12 +106,12 @@ public:
_is_open_for_write(false),
_write_offset(0) {}
~CDevNode() = default;
~CDevNode() override = default;
virtual int open(cdev::file_t *handlep);
virtual int close(cdev::file_t *handlep);
virtual ssize_t write(cdev::file_t *handlep, const char *buffer, size_t buflen);
virtual ssize_t read(cdev::file_t *handlep, char *buffer, size_t buflen);
int open(cdev::file_t *handlep) override;
int close(cdev::file_t *handlep) override;
ssize_t write(cdev::file_t *handlep, const char *buffer, size_t buflen) override;
ssize_t read(cdev::file_t *handlep, char *buffer, size_t buflen) override;
private:
bool _is_open_for_write;
size_t _write_offset;

6
src/lib/drivers/led/led.cpp

@ -60,10 +60,10 @@ class LED : cdev::CDev @@ -60,10 +60,10 @@ class LED : cdev::CDev
{
public:
LED();
virtual ~LED() = default;
~LED() override = default;
virtual int init();
virtual int ioctl(cdev::file_t *filp, int cmd, unsigned long arg);
int init() override;
int ioctl(cdev::file_t *filp, int cmd, unsigned long arg) override;
};
LED::LED() : CDev(LED0_DEVICE_PATH)

2
src/lib/rc/rc_tests/RCTest.cpp

@ -26,7 +26,7 @@ extern "C" __EXPORT int rc_tests_main(int argc, char *argv[]); @@ -26,7 +26,7 @@ extern "C" __EXPORT int rc_tests_main(int argc, char *argv[]);
class RCTest : public UnitTest
{
public:
virtual bool run_tests();
bool run_tests() override;
private:
bool crsfTest();

2
src/modules/airspeed_selector/airspeed_selector_main.cpp

@ -76,7 +76,7 @@ public: @@ -76,7 +76,7 @@ public:
AirspeedModule();
~AirspeedModule();
~AirspeedModule() override;
/** @see ModuleBase */
static int task_spawn(int argc, char *argv[]);

4
src/modules/commander/commander_tests/state_machine_helper_test.cpp

@ -47,9 +47,9 @@ class StateMachineHelperTest : public UnitTest @@ -47,9 +47,9 @@ class StateMachineHelperTest : public UnitTest
{
public:
StateMachineHelperTest() = default;
virtual ~StateMachineHelperTest() = default;
~StateMachineHelperTest() override = default;
virtual bool run_tests();
bool run_tests() override;
private:
bool armingStateTransitionTest();

2
src/modules/load_mon/load_mon.cpp

@ -71,7 +71,7 @@ class LoadMon : public ModuleBase<LoadMon>, public ModuleParams, public px4::Sch @@ -71,7 +71,7 @@ class LoadMon : public ModuleBase<LoadMon>, public ModuleParams, public px4::Sch
{
public:
LoadMon();
~LoadMon();
~LoadMon() override;
static int task_spawn(int argc, char *argv[]);

2
src/modules/local_position_estimator/local_position_estimator_main.cpp

@ -51,7 +51,7 @@ extern "C" __EXPORT int local_position_estimator_main(int argc, char *argv[]); @@ -51,7 +51,7 @@ extern "C" __EXPORT int local_position_estimator_main(int argc, char *argv[]);
class LocalPositionEstimatorModule : public ModuleBase<LocalPositionEstimatorModule>
{
public:
virtual ~LocalPositionEstimatorModule() = default;
~LocalPositionEstimatorModule() override = default;
/** @see ModuleBase */
static int task_spawn(int argc, char *argv[]);

428
src/modules/mavlink/mavlink_messages.cpp

File diff suppressed because it is too large Load Diff

2
src/modules/mc_pos_control/mc_pos_control_main.cpp

@ -85,7 +85,7 @@ class MulticopterPositionControl : public ModuleBase<MulticopterPositionControl> @@ -85,7 +85,7 @@ class MulticopterPositionControl : public ModuleBase<MulticopterPositionControl>
public:
MulticopterPositionControl();
virtual ~MulticopterPositionControl() override;
~MulticopterPositionControl() override;
/** @see ModuleBase */
static int task_spawn(int argc, char *argv[]);

8
src/modules/simulator/gpssim/gpssim.cpp

@ -87,11 +87,11 @@ class GPSSIM : public VirtDevObj @@ -87,11 +87,11 @@ class GPSSIM : public VirtDevObj
public:
GPSSIM(bool fake_gps, bool enable_sat_info,
int fix_type, int num_sat, int noise_multiplier);
virtual ~GPSSIM();
~GPSSIM() override;
virtual int init();
int init() override;
virtual int devIOCTL(unsigned long cmd, unsigned long arg);
int devIOCTL(unsigned long cmd, unsigned long arg) override;
void set(int fix_type, int num_sat, int noise_multiplier);
@ -101,7 +101,7 @@ public: @@ -101,7 +101,7 @@ public:
void print_info();
protected:
virtual void _measure() {}
void _measure() override {}
private:

8
src/modules/simulator/ledsim/led.cpp

@ -63,13 +63,13 @@ class LED : public VirtDevObj @@ -63,13 +63,13 @@ class LED : public VirtDevObj
{
public:
LED();
virtual ~LED() = default;
~LED() override = default;
virtual int init();
virtual int devIOCTL(unsigned long cmd, unsigned long arg);
int init() override;
int devIOCTL(unsigned long cmd, unsigned long arg) override;
protected:
virtual void _measure() {}
void _measure() override {}
};
LED::LED() :

Loading…
Cancel
Save