diff options
| -rw-r--r-- | defererr.go | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/defererr.go b/defererr.go index e7df464..837fa90 100644 --- a/defererr.go +++ b/defererr.go @@ -71,6 +71,15 @@ func checkFunctions(pass *analysis.Pass, node ast.Node) { } errorReturnField := funcDecl.Type.Results.List[errorReturnIndex] + // Idea: Set this to the end token.Pos of the first `defer` + // closure. Look for `return`s after that in funcDecl.Body and + // ensure they include the error variable. + firstDeferEndPos := -1 + + // Is it possible to generalise this to other types, and look for + // anything set in `defer` with the same type as a result in the + // return signature? + // TODO: Move to checkDeferFunc() // Should we make this an ast.Visitor to store some state for `return` checking? ast.Inspect( |