summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKieran Bingham <kieran.bingham@ideasonboard.com>2021-06-25 00:58:49 +0100
committerKieran Bingham <kieran.bingham@ideasonboard.com>2021-08-11 10:33:40 +0100
commit0e1ff86e78ae1813b29972ce4a682ad99dbb41e3 (patch)
tree65843bb5a3d776304029090a6bd8c6bb211317f7
parente552981c8f77981390dc4241525276648d616a37 (diff)
clang-format: Regroup sort orders
Utilise the clang-format header sort to provide a regex based pattern match for our header inclusion coding style. The rules are updated to match as closely as possible the existing practices and the documentation is updated accordingly. Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Reviewed-by: Jean-Michel Hautbois <jeanmichel.hautbois@ideasonboard.com> Signed-off-by: Kieran Bingham <kieran.bingham@ideasonboard.com>
-rw-r--r--.clang-format53
-rw-r--r--Documentation/coding-style.rst17
2 files changed, 64 insertions, 6 deletions
diff --git a/.clang-format b/.clang-format
index 3a8a896e..ff60b928 100644
--- a/.clang-format
+++ b/.clang-format
@@ -66,10 +66,57 @@ ExperimentalAutoDetectBinPacking: false
FixNamespaceComments: true
ForEachMacros:
- 'udev_list_entry_foreach'
-IncludeBlocks: Preserve
+SortIncludes: true
+IncludeBlocks: Regroup
IncludeCategories:
- - Regex: '.*'
- Priority: 1
+ # Headers matching the name of the component are matched automatically.
+ # Priority 1
+ # Other library headers (explicit overrides to match before system headers)
+ - Regex: '(<jpeglib.h>|<libudev.h>|<tiffio.h>|<xf86drm.h>|<xf86drmMode.h>|<yaml.h>)'
+ Priority: 9
+ # Qt includes (match before C++ standard library)
+ - Regex: '<Q([A-Za-z0-9\-_])+>'
+ Priority: 9
+ CaseSensitive: true
+ # Headers in <> with an extension. (+system libraries)
+ - Regex: '<([A-Za-z0-9\-_])+\.h>'
+ Priority: 2
+ # System headers
+ - Regex: '<sys/.*>'
+ Priority: 2
+ # C++ standard library includes (no extension)
+ - Regex: '<([A-Za-z0-9\-_/])+>'
+ Priority: 2
+ # Linux headers, as a second group/subset of system headers
+ - Regex: '<linux/.*>'
+ Priority: 3
+ # Headers for libcamera Base support
+ - Regex: '<libcamera/base/private.h>'
+ Priority: 4
+ - Regex: '<libcamera/base/.*\.h>'
+ Priority: 5
+ # Public API Headers for libcamera, which are not in a subdir (i.e. ipa/,internal/)
+ - Regex: '<libcamera/([A-Za-z0-9\-_])+.h>'
+ Priority: 6
+ # IPA Interfaces
+ - Regex: '<libcamera/ipa/.*\.h>'
+ Priority: 7
+ # libcamera Internal headers in ""
+ - Regex: '"libcamera/internal/.*\.h"'
+ Priority: 8
+ # Other libraries headers with one group per library (.h or .hpp)
+ - Regex: '<.*/.*\.hp*>'
+ Priority: 9
+ # local modular includes "path/file.h" (.h or .hpp)
+ - Regex: '"(.*/)+.*\.hp*"'
+ Priority: 10
+ # Other local headers "file.h" with extension (.h or .hpp)
+ - Regex: '".*.hp*"'
+ Priority: 11
+ # Any unmatched line, separated from the last group
+ - Regex: '"*"'
+ Priority: 100
+
IncludeIsMainRegex: '(_test)?$'
IndentCaseLabels: false
IndentPPDirectives: None
diff --git a/Documentation/coding-style.rst b/Documentation/coding-style.rst
index 5db7f480..ef3a0d17 100644
--- a/Documentation/coding-style.rst
+++ b/Documentation/coding-style.rst
@@ -70,19 +70,30 @@ macro. For .cpp files, if the file implements an API declared in a header file,
that header file shall be included first in order to ensure it is
self-contained.
+While the following list is extensive, it documents the expected behaviour
+defined by the clang-format configuration and tooling should assist with
+ordering.
+
The headers shall be grouped and ordered as follows:
1. The header declaring the API being implemented (if any)
2. The C and C++ system and standard library headers
-3. Other libraries' headers, with one group per library
-4. Other project's headers
+3. Linux kernel headers
+4. The libcamera base private header if required
+5. The libcamera base library headers
+6. The libcamera public API headers
+7. The libcamera IPA interfaces
+8. The internal libcamera headers
+9. Other libraries' headers, with one group per library
+10. Local headers grouped by subdirectory
+11. Any local headers
Groups of headers shall be separated by a single blank line. Headers within
each group shall be sorted alphabetically.
System and library headers shall be included with angle brackets. Project
headers shall be included with angle brackets for the libcamera public API
-headers, and with double quotes for other libcamera headers.
+headers, and with double quotes for internal libcamera headers.
C++ Specific Rules