Sortincludes
All threads resolved!
All threads resolved!
Update header sorting.
- Add missing C++ standard library headers to includesorter.py
- Work around some logic errors in the way includesorter.py enforces the stated scoping of sorted include blocks. Document.
- Apply updated includesorter.py
Refs #3297 (closed)
Edited by M. Eric Irrgang
Merge request reports
Activity
This merge request is marked WIP because it depends on !98 (merged), but !98 (merged) has merge request dependencies, and so cannot itself be set as a merge request dependency of this one.
- Resolved by Roland Schulz
Instead of maintaining our own include sorter why don't we let clang-format do that? It also the IncludeCategories option to keep our grouping. Is something else missing?
Advantage would be that clang-format aware IDEs would automatically do it correctly and one wouldn't have to remember to manual run the include-sorter.
Edited by Roland Schulz
- Resolved by M. Eric Irrgang
mentioned in issue #3512 (closed)
mentioned in merge request !98 (merged)
added 17 commits
-
62b5a603...175d3bb6 - 16 commits from branch
master
- 44208ac3 - Update header sorting.
-
62b5a603...175d3bb6 - 16 commits from branch
Please register or sign in to reply