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

Consider enabling misc-include-cleaner #11664

Open
schwehr opened this issue Jan 15, 2025 · 0 comments
Open

Consider enabling misc-include-cleaner #11664

schwehr opened this issue Jan 15, 2025 · 0 comments

Comments

@schwehr
Copy link
Member

schwehr commented Jan 15, 2025

Feature description

https://clang.llvm.org/extra/clang-tidy/checks/misc/include-cleaner.html

e.g. with frmts/vrt/pixelfunctions.cpp

 std::unique_ptr<gdal::MathExpression> poExpression;

gives:

no header providing "std::unique_ptr" is directly included

Additional context

This is more important for the headers than for the cpp files. A part of self-contained headers is required for pre-compiled headers and can decrease build times for large projects that depend on GDAL.

I would expect it to take a while to gradually work up to the point where all of them are fixed.

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

No branches or pull requests

1 participant