From 67b36bbfcf6ad842c75bf00d3cfc30ea09d6a80d Mon Sep 17 00:00:00 2001 From: Mathew Sutcliffe Date: Thu, 11 Aug 2016 16:19:44 +0100 Subject: [PATCH] refs #721 Trying the unaltered name first is slower because in nearly all cases, it just leads to comparing the names twice. --- src/blackmisc/aviation/airlineicaocode.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/blackmisc/aviation/airlineicaocode.cpp b/src/blackmisc/aviation/airlineicaocode.cpp index 0def1ee66..25a5d032d 100644 --- a/src/blackmisc/aviation/airlineicaocode.cpp +++ b/src/blackmisc/aviation/airlineicaocode.cpp @@ -135,9 +135,6 @@ namespace BlackMisc bool CAirlineIcaoCode::isContainedInSimplifiedName(const QString &candidate) const { if (candidate.isEmpty() || !this->hasName()) { return false; } - // try unaltered name first (should be faster) - if (this->getName().contains(candidate, Qt::CaseInsensitive)) { return true; } - auto simplifiedName = makeRange(getName().begin(), getName().end()).findBy([](QChar c) { return c.isLetter(); }); auto it = std::search(simplifiedName.begin(), simplifiedName.end(), candidate.begin(), candidate.end(), [](QChar a, QChar b) { return a.toUpper() == b.toUpper(); }); return it != simplifiedName.end();