Style

src/blackcore/context/contextsimulator.cpp

@@ -63,7 +63,7 @@ namespace BlackCore
 
         bool IContextSimulator::isSimulatorAvailable() const
         {
-            return CBuildConfig::isCompiledWithFlightSimulatorSupport() && !getSimulatorPluginInfo().isUnspecified();
+            return CBuildConfig::isCompiledWithFlightSimulatorSupport() && !this->getSimulatorPluginInfo().isUnspecified();
         }
 
         bool IContextSimulator::isSimulatorSimulating() const

src/blackgui/components/simulatorselector.cpp

@@ -126,7 +126,7 @@ namespace BlackGui
             if (sGui && sGui->supportsContexts() && sGui->getIContextSimulator())
             {
                 const CSimulatorPluginInfo pluginInfo = sGui->getIContextSimulator()->getSimulatorPluginInfo();
-                if (!this->isSingleSelection()) { this->setMode(RadioButtons); }
+                if (!this->isSingleSelection()) { this->setMode(RadioButtons); } // only one sim can be connected
 
                 if (pluginInfo.isValid())
                 {
@@ -319,12 +319,14 @@ namespace BlackGui
             if (this->isSingleSelection())
             {
                 // single
-                m_currentSimulator.set(this->getValue());
+                const CSimulatorInfo sim = this->getValue();
+                m_currentSimulator.set(sim);
             }
             else
             {
                 // multiple
-                m_currentSimulators.set(this->getValue());
+                const CSimulatorInfo sim = this->getValue();
+                m_currentSimulators.set(sim);
             }
         }
 

src/blackgui/editors/aircrafticaoform.cpp

@@ -154,8 +154,8 @@ namespace BlackGui
         CStatusMessageList CAircraftIcaoForm::validate(bool nested) const
         {
             Q_UNUSED(nested);
-            CAircraftIcaoCode code(getValue());
+            const CAircraftIcaoCode code(this->getValue());
-            CStatusMessageList msgs(code.validate());
+            const CStatusMessageList msgs(code.validate());
             ui->val_Indicator->setState(msgs);
             return msgs;
         }

src/blackgui/editors/aircrafticaoform.h

@@ -11,8 +11,8 @@
 #ifndef BLACKGUI_EDITORS_AIRCRAFTICAOFORM_H
 #define BLACKGUI_EDITORS_AIRCRAFTICAOFORM_H
 
-#include "blackgui/blackguiexport.h"
 #include "blackgui/editors/form.h"
+#include "blackgui/blackguiexport.h"
 #include "blackmisc/aviation/aircrafticaocode.h"
 #include "blackmisc/statusmessagelist.h"
 #include "blackmisc/variant.h"
@@ -20,8 +20,6 @@
 #include <QObject>
 #include <QScopedPointer>
 
-class QWidget;
-
 namespace Ui { class CAircraftIcaoForm; }
 namespace BlackGui
 {

src/blackgui/editors/coordinateform.cpp

@@ -77,7 +77,6 @@ namespace BlackGui
             if (coordinate == m_coordinate) { return false; }
             m_coordinate = coordinate;
 
-
             const CLatitude lat = coordinate.latitude();
             const QString latWgs = lat.toWgs84();
             ui->le_Latitude->setText(latWgs);

src/blackgui/editors/form.h

@@ -35,7 +35,7 @@ namespace BlackGui
             explicit CForm(QWidget *parent = nullptr);
 
             //! Destructor
-            virtual ~CForm();
+            virtual ~CForm() override;
 
             //! Set editable
             virtual void setReadOnly(bool readOnly) = 0;

src/plugins/simulator/emulated/simulatoremulatedmonitordialog.h

@@ -42,7 +42,7 @@ namespace BlackSimPlugin
             explicit CSimulatorEmulatedMonitorDialog(CSimulatorEmulated *simulator, QWidget *parent = nullptr);
 
             //! Dtor
-            virtual ~CSimulatorEmulatedMonitorDialog();
+            virtual ~CSimulatorEmulatedMonitorDialog() override;
 
             //! \copydoc BlackGui::Components::CLogComponent::appendStatusMessageToList
             void appendStatusMessageToList(const BlackMisc::CStatusMessage &statusMessage);

src/plugins/simulator/emulated/simulatoremulatedmonitordialog.ui

@@ -68,7 +68,7 @@
    <item>
     <widget class="QTabWidget" name="tw_SwiftMonitorDialog">
      <property name="currentIndex">
-      <number>1</number>
+      <number>0</number>
      </property>
      <widget class="QWidget" name="tb_AircraftSituation">
       <attribute name="title">

src/plugins/simulator/fsxcommon/simulatorfsxsimconnectproc.cpp

@@ -80,7 +80,7 @@ namespace BlackSimPlugin
                         {
                             if (trace.isValid())
                             {
-                                // it can happen the object is not yet
+                                // it can happen the object is not yet existing
                                 CSimConnectObject simObject = simulatorFsxP3D->getSimObjectForTrace(trace);
                                 if (simObject.isInvalid()) { simObject = trace.simObject; } // take the one in the trace
                                 if (simObject.isValid())