summaryrefslogtreecommitdiffstats
path: root/src/memberdef.cpp
diff options
context:
space:
mode:
authorDimitri van Heesch <doxygen@gmail.com>2019-08-31 11:05:46 (GMT)
committerDimitri van Heesch <doxygen@gmail.com>2019-08-31 11:05:46 (GMT)
commit361f3e10a86370a568733a7e4d862b180cf45fd9 (patch)
treec5364138ce0ae99ad40df61aed9f2653e51e79cc /src/memberdef.cpp
parent8a641dad0a1250d1f4646e4d6b1803c350a0ef71 (diff)
downloadDoxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.zip
Doxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.tar.gz
Doxygen-361f3e10a86370a568733a7e4d862b180cf45fd9.tar.bz2
Made check for void return more strict
Diffstat (limited to 'src/memberdef.cpp')
-rw-r--r--src/memberdef.cpp28
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;
}