[AST] Remove redundant 'return false' from 'hasValueSemantics()'

theblixguy · theblixguy · commit 37b40203a797 · 2019-10-15T01:35:46.000+01:00
diff --git a/lib/AST/DeclContext.cpp b/lib/AST/DeclContext.cpp
@@ -1039,8 +1039,6 @@ bool DeclContext::hasValueSemantics() const {
   if (getExtendedProtocolDecl())
     return !isClassConstrainedProtocolExtension();
   return !getDeclaredInterfaceType()->hasReferenceSemantics();
-
-  return false;
 }
 
 bool DeclContext::isClassConstrainedProtocolExtension() const {

Original file line number	Diff line number	Diff line change
`@@ -1039,8 +1039,6 @@ bool DeclContext::hasValueSemantics() const {`
`1039`	`1039`	`if (getExtendedProtocolDecl())`
`1040`	`1040`	`return !isClassConstrainedProtocolExtension();`
`1041`	`1041`	`return !getDeclaredInterfaceType()->hasReferenceSemantics();`
`1042`		`-`
`1043`		`- return false;`
`1044`	`1042`	`}`
`1045`	`1043`
`1046`	`1044`	`bool DeclContext::isClassConstrainedProtocolExtension() const {`