Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

simple: in Go 1.20, simplify *(*ArrayType)(slice) to ArrayType(slice) #1367

Open
ainar-g opened this issue Feb 14, 2023 · 2 comments
Open

simple: in Go 1.20, simplify *(*ArrayType)(slice) to ArrayType(slice) #1367

ainar-g opened this issue Feb 14, 2023 · 2 comments
Labels
needs-decision We have to decide if this check is feasible and desirable new-check

Comments

@ainar-g
Copy link
Contributor

ainar-g commented Feb 14, 2023

See golang/go#46505 and also https://go.dev/play/p/gEFLAGUrov0 for an example of the code.

@ainar-g ainar-g added the needs-triage Newly filed issue that needs triage label Feb 14, 2023
@dominikh dominikh added needs-decision We have to decide if this check is feasible and desirable new-check and removed needs-triage Newly filed issue that needs triage labels Feb 15, 2023
@dominikh
Copy link
Owner

That's an annoying one. It is barely an improvement and not really worth the churn. I wish gofmt -s would do this in the future, but that seems unlikely.

@ainar-g
Copy link
Contributor Author

ainar-g commented Feb 15, 2023

git grep -F -e '*(*'

has found most of these in my code at least, heh. Leaving it here in case people want a way to find this kind of code in the meantime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-decision We have to decide if this check is feasible and desirable new-check
Projects
None yet
Development

No branches or pull requests

2 participants