SDL2 testtimer query

Open Beginner friendly
#16,053 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
65/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Quiet
Tech stack
c
Domain
testing-qa

Research direction

Start with test/testtimer.c around line 83 and src/test/SDL_test_common.c around line 1135, then compare the corresponding SDL3 implementation at src/test/SDL_test_common.c around line 1496. Run the SDL2 testtimer test and verify that its requested initialization flags are handled consistently with SDL3.

Written by the indexing model from the issue text.

Description

testtimer passes SDL_INIT_TIMER to test framework: https://github.com/libsdl-org/SDL/blob/SDL2/test/testtimer.c#L83

but CommonInit seems to ignore this flag https://github.com/libsdl-org/SDL/blob/SDL2/src/test/SDL_test_common.c#L1135

My question is that should SDL2 CommonInit pass flags to SDL_InitSubSystem like SDL3 does? https://github.com/libsdl-org/SDL/blob/main/src/test/SDL_test_common.c#L1496

Dominant language
C
Stars
16.6k
Forks
3k
Avg merge
15h 40m
Merged PRs (30d)
70

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from libsdl-org/SDL

All issues in libsdl-org/SDL

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.