-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
*: Fix embedded mutexes leaking methods #7833
Conversation
@s111 is there a linter/tool for detecting this? |
@heyitsanthony I've written a quick extension of golint for my own use: https://github.com/s111/lint. |
@s111 any way that patch can get into golint upstream or some other linter? If it's a style problem worth fixing I'd like to be able to catch it as part of CI... |
Codecov Report
@@ Coverage Diff @@
## master #7833 +/- ##
=========================================
Coverage ? 75.91%
=========================================
Files ? 332
Lines ? 26177
Branches ? 0
=========================================
Hits ? 19871
Misses ? 4887
Partials ? 1419
Continue to review full report at Codecov.
|
@sebschrader If your patch s111/lint@d196a16 goes in upstream, it would be great. lgtm. Thanks! |
Hey @gyuho, you seem to have erroneously mentioned me here. |
@sebschrader Apology! I meant to mention @s111 Thanks! |
44ca396
to
4301f49
Compare
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This pull request cleans up all exported structs that were leaking the Lock/Unlock methods from an embedded mutex.