From 361f3e10a86370a568733a7e4d862b180cf45fd9 Mon Sep 17 00:00:00 2001 From: Dimitri van Heesch Date: Sat, 31 Aug 2019 13:05:46 +0200 Subject: Made check for void return more strict --- src/memberdef.cpp | 28 ++++++++++++++-------------- 1 file 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; } -- cgit v0.12