From bba31cef71694feba0248a37edc1c843a9405ce4 Mon Sep 17 00:00:00 2001
From: Paul Wicking <paul.wicking@qt.io>
Date: Tue, 27 Feb 2018 15:55:39 +0100
Subject: Doc: Fix qdoc/clang diagnostic warnings

Update templated function doc signatures to include correct return type.

Change-Id: Iaca1ae3ebccf09b328e0cb66c27787df5052f9a0
Reviewed-by: Paul Wicking <paul.wicking@qt.io>
Reviewed-by: Martin Smith <martin.smith@qt.io>
---
 .../api/qabstractxmlforwarditerator.cpp            | 22 +++++++++++-----------
 src/xmlpatterns/api/qabstractxmlpullprovider.cpp   | 12 ++++++------
 2 files changed, 17 insertions(+), 17 deletions(-)

diff --git a/src/xmlpatterns/api/qabstractxmlforwarditerator.cpp b/src/xmlpatterns/api/qabstractxmlforwarditerator.cpp
index 8309722..a2c4126 100644
--- a/src/xmlpatterns/api/qabstractxmlforwarditerator.cpp
+++ b/src/xmlpatterns/api/qabstractxmlforwarditerator.cpp
@@ -97,7 +97,7 @@
  */
 
 /*!
-  \fn T QAbstractXmlForwardIterator::current() const = 0;
+  \fn T QAbstractXmlForwardIterator::current() const = 0
 
   Returns the current item in the sequence. If this function is called
   before the first call to next(), a null object is returned. If the
@@ -105,7 +105,7 @@
  */
 
 /*!
-  \fn qint64 QAbstractXmlForwardIterator::position() const = 0;
+  \fn qint64 QAbstractXmlForwardIterator::position() const = 0
 
    Returns the current position in the sequence represented
    by \e this.
@@ -115,7 +115,7 @@
  */
 
 /*!
-  \fn bool qIsForwardIteratorEnd(const T &unit)
+  \fn template<typename T> bool qIsForwardIteratorEnd(const T &unit)
   \since 4.4
   \relates QAbstractXmlForwardIterator
 
@@ -150,7 +150,7 @@
  */
 
 /*!
-  \fn QAbstractXmlForwardIterator<T>::Ptr QAbstractXmlForwardIterator::toReversed();
+  \fn template<typename T> QAbstractXmlForwardIterator<T>::Ptr QAbstractXmlForwardIterator<T>::toReversed()
   \internal
 
   Returns a reverse iterator for the sequence.
@@ -164,7 +164,7 @@
  */
 
 /*!
-  \fn QList<T> QAbstractXmlForwardIterator<T>::toList();
+  \fn template<typename T> QList<T> QAbstractXmlForwardIterator<T>::toList()
   \internal
 
    Performs a copy of this QAbstractXmlForwardIterator(with copy()),
@@ -178,7 +178,7 @@
  */
 
 /*!
-  \fn T QAbstractXmlForwardIterator::last();
+  \fn T QAbstractXmlForwardIterator::last()
   \internal
 
    Returns the item at the end of this QAbstractXmlForwardIterator.
@@ -186,13 +186,13 @@
  */
 
 /*!
-  \fn T QAbstractXmlForwardIterator::isEmpty();
+  \fn T QAbstractXmlForwardIterator::isEmpty()
   \internal
   Returns true if the sequence is empty.
  */
 
 /*!
-  \fn qint64 QAbstractXmlForwardIterator::sizeHint() const;
+  \fn qint64 QAbstractXmlForwardIterator::sizeHint() const
   \internal
 
   Gives a hint to the size of the contained sequence. The hint is
@@ -202,7 +202,7 @@
  */
 
 /*!
-  \fn typename QAbstractXmlForwardIterator<T>::Ptr QAbstractXmlForwardIterator::copy() const;
+  \fn template <typename T> QAbstractXmlForwardIterator<T>::Ptr QAbstractXmlForwardIterator<T>::copy() const
   \internal
 
    Copies this QAbstractXmlForwardIterator and returns the copy.
@@ -232,7 +232,7 @@
  */
 
 /*!
-  \fn QPatternist::ListIteratorPlatform::ListIteratorPlatform(const ListType &list);
+  \fn QPatternist::ListIteratorPlatform::ListIteratorPlatform(const ListType &list)
 
   Constructs a ListIteratorPlatform that walks the given \a list.
  */
@@ -256,7 +256,7 @@
  */
 
 /*!
-   \fn QPatternist::makeListIterator(const QList<T> &qList)
+   \fn template<typename T> QAbstractXmlForwardIterator<T>::Ptr QPatternist::makeListIterator(const QList<T> &qList)
    \relates QPatternist::ListIterator
 
    An object generator for ListIterator.
diff --git a/src/xmlpatterns/api/qabstractxmlpullprovider.cpp b/src/xmlpatterns/api/qabstractxmlpullprovider.cpp
index 361c231..359bc81 100644
--- a/src/xmlpatterns/api/qabstractxmlpullprovider.cpp
+++ b/src/xmlpatterns/api/qabstractxmlpullprovider.cpp
@@ -112,14 +112,14 @@ AbstractXmlPullProvider::~AbstractXmlPullProvider()
 }
 
 /*!
-  \fn Event AbstractXmlPullProvider::next() = 0;
+  \fn QPatternist::AbstractXmlPullProvider::Event QPatternist::AbstractXmlPullProvider::next() = 0
   Advances this AbstractXmlPullProvider, and returns the new event.
 
   \sa current()
  */
 
 /*!
-  \fn Event AbstractXmlPullProvider::current() const = 0;
+  \fn QPatternist::AbstractXmlPullProvider::Event QPatternist::AbstractXmlPullProvider::current() const = 0
   Returns the event that next() returned the last time it was called. It doesn't
   alter this AbstractXmlPullProvider.
 
@@ -130,7 +130,7 @@ AbstractXmlPullProvider::~AbstractXmlPullProvider()
  */
 
 /*!
-  \fn QName AbstractXmlPullProvider::name() const = 0;
+  \fn QPatternist::AbstractXmlPullProvider::QName QPatternist::AbstractXmlPullProvider::name() const = 0
   If the current event is StartElement,
   EndElement, ProcessingInstruction, Attribute, or Namespace, the node's name is returned.
 
@@ -144,14 +144,14 @@ AbstractXmlPullProvider::~AbstractXmlPullProvider()
  */
 
 /*!
-  \fn QVariant AbstractXmlPullProvider::atomicValue() const = 0;
+  \fn QVariant QPatternist::AbstractXmlPullProvider::atomicValue() const = 0
 
   If current() event is AtomicValue, the atomic value is returned as a QVariant.
   In all other cases, this function returns a null QVariant.
  */
 
 /*!
- \fn QString AbstractXmlPullProvider::stringValue() const = 0;
+ \fn QString QPatternist::AbstractXmlPullProvider::stringValue() const = 0
 
   If current() is Text, the text node's value is returned.
 
@@ -165,7 +165,7 @@ AbstractXmlPullProvider::~AbstractXmlPullProvider()
   */
 
 /*!
- \fn QHash<QXmlName, QString> AbstractXmlPullProvider::attributes() = 0;
+ \fn QHash<QXmlName, QString> QPatternist::AbstractXmlPullProvider::attributes() = 0
 
   If the current() is Element, the attributes of the element are returned,
   an empty list of attributes otherwise.
-- 
cgit v1.2.1