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.

1 files changed, 0 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
 				}