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

checksrc.bat: remove explicit SNPRINTF bypass #16032

Closed
wants to merge 1 commit into from

Conversation

jay
Copy link
Member

@jay jay commented Jan 17, 2025

  • Remove the command line argument that accepts SNPRINTF violations in docs/examples.

This is a follow-up to c445b74 which introduced a different management of banned functions and removed the SNPRINTF rule in favor of banning snprintf in lib. There's no longer a SNPRINTF warning to suppress.

Closes #xxxxx

- Remove the command line argument that accepts SNPRINTF violations in
  docs/examples.

This is a follow-up to c445b74 which introduced a different management
of banned functions and removed the SNPRINTF rule in favor of banning
snprintf in lib. There's no longer a SNPRINTF warning to suppress.

Closes #xxxxx
@jay jay added the Windows Windows-specific label Jan 17, 2025
@github-actions github-actions bot added the build label Jan 17, 2025
@jay jay closed this in 5699827 Jan 19, 2025
@jay jay deleted the checksrc_windows_snprintf branch January 19, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Windows Windows-specific
Development

Successfully merging this pull request may close these issues.

2 participants