Ref T171, Ref T136, removed the remote provider's "remove aircraft part", as it is not used:

* remove is handled by context call logicallyRemoveRemoteAircraft
* so far no need to use the remote provider function
* can be used again whenever needed
This commit is contained in:
Klaus Basan
2017-10-12 00:42:50 +02:00
parent bef47bf8ed
commit 418d0aa628
4 changed files with 3 additions and 17 deletions

View File

@@ -539,11 +539,6 @@ namespace BlackCore
Q_UNUSED(parts);
}
void CSimulatorCommon::onRemoteProviderRemovedAircraft(const CCallsign &callsign)
{
Q_UNUSED(callsign);
}
void CSimulatorCommon::reset()
{
m_statsUpdateAircraftCountMs = 0;
@@ -596,8 +591,8 @@ namespace BlackCore
void CSimulatorCommon::rapOnRemoteProviderRemovedAircraft(const CCallsign &callsign)
{
if (!this->isConnected()) return;
this->onRemoteProviderRemovedAircraft(callsign);
Q_UNUSED(callsign);
// currently not used, the calls are handled by context call logicallyRemoveRemoteAircraft
}
void CSimulatorCommon::callPhysicallyAddRemoteAircraft(const CSimulatedAircraft &remoteAircraft)

View File

@@ -152,9 +152,6 @@ namespace BlackCore
//! Provider added parts
virtual void onRemoteProviderAddedAircraftParts(const BlackMisc::Aviation::CCallsign &callsign, const BlackMisc::Aviation::CAircraftParts &parts);
//! Provider removed aircraft
virtual void onRemoteProviderRemovedAircraft(const BlackMisc::Aviation::CCallsign &callsign);
//! @}
//! New plugin info
@@ -240,6 +237,7 @@ namespace BlackCore
void rapOnRemoteProviderAddedAircraftParts(const BlackMisc::Aviation::CCallsign &callsign, const BlackMisc::Aviation::CAircraftParts &parts);
void rapOnRemoteProviderRemovedAircraft(const BlackMisc::Aviation::CCallsign &callsign);
// call with counters updated
void callPhysicallyAddRemoteAircraft(const BlackMisc::Simulation::CSimulatedAircraft &remoteAircraft);
void callPhysicallyRemoveRemoteAircraft(const BlackMisc::Aviation::CCallsign &remoteCallsign);

View File

@@ -582,12 +582,6 @@ namespace BlackSimPlugin
m_traffic->setPlaneTransponder(callsign.asString(), 2000, true, false);
}
void CSimulatorXPlane::onRemoteProviderRemovedAircraft(const CCallsign &callsign)
{
Q_UNUSED(callsign);
//! \todo call removeRemoteAircraft or just let removeRemoteAircraft handle it?
}
bool CSimulatorXPlane::physicallyRemoveRemoteAircraft(const BlackMisc::Aviation::CCallsign &callsign)
{
if (!isConnected()) { return false; } //! \fixme why is this method called when not connected?

View File

@@ -111,7 +111,6 @@ namespace BlackSimPlugin
//! @{
virtual void onRemoteProviderAddedAircraftSituation(const BlackMisc::Aviation::CAircraftSituation &situation) override;
virtual void onRemoteProviderAddedAircraftParts(const BlackMisc::Aviation::CCallsign &callsign, const BlackMisc::Aviation::CAircraftParts &parts) override;
virtual void onRemoteProviderRemovedAircraft(const BlackMisc::Aviation::CCallsign &callsign) override;
//! @}
//! \copydoc BlackCore::ISimulator::isConnected