Formatting, tweaks

src/blackcore/context_simulator.h

@@ -108,6 +108,6 @@ namespace BlackCore
         IContextSimulator(CRuntimeConfig::ContextMode mode, CRuntime *runtime) : CContext(mode, runtime) {}
     };
 
-} // namespace BlackCore
+} // namespace
 
 #endif // guard

src/blackcore/simulator.h

@@ -111,11 +111,7 @@ namespace BlackCore
         //! ISimulatorVirtual destructor
         virtual ~ISimulatorFactory() {}
 
-        /*!
+        //! Create a new instance
-         * ISimulatorCreate a new instance
-         * \param parent
-         * \return
-         */
         virtual ISimulator *create(QObject *parent = nullptr) = 0;
 
         //! Simulator info

src/blackgui/aircraftlistmodel.h

@@ -14,22 +14,17 @@
 namespace BlackGui
 {
     /*!
-     * \brief Aircraft list model
+     * Aircraft list model
      */
     class CAircraftListModel : public CListModelBase<BlackMisc::Aviation::CAircraft, BlackMisc::Aviation::CAircraftList>
     {
 
     public:
 
-        /*!
+        //! Constructor
-         * \brief Constructor
-         * \param parent
-         */
         explicit CAircraftListModel(QObject *parent = nullptr);
 
-        /*!
+        //! Destructor
-         * \brief Destructor
-         */
         virtual ~CAircraftListModel() {}
 
     };

src/blackgui/aircraftview.h

@@ -7,7 +7,7 @@
 namespace BlackGui
 {
     /*!
-     * \brief Aircrafts view
+     * Aircrafts view
      */
     class CAircraftView : public CViewBase<CAircraftListModel>
     {

src/blackgui/airportlistmodel.h

@@ -15,10 +15,10 @@ namespace BlackGui
 
     public:
 
-        //! \brief Constructor
+        //! Constructor
         explicit CAirportListModel(QObject *parent = nullptr);
 
-        //! \brief Destructor
+        //! Destructor
         virtual ~CAirportListModel() {}
     };
 }

src/blackgui/listmodelbase.cpp

@@ -7,6 +7,7 @@
 #include "blackmisc/statusmessagelist.h"
 #include "blackmisc/avatcstationlist.h"
 #include "blackmisc/avaircraftlist.h"
+#include "blackmisc/avairportlist.h"
 #include "blackmisc/nwserverlist.h"
 #include "blackmisc/nwuserlist.h"
 #include "blackmisc/nwclientlist.h"
@@ -228,6 +229,7 @@ namespace BlackGui
     template class CListModelBase<BlackMisc::CStatusMessage, BlackMisc::CStatusMessageList>;
     template class CListModelBase<BlackMisc::Aviation::CAtcStation, BlackMisc::Aviation::CAtcStationList>;
     template class CListModelBase<BlackMisc::Aviation::CAircraft, BlackMisc::Aviation::CAircraftList>;
+    template class CListModelBase<BlackMisc::Aviation::CAirport, BlackMisc::Aviation::CAirportList>;
     template class CListModelBase<BlackMisc::Network::CServer, BlackMisc::Network::CServerList>;
     template class CListModelBase<BlackMisc::Network::CUser, BlackMisc::Network::CUserList>;
     template class CListModelBase<BlackMisc::Network::CClient, BlackMisc::Network::CClientList>;

src/blackgui/timerbasedcomponent.cpp

@@ -19,7 +19,9 @@ namespace BlackGui
     void CTimerBasedComponent::setUpdateInterval(int milliSeconds)
     {
         if (milliSeconds < 100)
+        {
             this->m_timer->stop();
+        }
         else
         {
             this->m_timer->setInterval(milliSeconds);

src/plugins/simulator/xplane/simulator_xplane.cpp

@@ -239,7 +239,7 @@ namespace BlackSimPlugin
         }
 
         void CSimulatorXPlane::addRemoteAircraft(const BlackMisc::Aviation::CCallsign &callsign, const QString &type,
-            const BlackMisc::Aviation::CAircraftSituation &initialSituation)
+                const BlackMisc::Aviation::CAircraftSituation &initialSituation)
         {
             Q_UNUSED(initialSituation); // fixed gcc warning
             if (! isConnected()) { return; }
@@ -249,18 +249,18 @@ namespace BlackSimPlugin
         }
 
         void CSimulatorXPlane::addAircraftSituation(const BlackMisc::Aviation::CCallsign &callsign,
-            const BlackMisc::Aviation::CAircraftSituation &situ)
+                const BlackMisc::Aviation::CAircraftSituation &situ)
         {
             if (! isConnected()) { return; }
             if (! m_planes.contains(callsign.asString())) { addRemoteAircraft(callsign, "A320", situ); } // FIXME should not be needed here IMHO
             using namespace BlackMisc::PhysicalQuantities;
             m_traffic->setPlanePosition(callsign.asString(),
-                situ.latitude().value(CAngleUnit::deg()),
+                                        situ.latitude().value(CAngleUnit::deg()),
-                situ.longitude().value(CAngleUnit::deg()),
+                                        situ.longitude().value(CAngleUnit::deg()),
-                situ.getAltitude().value(CLengthUnit::ft()),
+                                        situ.getAltitude().value(CLengthUnit::ft()),
-                situ.getPitch().value(CAngleUnit::deg()),
+                                        situ.getPitch().value(CAngleUnit::deg()),
-                situ.getBank().value(CAngleUnit::deg()),
+                                        situ.getBank().value(CAngleUnit::deg()),
-                situ.getHeading().value(CAngleUnit::deg()));
+                                        situ.getHeading().value(CAngleUnit::deg()));
             m_traffic->setPlaneSurfaces(callsign.asString(), true, 0, 0, 0, 0, 0, 0, 0, 0, 0, true, true, true, true, 0); // TODO landing gear, lights, control surfaces
             m_traffic->setPlaneTransponder(callsign.asString(), 2000, true, false); // TODO transponder
         }