diff options
| -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; } |