tree 3ee25f41ec62e6926ae6b3aeb527709b4fbcb918
parent 77adbdfd2c36573cc014b7203643186f6e4bbd78
author smasher164 <aindurti@gmail.com> 1577766734 -0500
committer Robert Griesemer <gri@golang.org> 1582146794 +0000

go/analysis: add pass to check string(int) conversions

Issue golang/go#3939 proposes to remove string(int) from the language on the
grounds that it produces non-obvious results that can't be statically checked.
An intermediate step is to have go vet check for these sorts of conversions.
This change adds an analysis pass to check for string(int) conversions. It
suggests a fix to replace string(int) with string(rune(int)).

Updates golang/go#32479.

Change-Id: Ifafd6d74f9bd4a903ce6b29ac3a3c7a15f8a1ad9
Reviewed-on: https://go-review.googlesource.com/c/tools/+/212919
Reviewed-by: Robert Griesemer <gri@golang.org>
Reviewed-by: Alan Donovan <adonovan@google.com>
Run-TryBot: Robert Griesemer <gri@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>