1 files changed, 12 insertions, 12 deletions
diff --git a/src/tools/uic/cpp/cppwriteincludes.cpp b/src/tools/uic/cpp/cppwriteincludes.cpp
index a99d6adf07..dcbe400224 100644
--- a/src/tools/uic/cpp/cppwriteincludes.cpp
+++ b/src/tools/uic/cpp/cppwriteincludes.cpp
@@ -114,8 +114,9 @@ void WriteIncludes::acceptUI(DomUI *node)
 
     TreeWalker::acceptUI(node);
 
-    if (!m_uic->option().includeFile.isEmpty())
-        m_globalIncludes.insert(m_uic->option().includeFile, true);
+    const auto includeFile = m_uic->option().includeFile;
+    if (!includeFile.isEmpty())
+        m_globalIncludes.insert(includeFile);
 
     writeHeaders(m_globalIncludes, true);
     writeHeaders(m_localIncludes, false);
@@ -272,10 +273,11 @@ void WriteIncludes::insertInclude(const QString &header, bool global)
         fprintf(stderr, "%s %s %d\n", Q_FUNC_INFO, qPrintable(header), global);
 
     OrderedSet &includes = global ?  m_globalIncludes : m_localIncludes;
-    if (includes.contains(header))
+    // Insert (if not already done).
+    const bool isNewHeader = includes.insert(header).second;
+    if (!isNewHeader)
         return;
-    // Insert. Also remember base name for quick check of suspicious custom plugins
-    includes.insert(header, false);
+    // Also remember base name for quick check of suspicious custom plugins
     const QString lowerBaseName = QFileInfo(header).completeBaseName ().toLower();
     m_includeBaseNames.insert(lowerBaseName);
 }
@@ -286,13 +288,11 @@ void WriteIncludes::writeHeaders(const OrderedSet &headers, bool global)
     const QChar closingQuote = global ? QLatin1Char('>') : QLatin1Char('"');
 
     // Check for the old headers 'qslider.h' and replace by 'QtGui/QSlider'
-    const OrderedSet::const_iterator cend = headers.constEnd();
-    for (OrderedSet::const_iterator sit = headers.constBegin(); sit != cend; ++sit) {
-        const StringMap::const_iterator hit = m_oldHeaderToNewHeader.constFind(sit.key());
-        const bool mapped =  hit != m_oldHeaderToNewHeader.constEnd();
-        const  QString header =  mapped ? hit.value() : sit.key();
-        if (!QStringRef(&header).trimmed().isEmpty())
-            m_output << "#include " << openingQuote << header << closingQuote << QLatin1Char('\n');
+    for (const QString &header : headers) {
+        const QString value = m_oldHeaderToNewHeader.value(header, header);
+        const auto trimmed = QStringRef(&value).trimmed();
+        if (!trimmed.isEmpty())
+            m_output << "#include " << openingQuote << trimmed << closingQuote << QLatin1Char('\n');
     }
 }