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

singleuse: detect singleuse functions in CI #11932

Closed
wants to merge 6 commits into from
Closed

Conversation

bagder
Copy link
Member

@bagder bagder commented Sep 25, 2023

Functions that are non-static but not used outside its file should be made static.

@github-actions github-actions bot added the CI Continuous Integration label Sep 25, 2023
This should reduce false-positive to almost zero. Checks for presence in
unit tests if --unit is specified, which is intended for debug builds
where unit testing is enabled.
Use --unit for configure --enable-debug builds
Previously put there for use with the intel compiler
@bagder bagder closed this in d27576b Sep 25, 2023
bagder added a commit that referenced this pull request Sep 25, 2023
Use --unit for configure --enable-debug builds

Closes #11932
bagder added a commit that referenced this pull request Sep 25, 2023
Previously put there for use with the intel compiler

Closes #11932
@bagder bagder deleted the bagder/singleuse branch September 25, 2023 15:07
vszakats added a commit to vszakats/curl that referenced this pull request May 14, 2024
vszakats added a commit to vszakats/curl that referenced this pull request May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build CI Continuous Integration script
Development

Successfully merging this pull request may close these issues.

None yet

1 participant