ocornut / ocornut/imgui_test_engine
TestSuite: imgui_app.h for Metal
Nobody has claimed this yet.
- Dominant language
- C++
- Stars
- 629
- Forks
- 83
- PR merge metrics
- No merged PRs in 30d
Description
OF course, "// THIS IS FOR OUR OWN USE AND IS NOT SUPPORTED."
It would be nice to have the struct / types / etc clearly defined in the header rather than in the cpp file.
Please let me know if you'd rather not have me add 'issues' such as this. Alternatively, when I have some free time I can tackle it myself and make a request (lmk)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with imgui_app.h and its corresponding .cpp file, focusing on the Metal test-suite app implementation. Identify the structs and types currently defined only in the .cpp, then check their uses before deciding which definitions belong in the header. Done means the relevant types are clearly declared in imgui_app.h without breaking the Metal build or existing test-suite usage.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100