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

macOS: silence sprintf deprecation #509

Merged
merged 1 commit into from
Feb 19, 2023
Merged

macOS: silence sprintf deprecation #509

merged 1 commit into from
Feb 19, 2023

Conversation

Youw
Copy link
Member

@Youw Youw commented Feb 19, 2023

I've started getting this recently

@Youw Youw merged commit eecbe74 into master Feb 19, 2023
@Youw Youw deleted the macos-warning branch February 19, 2023 15:38
@sezero
Copy link
Contributor

sezero commented Feb 22, 2023

The rationale is wrong: snprintf has always been available in macosx, since its initial 10.1 release

@Youw
Copy link
Member Author

Youw commented Feb 22, 2023

I have to admit - I didn't check the macOS-specific background of this. Only checked that it is not available in C99 standard.
If anyone provides a PR with a fix before I can get my hands to it - that would be great.

@sezero
Copy link
Contributor

sezero commented Feb 22, 2023

I have to admit - I didn't check the macOS-specific background of this. Only checked that it is not available in C99 standard. If anyone provides a PR with a fix before I can get my hands to it - that would be great.

Here you go: #511

Youw pushed a commit that referenced this pull request Feb 22, 2023
@mcuee mcuee added the macOS Related to macOS backend label Apr 5, 2023
sezero added a commit to sezero/SDL that referenced this pull request Apr 24, 2023
slouken pushed a commit to libsdl-org/SDL that referenced this pull request Apr 24, 2023
slouken pushed a commit to libsdl-org/SDL that referenced this pull request Apr 24, 2023
sezero added a commit to sezero/SDL that referenced this pull request May 17, 2023
lexi-the-cute pushed a commit to lexi-the-cute/SDL that referenced this pull request May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
macOS Related to macOS backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants