Skip to content

Mark C++ mutex helper functions as unused#7568

Merged
da-woods merged 1 commit intocython:masterfrom
da-woods:cpp-mutex-unused
Mar 18, 2026
Merged

Mark C++ mutex helper functions as unused#7568
da-woods merged 1 commit intocython:masterfrom
da-woods:cpp-mutex-unused

Conversation

@da-woods
Copy link
Copy Markdown
Contributor

@da-woods da-woods commented Mar 14, 2026

Also unmark a few of them as static - they're in an anonymous namespace which is already basically equivalent.

Fixes #7560

Also unmark a few of them as static - they're in an anonymous
namespace which is already basically equivalent.

Fixes cython#7562
@ngoldbaum
Copy link
Copy Markdown
Contributor

Thanks!

@da-woods da-woods merged commit 8efca86 into cython:master Mar 18, 2026
88 checks passed
da-woods added a commit that referenced this pull request Mar 18, 2026
Also unmark a few of them as static - they're in an anonymous namespace
which is already basically equivalent.

Fixes #7560
@da-woods
Copy link
Copy Markdown
Contributor Author

3.2.x commit 139a39b

@da-woods da-woods deleted the cpp-mutex-unused branch March 18, 2026 07:36
@da-woods da-woods added this to the 3.2.5 milestone Mar 18, 2026
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.

[BUG] Unused function warnings using py_safe_call_object_once

2 participants