qdoc: Stop mistreating the QtQuick QML module

Node::isQtQuickNode() returned true for any QML node under
a module named "QtQuick". The reason why modules needed
special handling based on their name is lost to history.

Get rid of isQtQuickNode() and the code that uses it.

Change-Id: Ia0334b9a16a07bedecd51f9428733f840f8bd913
Reviewed-by: Paul Wicking <paul.wicking@qt.io>

1 files changed, 6 insertions, 11 deletions

diff --git a/src/qdoc/qdoc/src/qdoc/generator.cpp b/src/qdoc/qdoc/src/qdoc/generator.cpp
index fa30e93c8..9ba4de976 100644
--- a/src/qdoc/qdoc/src/qdoc/generator.cpp
+++ b/src/qdoc/qdoc/src/qdoc/generator.cpp
@@ -168,20 +168,15 @@ int Generator::appendSortedQmlNames(Text &text, const Node *base, const NodeList
     QMap<QString, Text> classMap;
 
     for (const auto sub : subs) {
-        Text text;
-        if (!base->isQtQuickNode() || !sub->isQtQuickNode()
-            || (base->logicalModuleName() == sub->logicalModuleName())) {
-            appendFullName(text, sub, base);
-            classMap[text.toString().toLower()] = text;
-        }
+        Text full_name;
+        appendFullName(full_name, sub, base);
+        classMap[full_name.toString().toLower()] = full_name;
     }
 
     int index = 0;
-    const QStringList names = classMap.keys();
-    for (const auto &name : names) {
-        text << classMap[name];
-        text << Utilities::comma(index++, names.size());
-    }
+    const auto &names = classMap.keys();
+    for (const auto &name : names)
+        text << classMap[name] << Utilities::comma(index++, names.size());
     return index;
 }