QDoc: Extract method from DocParser::insertTarget

`DocParser::insertTarget` has a conditional check at the start of the
method, that, if satisfied, causes QDoc to generate a warning. The rest
of the method is contained in the else block that follows.

This patch extracts a method, `DocParser::warnAboutPreexistingTarget`,
that is responsible for issuing that warning, and modifies
`DocParser::insertTarget` to return early if the condition is
satisfied. This allows further refactoring of the latter method.
The new method is fully documented. As this new method impacts the
documentation for QDoc warnings, that is also modified in this patch.

Task-number: QTBUG-113126
Change-Id: Iba54e67862555fba71ec774261e0e0fbea638238
Reviewed-by: Topi Reiniö <topi.reinio@qt.io>

1 files changed, 15 insertions, 2 deletions

diff --git a/src/qdoc/qdoc/docparser.cpp b/src/qdoc/qdoc/docparser.cpp
index cb6b7788e..ca17fefba 100644
--- a/src/qdoc/qdoc/docparser.cpp
+++ b/src/qdoc/qdoc/docparser.cpp
@@ -1293,11 +1293,24 @@ QString DocParser::detailsUnknownCommand(const QSet<QString> &metaCommandSet, co
     return QStringLiteral("Maybe you meant '\\%1'?").arg(best);
 }
 
+/*!
+    \internal
+
+    Issues a warning about the duplicate definition of a target or keyword in
+    at \a location. \a duplicateDefinition is the target being processed; the
+    already registered definition is \a previousDefinition.
+ */
+static void warnAboutPreexistingTarget(const Location &location, const QString &duplicateDefinition, const QString &previousDefinition)
+{
+    location.warning(
+            QStringLiteral("Duplicate target name '%1'. The previous occurrence is here: %2")
+                    .arg(duplicateDefinition, previousDefinition));
+}
+
 void DocParser::insertTarget(const QString &target, bool keyword)
 {
     if (m_targetMap.contains(target)) {
-        location().warning(QStringLiteral("Duplicate target name '%1'").arg(target));
-        m_targetMap[target].warning(QStringLiteral("(The previous occurrence is here)"));
+        return warnAboutPreexistingTarget(location(), target, m_targetMap[target].toString());
     } else {
         m_targetMap.insert(target, location());
         m_private->constructExtra();