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

IWYU support (part 5): Yet more SDL fixes #27810

Merged
merged 2 commits into from
Jan 23, 2019

Conversation

jbytheway
Copy link
Contributor

Summary

SUMMARY: None

Purpose of change

The SDL headers continue to be the bane of IWYU support. But I think I finally have them working as desired now.

Describe the solution

Add a bunch of new symbol mappings for the symbols used from SDL.h and SDL_ttf.h, because these two need special handling.

The problem without these changes is that IWYU forces <SDL.h> to change to "SDL.h". Not sure why; probably a bug, but easier to work around this way than fix in IWYU.

Also updated the docs a little bit.

Sometimes IWYU decides to use "SDL.h" when we want <SDL.h>.  Add
mappings to point it in the right direction.
Mention the complexity of the SDL headers.

Link to JSON style guide.

Formatting.
@ZhilkinSerg ZhilkinSerg added SDL: Tiles / Sound Tiles visual interface and sounds. [C++] Changes (can be) made in C++. Previously named `Code` Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style <Documentation> Design documents, internal info, guides and help. and removed [C++] Changes (can be) made in C++. Previously named `Code` labels Jan 23, 2019
@ZhilkinSerg ZhilkinSerg merged commit 5a71779 into CleverRaven:master Jan 23, 2019
@jbytheway jbytheway deleted the iwyu_sdl_fixes branch January 23, 2019 23:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Code: Infrastructure / Style / Static Analysis Code internal infrastructure and style <Documentation> Design documents, internal info, guides and help. SDL: Tiles / Sound Tiles visual interface and sounds.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants