Made check for void return more strict

doxygen · Aug 31, 2019 · 361f3e1 · 361f3e1
1 parent 8a641da
commit 361f3e1
Showing 1 changed file with 14 additions and 14 deletions.
diff --git a/src/memberdef.cpp b/src/memberdef.cpp
@@ -4201,25 +4201,25 @@ void MemberDefImpl::detectUndocumentedParams(bool hasParamCommand,bool hasReturn
     m_impl->hasDocumentedReturnType = TRUE;
   }
   else if ( // see if return type is documented in a function w/o return type
-      hasReturnCommand &&
-      (
-       (returnType.find("void")!=-1 && returnType.find('*')==-1)      || // void return type
-       returnType.find("subroutine")!=-1 || // fortran subroutine
-       isConstructor()      || // a constructor
-       isDestructor()          // or destructor
-      )
-      )
+            hasReturnCommand &&
+            (
+              returnType=="void"   || // void return type
+              returnType.find("subroutine")!=-1 || // fortran subroutine
+              isConstructor()      || // a constructor
+              isDestructor()          // or destructor
+            )
+          )
   {
     warn_doc_error(getDefFileName(),getDefLine(),"documented empty return type of %s",
                           qualifiedName().data());
   }
   else if ( // see if return needs to documented 
-      m_impl->hasDocumentedReturnType ||
-      (returnType.find("void")!=-1 && returnType.find('*')==-1)      || // void return type
-      returnType.find("subroutine")!=-1 || // fortran subroutine
-      isConstructor() || // a constructor
-      isDestructor()     // or destructor
-      )
+            m_impl->hasDocumentedReturnType ||
+           returnType=="void" || // void return type
+           returnType.find("subroutine")!=-1 || // fortran subroutine
+           isConstructor()    || // a constructor
+           isDestructor()        // or destructor
+          )
   {
     m_impl->hasDocumentedReturnType = TRUE;
   }