From 4e250b79b2914840958e4b1a0591120bbee425bb Mon Sep 17 00:00:00 2001 From: Max Neverov Date: Sat, 14 Feb 2026 07:07:10 +0100 Subject: [PATCH 1/2] Bump go, x/tools. --- go.mod | 8 ++++---- go.sum | 12 ++++++------ 2 files changed, 10 insertions(+), 10 deletions(-) diff --git a/go.mod b/go.mod index 8b04450..b6f11ed 100644 --- a/go.mod +++ b/go.mod @@ -1,10 +1,10 @@ module github.com/mneverov/protectedby -go 1.24.5 +go 1.26 -require golang.org/x/tools v0.35.0 +require golang.org/x/tools v0.42.0 require ( - golang.org/x/mod v0.26.0 // indirect - golang.org/x/sync v0.16.0 // indirect + golang.org/x/mod v0.33.0 // indirect + golang.org/x/sync v0.19.0 // indirect ) diff --git a/go.sum b/go.sum index 58f0d0b..5e2a226 100644 --- a/go.sum +++ b/go.sum @@ -1,8 +1,8 @@ github.com/google/go-cmp v0.6.0 h1:ofyhxvXcZhMsU5ulbFiLKl/XBFqE1GSq7atu8tAmTRI= github.com/google/go-cmp v0.6.0/go.mod h1:17dUlkBOakJ0+DkrSSNjCkIjxS6bF9zb3elmeNGIjoY= -golang.org/x/mod v0.26.0 h1:EGMPT//Ezu+ylkCijjPc+f4Aih7sZvaAr+O3EHBxvZg= -golang.org/x/mod v0.26.0/go.mod h1:/j6NAhSk8iQ723BGAUyoAcn7SlD7s15Dp9Nd/SfeaFQ= -golang.org/x/sync v0.16.0 h1:ycBJEhp9p4vXvUZNszeOq0kGTPghopOL8q0fq3vstxw= -golang.org/x/sync v0.16.0/go.mod h1:1dzgHSNfp02xaA81J2MS99Qcpr2w7fw1gpm99rleRqA= -golang.org/x/tools v0.35.0 h1:mBffYraMEf7aa0sB+NuKnuCy8qI/9Bughn8dC2Gu5r0= -golang.org/x/tools v0.35.0/go.mod h1:NKdj5HkL/73byiZSJjqJgKn3ep7KjFkBOkR/Hps3VPw= +golang.org/x/mod v0.33.0 h1:tHFzIWbBifEmbwtGz65eaWyGiGZatSrT9prnU8DbVL8= +golang.org/x/mod v0.33.0/go.mod h1:swjeQEj+6r7fODbD2cqrnje9PnziFuw4bmLbBZFrQ5w= +golang.org/x/sync v0.19.0 h1:vV+1eWNmZ5geRlYjzm2adRgW2/mcpevXNg50YZtPCE4= +golang.org/x/sync v0.19.0/go.mod h1:9KTHXmSnoGruLpwFjVSX0lNNA75CykiMECbovNTZqGI= +golang.org/x/tools v0.42.0 h1:uNgphsn75Tdz5Ji2q36v/nsFSfR/9BRFvqhGBaJGd5k= +golang.org/x/tools v0.42.0/go.mod h1:Ma6lCIwGZvHK6XtgbswSoWroEkhugApmsXyrUmBhfr0= From 2b6d6c98180a536f63dfe8f6043b4cef2c59157e Mon Sep 17 00:00:00 2001 From: Max Neverov Date: Sat, 14 Feb 2026 07:09:12 +0100 Subject: [PATCH 2/2] Refactoring: simplify return. --- protectedby/analyzer.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/protectedby/analyzer.go b/protectedby/analyzer.go index 750d669..892bf63 100644 --- a/protectedby/analyzer.go +++ b/protectedby/analyzer.go @@ -254,11 +254,7 @@ func checkLocksUsed(pass *analysis.Pass, m map[string]*protectedData) []*analysi // Access to a protected field can be deferred. Skip node if this is a defer statement // that is not the same where the field is accessed. if deferStmt, ok := curr.(*ast.DeferStmt); ok { - if deferStmt == u.deferStmt { - return true - } else { - return false - } + return deferStmt == u.deferStmt } cexpr, ok := curr.(*ast.CallExpr)