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?
to your account
This is a common mistake pattern
x = atomic.AddUint64(&x, 1)
Should be easy to detect with vet. A little obscure perhaps but easy.
The text was updated successfully, but these errors were encountered:
Another common misuse is Add(&x, 0) it should be replaced with Load(&x).
Labels changed: added go1.1.
Labels changed: added size-m.
Labels changed: added suggested.
Comment 5 by divoxx:
My attempt to add this feature to go vet: https://golang.org/cl/7097048
This is my first contribution to the project, let me know if I'm not doing anything
This issue was closed by revision 6073176.
Status changed to Fixed.
Successfully merging a pull request may close this issue.