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

lib: mutexes don't need no shadow #17937

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eqvinox
Copy link
Contributor

@eqvinox eqvinox commented Jan 28, 2025

The _once loop variable will result in a -Wshadow warning when that is turned on. Use __COUNTER__ to give these variables distinct names, like is already done with _mtx_.

(and because I touched it, clang-format wants it reformatted... ohwell.)

The `_once` loop variable will result in a `-Wshadow` warning when that
is turned on.  Use `__COUNTER__` to give these variables distinct names,
like is already done with `_mtx_`.

(and because I touched it, clang-format wants it reformatted... ohwell.)

Signed-off-by: David Lamparter <equinox@opensourcerouting.org>
@eqvinox
Copy link
Contributor Author

eqvinox commented Jan 28, 2025

(CI build VM hung after build for some reason [after completing the build, even], killed it and kicked it to retry)

@donaldsharp
Copy link
Member

ci:rerun

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants