Commit ea63d012 authored by Tomaz Canabrava's avatar Tomaz Canabrava Committed by Daniel Agar

UAS: Use qobject_cast instead of dynamic_cast

qobject_cast is times faster dynamic_cast for objects that
inherit QObject. also, use auto keyword to not duplicate
class names.
parent baf44585
...@@ -1094,7 +1094,7 @@ void UAS::pairRX(int rxType, int rxSubType) ...@@ -1094,7 +1094,7 @@ void UAS::pairRX(int rxType, int rxSubType)
#ifndef __mobile__ #ifndef __mobile__
void UAS::enableHilJSBSim(bool enable, QString options) void UAS::enableHilJSBSim(bool enable, QString options)
{ {
QGCJSBSimLink* link = dynamic_cast<QGCJSBSimLink*>(simulation); auto* link = qobject_cast<QGCJSBSimLink*>(simulation);
if (!link) { if (!link) {
// Delete wrong sim // Delete wrong sim
if (simulation) { if (simulation) {
...@@ -1104,7 +1104,7 @@ void UAS::enableHilJSBSim(bool enable, QString options) ...@@ -1104,7 +1104,7 @@ void UAS::enableHilJSBSim(bool enable, QString options)
simulation = new QGCJSBSimLink(_vehicle, options); simulation = new QGCJSBSimLink(_vehicle, options);
} }
// Connect Flight Gear Link // Connect Flight Gear Link
link = dynamic_cast<QGCJSBSimLink*>(simulation); link = qobject_cast<QGCJSBSimLink*>(simulation);
link->setStartupArguments(options); link->setStartupArguments(options);
if (enable) if (enable)
{ {
...@@ -1123,7 +1123,7 @@ void UAS::enableHilJSBSim(bool enable, QString options) ...@@ -1123,7 +1123,7 @@ void UAS::enableHilJSBSim(bool enable, QString options)
#ifndef __mobile__ #ifndef __mobile__
void UAS::enableHilXPlane(bool enable) void UAS::enableHilXPlane(bool enable)
{ {
QGCXPlaneLink* link = dynamic_cast<QGCXPlaneLink*>(simulation); auto* link = qobject_cast<QGCXPlaneLink*>(simulation);
if (!link) { if (!link) {
if (simulation) { if (simulation) {
stopHil(); stopHil();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment