checkFunctions: Keep `continue` in return value loop

Looks like the return value type is always an `*ast.Ident`. Use
`continue` instead of `return true` to be more tolerant of unexpected
values.

2 files changed, 4 insertions, 3 deletions

diff --git a/defererr.go b/defererr.go
index 126f824..96c5569 100644
--- a/defererr.go
+++ b/defererr.go
@@ -81,9 +81,6 @@ func checkFunctions(pass *analysis.Pass, node ast.Node) {
 			for i, returnVal := range funcDecl.Type.Results.List {
 				returnIdent, ok := returnVal.Type.(*ast.Ident)
 				if !ok {
-					// TODO: Do we need to `return true` here? Does this check
-					// mean that variables are not declared in the return
-					// signature?
 					continue
 				}
 
diff --git a/testdata/signature.go b/testdata/signature.go
index 2606c22..0da2bb0 100644
--- a/testdata/signature.go
+++ b/testdata/signature.go
@@ -102,3 +102,7 @@ func good() (err error) {
 
 	return err
 }
+
+func noErrorInReturn() string {
+	return "test"
+}