-
-
Notifications
You must be signed in to change notification settings - Fork 32.6k
Closed
Labels
c++Issues and PRs that require attention from people who are familiar with C++.Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.Issues and PRs related to general changes in the lib or src directory.questionIssues that look for answers.Issues that look for answers.
Description
- Version: master
- Subsystem: src
Just noticed this when doing #16122, clang-format
has a different idea of alphabetical orders and put -inl.h
includes before .h
includes. Is there a reason that we include them both? Usually including -inl.h
is enough because they should include .h
as well, and they should be self-contained.
Metadata
Metadata
Assignees
Labels
c++Issues and PRs that require attention from people who are familiar with C++.Issues and PRs that require attention from people who are familiar with C++.lib / srcIssues and PRs related to general changes in the lib or src directory.Issues and PRs related to general changes in the lib or src directory.questionIssues that look for answers.Issues that look for answers.