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

fix: thread-sanitizer warnings in SandboxedTests #4131

Merged
merged 2 commits into from Nov 10, 2022

Conversation

ckerr
Copy link
Member

@ckerr ckerr commented Nov 10, 2022

fix: thread-sanitizer warnings in SandboxedTest::blockingTorrentVerify

fix: thread-sanitizer warnings in SandboxedTest::createTorrentAndWaitForVerifyDone()

fix: thread-sanitizer warnings in SandboxedTest::createTorrentAndWaitForVerifyDone()
@ckerr ckerr merged commit 2d2270c into main Nov 10, 2022
@ckerr ckerr deleted the fix/thread-santizer-warnings branch November 10, 2022 05:56
@GaryElshaw
Copy link
Contributor

GaryElshaw commented Nov 10, 2022

Charles, just about every file in libtransmission now has this warning during make, is that a foreseen change? @ckerr

/libtransmission/session.h:1006:40: warning: declaration requires an exit-time destructor [-Wexit-time-destructors] static inline std::recursive_mutex session_mutex;

@ckerr
Copy link
Member Author

ckerr commented Nov 10, 2022

@GaryElshaw I've got another small thread PR in the works; I'll fix it there. Thanks for the heads-up; I'm not seeing that warning here on my build 🤷‍♂️

@GaryElshaw
Copy link
Contributor

No worries!

@nevack
Copy link
Member

nevack commented Nov 10, 2022

@GaryElshaw I've got another small thread PR in the works; I'll fix it there. Thanks for the heads-up; I'm not seeing that warning here on my build man_shrugging

I also see it, maybe Xcode's AppleClang specific

@ckerr
Copy link
Member Author

ckerr commented Nov 26, 2022

Notes: Turned on more clang-tidy checks and fixed warnings.

ckerr added a commit that referenced this pull request Feb 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

3 participants