typeparams: preserve AST position in copytermlist
When generating SelectorExpr using ast.NewIdent, the generated
AST nodes get token.NoPos. During code formatting with go/printer
this position loss caused line-number mismatches between the
final parameter and the closing parenthesis, triggering spurious
trailing commas.
For example, without this change, the copytermlist script produced:
func (x *term) includes(t types.Type,) bool {
This is an issue preexisted before CL 783560.
With this change, the script produces:
func (x *term) includes(t types.Type) bool {
Change-Id: I86a11d9fd7e650a2a00252d2d84861e7a007f21c
Reviewed-on: https://go-review.googlesource.com/c/exp/+/784560
Reviewed-by: Alex Putman <aputman@golang.org>
Auto-Submit: Hyang-Ah Hana Kim <hyangah@gmail.com>
LUCI-TryBot-Result: golang-scoped@luci-project-accounts.iam.gserviceaccount.com <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This subrepository holds experimental and deprecated (in the old directory) packages.
The idea for this subrepository originated as the pkg/exp directory of the main repository, but its presence there made it unavailable to users of the binary downloads of the Go installation. The subrepository has therefore been created to make it possible to go get these packages.
Warning: Packages here are experimental and unreliable. Some may one day be promoted to the main repository or other subrepository, or they may be modified arbitrarily or even disappear altogether.
In short, code in this subrepository is not subject to the Go 1 compatibility promise. (No subrepo is, but the promise is even more likely to be violated by go.exp than the others.)
Caveat emptor.