flightsimulatorplatfrom/pilotclient
1
0
Fork 0
mirror of https://github.com/swift-project/pilotclient.git synced 2026-03-31 21:15:33 +08:00
Code Issues Packages Projects Releases Wiki Activity

Ref T171, no assert when there is function (review of T171)

Browse Source
This commit is contained in:
Klaus Basan
2017-10-17 02:53:52 +02:00
parent bf390f592a
commit 56fb27b73a
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/blackcore/airspacemonitor.cpp
View File

@@ -217,15 +217,15 @@ namespace BlackCore
// bind does not allow to define connection type, so we use receiver as workaround
const QMetaObject::Connection uc; // unconnected
const QMetaObject::Connection c1 = situationFunction ? connect(this, &CAirspaceMonitor::addedAircraftSituation, receiver, situationFunction) : uc;
Q_ASSERT_X(c1, Q_FUNC_INFO, "connect failed");
Q_ASSERT_X(c1 || !situationFunction, Q_FUNC_INFO, "connect failed");
const QMetaObject::Connection c2 = partsFunction ? connect(this, &CAirspaceMonitor::addedAircraftParts, receiver, partsFunction) : uc;
Q_ASSERT_X(c2, Q_FUNC_INFO, "connect failed");
Q_ASSERT_X(c2 || !partsFunction, Q_FUNC_INFO, "connect failed");
const QMetaObject::Connection c3 = removedAircraftFunction ? connect(this, &CAirspaceMonitor::removedAircraft, receiver, removedAircraftFunction) : uc;
Q_ASSERT_X(c3, Q_FUNC_INFO, "connect failed");
Q_ASSERT_X(c3 || !removedAircraftFunction, Q_FUNC_INFO, "connect failed");
// trick is to use the Qt::QueuedConnection signal here
// analyzer (own thread) -> airspaceAircraftSnapshot -> AirspaceMonitor -> airspaceAircraftSnapshot queued in main thread
const QMetaObject::Connection c4 = aircraftSnapshotFunction ? connect(m_analyzer, &CAirspaceAnalyzer::airspaceAircraftSnapshot, receiver, aircraftSnapshotFunction, Qt::QueuedConnection) : uc;
Q_ASSERT_X(c4, Q_FUNC_INFO, "connect failed");
Q_ASSERT_X(c4 || !aircraftSnapshotFunction, Q_FUNC_INFO, "connect failed");
return QList<QMetaObject::Connection>({ c1, c2, c3, c4});
}