From 351e33cf8ed388c6eb76f61416edd3bb8e735665 Mon Sep 17 00:00:00 2001
From: Lorenz Meier <lm@inf.ethz.ch>
Date: Sat, 31 Oct 2015 18:14:01 +0100
Subject: [PATCH] Remove Flightgear and JSBSim and only leave the option which
 actually works.

---
 src/ui/QGCHilConfiguration.cc           | 28 +++++++++++++------------
 src/ui/QGCHilConfiguration.ui           |  7 +------
 src/ui/QGCHilFlightGearConfiguration.cc | 10 ++++-----
 3 files changed, 21 insertions(+), 24 deletions(-)

diff --git a/src/ui/QGCHilConfiguration.cc b/src/ui/QGCHilConfiguration.cc
index bdd163fe3f..ced0d79630 100644
--- a/src/ui/QGCHilConfiguration.cc
+++ b/src/ui/QGCHilConfiguration.cc
@@ -114,17 +114,19 @@ void QGCHilConfiguration::on_simComboBox_currentIndexChanged(int index)
             connect(xplane, SIGNAL(statusMessage(QString)), ui->statusLabel, SLOT(setText(QString)));
         }
     }
-    else if (4)
-    {
-        // Ensure the sim exists and is disabled
-        _vehicle->uas()->enableHilJSBSim(false, "");
-        QGCHilJSBSimConfiguration* hfgconf = new QGCHilJSBSimConfiguration(_vehicle, this);
-        hfgconf->show();
-        ui->simulatorConfigurationLayout->addWidget(hfgconf);
-        QGCJSBSimLink* jsb = dynamic_cast<QGCJSBSimLink*>(_vehicle->uas()->getHILSimulation());
-        if (jsb)
-        {
-            connect(jsb, SIGNAL(statusMessage(QString)), ui->statusLabel, SLOT(setText(QString)));
-        }
-    }
+// Disabling JSB Sim since its not well maintained,
+// but as refactoring is pending we're not ditching the code yet
+//    else if (4)
+//    {
+//        // Ensure the sim exists and is disabled
+//        _vehicle->uas()->enableHilJSBSim(false, "");
+//        QGCHilJSBSimConfiguration* hfgconf = new QGCHilJSBSimConfiguration(_vehicle, this);
+//        hfgconf->show();
+//        ui->simulatorConfigurationLayout->addWidget(hfgconf);
+//        QGCJSBSimLink* jsb = dynamic_cast<QGCJSBSimLink*>(_vehicle->uas()->getHILSimulation());
+//        if (jsb)
+//        {
+//            connect(jsb, SIGNAL(statusMessage(QString)), ui->statusLabel, SLOT(setText(QString)));
+//        }
+//    }
 }
diff --git a/src/ui/QGCHilConfiguration.ui b/src/ui/QGCHilConfiguration.ui
index f80a307e3b..82e9cc10f9 100644
--- a/src/ui/QGCHilConfiguration.ui
+++ b/src/ui/QGCHilConfiguration.ui
@@ -52,11 +52,6 @@
        <string>X-Plane 9</string>
       </property>
      </item>
-     <item>
-      <property name="text">
-       <string>JSBSim</string>
-      </property>
-     </item>
     </widget>
    </item>
    <item row="1" column="0" colspan="2">
@@ -69,7 +64,7 @@
    <item row="2" column="0" colspan="2">
     <widget class="QLabel" name="statusLabel">
      <property name="text">
-      <string></string>
+      <string/>
      </property>
     </widget>
    </item>
diff --git a/src/ui/QGCHilFlightGearConfiguration.cc b/src/ui/QGCHilFlightGearConfiguration.cc
index d183e300f3..e7e3c24697 100644
--- a/src/ui/QGCHilFlightGearConfiguration.cc
+++ b/src/ui/QGCHilFlightGearConfiguration.cc
@@ -29,18 +29,18 @@ QGCHilFlightGearConfiguration::QGCHilFlightGearConfiguration(Vehicle* vehicle, Q
     QStringList items;
     if (_vehicle->vehicleType() == MAV_TYPE_FIXED_WING)
     {
-        items << "EasyStar";
+        /*items << "EasyStar";*/
         items << "Rascal110-JSBSim";
-        items << "c172p";
+        /*items << "c172p";
         items << "YardStik";
-        items << "Malolo1";
+        items << "Malolo1";*/
         _mavSettingsSubGroup = _mavSettingsSubGroupFixedWing;
     }
-    else if (_vehicle->vehicleType() == MAV_TYPE_QUADROTOR)
+    /*else if (_vehicle->vehicleType() == MAV_TYPE_QUADROTOR)
     {
         items << "arducopter";
         _mavSettingsSubGroup = _mavSettingsSubGroupQuadRotor;
-    }
+    }*/
     else
     {
         // FIXME: Should disable all input, won't work. Show error message in the status label thing.
-- 
GitLab