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

Add example code for dialog filter and callback #528

Merged
merged 3 commits into from
May 9, 2024

Conversation

Semphriss
Copy link
Contributor

Replaces libsdl-org/SDL#9729, the same description applies.

This is my first PR here, please let me know if I got something wrong!

Semphriss and others added 2 commits May 9, 2024 16:37
Co-authored-by: Sam Lantinga <slouken@libsdl.org>
@icculus icculus merged commit 5d8aa84 into libsdl-org:main May 9, 2024
1 check failed
@icculus
Copy link
Contributor

icculus commented May 9, 2024

This looks good. I'm adding this and then fixing the syntax error.

@icculus
Copy link
Contributor

icculus commented May 9, 2024

Okay, make sure I didn't screw this up in 3a5ce94.

(Outside of this specific example: should this callback send in the filter array too, so we don't have to have a global to figure out what the filter argument is referring to? Let me know if I should open an issue about it.)

@Semphriss
Copy link
Contributor Author

Semphriss commented May 10, 2024

I reviewed the commit and everything looked fine.

One thing though, there are still \n's with SDL_Log in test/testdialog.c, should they be removed too?

@slouken
Copy link
Contributor

slouken commented May 10, 2024

SDL_Log messages should be terminated with "\n"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants