aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTeddy Wing2023-05-25 00:06:35 +0200
committerTeddy Wing2023-05-25 00:06:35 +0200
commit1fdc8a5d6296dbbcfe4b09e12e39484bbda82265 (patch)
treecc6b10d0b0088f57b6a2abc71a66efd4364327b4
parent8f7c7df68d955a2bba32f6be54119e1314410599 (diff)
downloadgodefererr-1fdc8a5d6296dbbcfe4b09e12e39484bbda82265.tar.bz2
Add note to check ident condition in return signature list loop
-rw-r--r--defererr.go3
1 files changed, 3 insertions, 0 deletions
diff --git a/defererr.go b/defererr.go
index 96c5569..126f824 100644
--- a/defererr.go
+++ b/defererr.go
@@ -81,6 +81,9 @@ 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
}