Mirror/ardupilot
5
0
Fork 0
mirror of https://github.com/ArduPilot/ardupilot synced 2025-01-03 06:28:27 -04:00
Code Issues Packages Projects Releases Wiki Activity

AP_Frsky_Telem: removed create() method for objects

Browse Source 
See discussion here:

  https://github.com/ArduPilot/ardupilot/issues/7331

we were getting some uninitialised variables. While it only showed up in
AP_SbusOut, it means we can't be sure it won't happen on other objects,
so safest to remove the approach

Thanks to assistance from Lucas, Peter and Francisco
This commit is contained in:
Andrew Tridgell 2017-12-13 12:06:12 +11:00
parent f326377a5c
commit c618a27ee0
1 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
libraries/AP_Frsky_Telem/AP_Frsky_Telem.h
View File

@ -113,11 +113,7 @@ for FrSky SPort Passthrough
class AP_Frsky_Telem { class AP_Frsky_Telem {
public: public:
static AP_Frsky_Telem create(AP_AHRS &ahrs, const AP_BattMonitor &battery, const RangeFinder &rng) { AP_Frsky_Telem(AP_AHRS &ahrs, const AP_BattMonitor &battery, const RangeFinder &rng);
return AP_Frsky_Telem{ahrs, battery, rng};
}
constexpr AP_Frsky_Telem(AP_Frsky_Telem &&other) = default;
/* Do not allow copies */ /* Do not allow copies */
AP_Frsky_Telem(const AP_Frsky_Telem &other) = delete; AP_Frsky_Telem(const AP_Frsky_Telem &other) = delete;
@ -146,8 +142,6 @@ public:
static ObjectArray<mavlink_statustext_t> _statustext_queue; static ObjectArray<mavlink_statustext_t> _statustext_queue;
private: private:
AP_Frsky_Telem(AP_AHRS &ahrs, const AP_BattMonitor &battery, const RangeFinder &rng);
AP_AHRS &_ahrs; AP_AHRS &_ahrs;
const AP_BattMonitor &_battery; const AP_BattMonitor &_battery;
const RangeFinder &_rng; const RangeFinder &_rng;