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 more C headers #245

Closed
wants to merge 1 commit into from
Closed

Add more C headers #245

wants to merge 1 commit into from

Conversation

steple
Copy link

@steple steple commented May 2, 2024

I stumbled upon these.

I stumbled upon these.
@steple steple requested a review from martis42 as a code owner May 2, 2024 18:16
@martis42
Copy link
Owner

martis42 commented May 2, 2024

@steple
Thanks for proposing this. I won't accept this PR. Not because I reject the proposal itself, but because I want to consistently add all C library and POSIX C library headers. The current list was based on https://en.cppreference.com/w/cpp/header which however ignores many common C headers.

@martis42
Copy link
Owner

martis42 commented May 2, 2024

P.S.: If you need a fast solution you can provide yourself include statements which shall be ignored, see this example: https://github.com/martis42/depend_on_what_you_use/tree/main/examples/ignoring_includes

@steple
Copy link
Author

steple commented May 2, 2024

extra_ignore_include_paths works perfectly for my use case, thank you.

@steple steple closed this May 2, 2024
@martis42
Copy link
Owner

martis42 commented May 3, 2024

@steple
#248 should help with your issue in the long term.

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.

2 participants