-
Notifications
You must be signed in to change notification settings - Fork 1
/
unuseparam.go
86 lines (72 loc) · 1.75 KB
/
unuseparam.go
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
package unuseparam
import (
"fmt"
"go/ast"
"go/types"
"golang.org/x/tools/go/analysis"
"golang.org/x/tools/go/analysis/passes/inspect"
"golang.org/x/tools/go/ast/inspector"
)
const doc = "unuseparam finds a unused parameter but its name is not _"
var Analyzer = &analysis.Analyzer{
Name: "unuseparam",
Doc: doc,
Run: run,
Requires: []*analysis.Analyzer{
inspect.Analyzer,
},
}
func run(pass *analysis.Pass) (interface{}, error) {
inspect := pass.ResultOf[inspect.Analyzer].(*inspector.Inspector)
nodeFilter := []ast.Node{
(*ast.FuncDecl)(nil),
(*ast.FuncLit)(nil),
}
ps := make(map[types.Object]*ast.Ident)
inspect.Preorder(nodeFilter, func(n ast.Node) {
switch n := n.(type) {
case *ast.FuncDecl:
params(pass, n.Type, ps)
case *ast.FuncLit:
params(pass, n.Type, ps)
}
})
unused(pass, ps)
return nil, nil
}
func params(pass *analysis.Pass, typ *ast.FuncType, ps map[types.Object]*ast.Ident) {
for _, field := range typ.Params.List {
for _, n := range field.Names {
obj := pass.TypesInfo.Defs[n]
if obj != nil && n.Name != "_" {
ps[obj] = n
}
}
}
}
func unused(pass *analysis.Pass, ps map[types.Object]*ast.Ident) {
for _, obj := range pass.TypesInfo.Uses {
if ps[obj] != nil {
ps[obj] = nil
}
}
for _, id := range ps {
if id == nil {
continue
}
fix := analysis.SuggestedFix{
Message: fmt.Sprintf(`%s is unused parameter, it can be changed to "_"`, id.Name),
TextEdits: []analysis.TextEdit{{
Pos: id.Pos(),
End: id.End(),
NewText: []byte("_"),
}},
}
pass.Report(analysis.Diagnostic{
Pos: id.Pos(),
End: id.End(),
Message: fmt.Sprintf("%s is unused parameter", id.Name),
SuggestedFixes: []analysis.SuggestedFix{fix},
})
}
}