Fix to avoid termination ("assert") noticed during testing

This commit is contained in:
Klaus Basan
2016-06-01 23:06:15 +02:00
parent 67512d9d1f
commit 2cc9f8d754

View File

@@ -407,12 +407,14 @@ namespace BlackCore
const QMap<QString, QString> CVatsimDataFileReader::clientPartsToMap(const QString &currentLine, const QStringList &clientSectionAttributes) const QMap<QString, QString> CVatsimDataFileReader::clientPartsToMap(const QString &currentLine, const QStringList &clientSectionAttributes)
{ {
QStringList clientParts = currentLine.split(':');
QMap<QString, QString> parts; QMap<QString, QString> parts;
if (currentLine.isEmpty()) { return parts; }
const QStringList clientParts = currentLine.split(':');
for (int i = 0; i < clientSectionAttributes.size(); i++) for (int i = 0; i < clientSectionAttributes.size(); i++)
{ {
BLACK_VERIFY_X(i < clientSectionAttributes.size(), Q_FUNC_INFO, "Wrong section attribute size"); BLACK_VERIFY_X(i < clientSectionAttributes.size(), Q_FUNC_INFO, "Wrong section attribute size");
BLACK_VERIFY_X(i < clientParts.size(), Q_FUNC_INFO, "Wrong parts size"); BLACK_VERIFY_X(i < clientParts.size(), Q_FUNC_INFO, "Wrong parts size");
if (i < clientSectionAttributes.size() || i < clientParts.size()) { continue; }
parts.insert(clientSectionAttributes.at(i).toLower(), clientParts.at(i)); parts.insert(clientSectionAttributes.at(i).toLower(), clientParts.at(i));
} }
return parts; return parts;