diff --git a/samples/blackmiscsim/samplesmodelmapping.cpp b/samples/blackmiscsim/samplesmodelmapping.cpp index 839b01c83..34f9c4650 100644 --- a/samples/blackmiscsim/samplesmodelmapping.cpp +++ b/samples/blackmiscsim/samplesmodelmapping.cpp @@ -67,10 +67,10 @@ namespace BlackSample streamOut << "Ambigious models: " << cfgParser.getAircraftCfgEntriesList().detectAmbiguousTitles().join(", ") << endl; // sync definitions, remove redundant ones - CAircraftMatcher matcher(CAircraftMatcher::All); + CAircraftMatcher matcher; matcher.setModelSet(vpRulesReader.getAsModelsFromCache(), CSimulatorInfo::FSX); - CAircraftIcaoCode icao("C172"); + const CAircraftIcaoCode icao("C172"); streamOut << "Searching for " << icao << endl; streamOut << matcher.getModelSet().findByIcaoDesignators(icao, CAirlineIcaoCode()) << endl; diff --git a/src/blackcore/context/contextsimulatorimpl.cpp b/src/blackcore/context/contextsimulatorimpl.cpp index 44e579b5d..bd4459227 100644 --- a/src/blackcore/context/contextsimulatorimpl.cpp +++ b/src/blackcore/context/contextsimulatorimpl.cpp @@ -66,7 +66,6 @@ namespace BlackCore CContextSimulator::registerHelp(); Q_ASSERT_X(sApp, Q_FUNC_INFO, "Need sApp"); - // m_enableMatchingMessages = CBuildConfig::isLocalDeveloperDebugBuild() || sApp->isDeveloperFlagSet(); m_enableMatchingMessages = true; // there seems to be no big disadavantage in always enabling it m_plugins->collectPlugins(); this->restoreSimulatorPlugins(); @@ -403,7 +402,7 @@ namespace BlackCore Q_UNUSED(c); // Once the simulator signaled it is ready to simulate, add all known aircraft - m_initallyAddAircrafts = true; + m_initallyAddAircraft = true; m_matchingMessages.clear(); // try to connect to simulator @@ -548,7 +547,7 @@ namespace BlackCore void CContextSimulator::onSimulatorStatusChanged(ISimulator::SimulatorStatus status) { - if (m_initallyAddAircrafts && status.testFlag(ISimulator::Simulating)) + if (m_initallyAddAircraft && status.testFlag(ISimulator::Simulating)) { // use network to initally add aircraft IContextNetwork *networkContext = this->getIContextNetwork(); @@ -562,7 +561,7 @@ namespace BlackCore BLACK_VERIFY_X(!simulatedAircraft.getCallsign().isEmpty(), Q_FUNC_INFO, "Need callsign"); this->xCtxAddedRemoteAircraftReadyForModelMatching(simulatedAircraft); } - m_initallyAddAircrafts = false; + m_initallyAddAircraft = false; } if (!status.testFlag(ISimulator::Connected)) { diff --git a/src/blackcore/context/contextsimulatorimpl.h b/src/blackcore/context/contextsimulatorimpl.h index 322a80795..c83744079 100644 --- a/src/blackcore/context/contextsimulatorimpl.h +++ b/src/blackcore/context/contextsimulatorimpl.h @@ -122,7 +122,7 @@ namespace BlackCore public: //! Destructor - virtual ~CContextSimulator(); + virtual ~CContextSimulator() override; //! Gracefully shut down, e.g. for plugin unloading void gracefulShutdown(); @@ -238,12 +238,12 @@ namespace BlackCore void initByLastUsedModelSet(); QPair> m_simulatorPlugin; //!< Currently loaded simulator plugin - CPluginManagerSimulator *m_plugins = nullptr; - BlackMisc::CRegularThread m_listenersThread; //!< waiting for plugin - CWeatherManager m_weatherManager { this }; - CAircraftMatcher m_aircraftMatcher; //!< Model matcher + CPluginManagerSimulator *m_plugins = nullptr; //!< plugin manager + BlackMisc::CRegularThread m_listenersThread; //!< waiting for plugin + CWeatherManager m_weatherManager { this }; //!< weather management + CAircraftMatcher m_aircraftMatcher { this }; //!< model matcher QMap m_matchingMessages; - bool m_initallyAddAircrafts = false; + bool m_initallyAddAircraft = false; bool m_enableMatchingMessages = true; bool m_isWeatherActivated = false; diff --git a/src/blackgui/components/modelmatchercomponent.h b/src/blackgui/components/modelmatchercomponent.h index 74ea98039..c780f218c 100644 --- a/src/blackgui/components/modelmatchercomponent.h +++ b/src/blackgui/components/modelmatchercomponent.h @@ -77,7 +77,7 @@ namespace BlackGui BlackMisc::Simulation::CAircraftModel defaultModel() const; QScopedPointer ui; - BlackCore::CAircraftMatcher m_matcher { BlackCore::CAircraftMatcher::All, this }; + BlackCore::CAircraftMatcher m_matcher { this }; //!< used matcher }; } // ns } // ns