qdoc: Ignore unnamed declarations when retrieving parameter names

The Clang visitor returns a CXCursor_ParmDecl also for parameter
declarations in the return type, as is the case with std::function
wrappers that take parameters. Handling these caused us to omit
some of the actual parameter names.

Skip declarations that do not provide a parameter name.

Fixes: QTBUG-67244
Change-Id: Id5d8d3302fe3580c10814735a0684b83b8d5ebb4
Reviewed-by: Paul Wicking <paul.wicking@qt.io>

2 files changed, 16 insertions, 0 deletions

diff --git a/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.cpp b/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.cpp
index 835b85336..eb5b79c67 100644
--- a/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.cpp
+++ b/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.cpp
@@ -125,6 +125,17 @@ void Test::someFunctionDefaultArg(int i, bool b = false)
     return;
 }
 
+/*!
+    \fn void Test::func(bool)
+    \internal
+*/
+
+/*!
+    \fn [funcPtr] void (*funcPtr(bool b, const char *s))(bool)
+
+    Returns a pointer to a function that takes a boolean. Uses \a b and \a s.
+*/
+
 // Documented below with an \fn command. Unnecessary but we support it, and it's used.
 int Test::someFunction(int v)
 {
diff --git a/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.h b/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.h
index b3dc3df2a..b950dfec9 100644
--- a/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.h
+++ b/tests/auto/qdoc/generatedoutput/testdata/testcpp/testcpp.h
@@ -56,6 +56,11 @@ using Specialized = Struct<int, T>;
     void obsoleteMember();
     void anotherObsoleteMember();
     void deprecatedMember();
+    void func(bool) {};
+    //! [funcPtr]
+    void (*funcPtr(bool b, const char *s))(bool) {
+        return func;
+    }
     inline void inlineFunction() {};
     virtual void virtualFun();