-
Notifications
You must be signed in to change notification settings - Fork 18.9k
cmd/vet: warn about string(int) type conversions #32479
Copy link
Copy link
Closed
Labels
FrozenDueToAgeNeedsFixThe path to resolution is known, but the work has not been done.The path to resolution is known, but the work has not been done.Proposal-Acceptedokay-after-beta1Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1release-blocker
Milestone
Metadata
Metadata
Assignees
Labels
FrozenDueToAgeNeedsFixThe path to resolution is known, but the work has not been done.The path to resolution is known, but the work has not been done.Proposal-Acceptedokay-after-beta1Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1Used by release team to mark a release-blocker issue as okay to resolve either before or after beta1release-blocker
#3939 proposes removing the explicit type conversion
string(i)whereihas an integer type other thanrune. That is a backward incompatible change, so we cannot make it until we can count on having modules everywhere, with the support modules provide for determining the version of the language to use to compile code.There is a step we can take today, which is to change
go vetto report conversions of integer types other thanrunetostring. If we think that removing this feature from the language is a good idea, then a vet check for it is a good idea, and it is one that we can implement today.