diff options
author | Dimitri van Heesch <doxygen@gmail.com> | 2019-08-31 11:05:46 (GMT) |
---|---|---|
committer | Dimitri van Heesch <doxygen@gmail.com> | 2019-08-31 11:05:46 (GMT) |
commit | 361f3e10a86370a568733a7e4d862b180cf45fd9 (patch) | |
tree | c5364138ce0ae99ad40df61aed9f2653e51e79cc /src | |
parent | 8a641dad0a1250d1f4646e4d6b1803c350a0ef71 (diff) | |
download | Doxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.zip Doxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.tar.gz Doxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.tar.bz2 |
Made check for void return more strict
Diffstat (limited to 'src')
-rw-r--r-- | src/memberdef.cpp | 28 |
1 files changed, 14 insertions, 14 deletions
diff --git a/src/memberdef.cpp b/src/memberdef.cpp index d79b37e..4f44baa 100644 --- 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; } |