Skip to content

Commit 9e38abc

Browse files
committed
Use argn for empty or ignored names, underscore prefix for conflict
1 parent 75cb29b commit 9e38abc

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

mockgen/mockgen.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -781,15 +781,19 @@ func (g *generator) getArgNames(m *model.Method, in bool) []string {
781781
for i, p := range params {
782782
name := p.Name
783783

784-
if name == "" || name == "_" || g.nameExistsAsPackage(name) {
784+
if name == "" || name == "_" {
785+
name = fmt.Sprintf("arg%d", i)
786+
} else if g.nameExistsAsPackage(name) {
785787
name = "_" + name
786788
}
787789
argNames[i] = name
788790
}
789791
if m.Variadic != nil && in {
790792
name := m.Variadic.Name
791793

792-
if name == "" || g.nameExistsAsPackage(name) {
794+
if name == "" {
795+
name = fmt.Sprintf("arg%d", len(params))
796+
} else if g.nameExistsAsPackage(name) {
793797
name = "_" + name
794798
}
795799
argNames = append(argNames, name)

0 commit comments

Comments
 (0)