aboutsummaryrefslogtreecommitdiffstats
path: root/testdata/signature.go
diff options
context:
space:
mode:
authorTeddy Wing2023-05-21 21:39:48 +0200
committerTeddy Wing2023-05-21 21:39:48 +0200
commit8efe2cc2f9495b60b5a27f818af7d666651aee4d (patch)
tree83fb472f7c31ddff74dc8920b3ced1b6120380f3 /testdata/signature.go
parentaf42c108f8f29fd6290940cc9fd6c5819d25ad74 (diff)
downloadgodefererr-8efe2cc2f9495b60b5a27f818af7d666651aee4d.tar.bz2
Report function signature declaration using variable name other than err
I had previously hard-coded the variable name "err" as the recommended name declaration in the function signature for simplicity. Now, use the name assigned in the `defer` instead, which is more correct.
Diffstat (limited to 'testdata/signature.go')
-rw-r--r--testdata/signature.go15
1 files changed, 15 insertions, 0 deletions
diff --git a/testdata/signature.go b/testdata/signature.go
index 73203ee..b668558 100644
--- a/testdata/signature.go
+++ b/testdata/signature.go
@@ -45,6 +45,21 @@ func returnedErrorMustMatchDeferErrorName() (err error) {
return err2 // want "does not return 'err'"
}
+func deferUsesUnconventionalErrName() error { // want "return signature should be '\\(anErr error\\)'"
+ var anErr error
+
+ anErr = nil
+ if anErr != nil {
+ return anErr
+ }
+
+ defer func() {
+ anErr = errors.New("defer error")
+ }()
+
+ return anErr
+}
+
func good() (err error) {
err = nil
if err != nil {