tree 3304d867436406c08095ff016d035c0d2eb95149
parent d30366863522954d3dd1358b141ea026f28dfec5
author Daniel Martí <mvdan@mvdan.cc> 1652113689 +0100
committer Daniel Martí <mvdan@mvdan.cc> 1652121938 +0000

go/ast/astutil: make Apply follow TypeParams fields

It completely ignored the TypeParams fields in FuncType and TypeSpec.
I was confused why one of my tools, which looks at *ast.Ident nodes,
saw the identifiers where type parameters were being used,
but not the identifiers where they were being declared.

We use the ForFuncType and ForTypeSpec helpers, just like in the rest of
the package, for consistency and backwards compatibility with Go 1.17.

Add a regression test as well.

Change-Id: I00b2dfe5e04d92d63e6d5e91c6466598c865ed0b
Reviewed-on: https://go-review.googlesource.com/c/tools/+/405194
Run-TryBot: Daniel Martí <mvdan@mvdan.cc>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>
gopls-CI: kokoro <noreply+kokoro@google.com>