Lustre Style Guide Includes: Difference between revisions

From Lustre Wiki
Jump to navigation Jump to search
(Creation)
 
(provide rationale for sorted headers)
 
(2 intermediate revisions by 2 users not shown)
Line 2: Line 2:
Includes should be ordered in the following way:
Includes should be ordered in the following way:
# linux headers
# linux headers
# libcfs headers
# lnet headers
# lnet headers
# lustre/include/lustre/* headers  
# lustre/include/lustre/* headers  
Line 7: Line 8:
# *_internal.h and any other "local.h" files
# *_internal.h and any other "local.h" files


Each group shall be sorted alphanumerically
Each group shall be sorted alphanumerically.
 
The rationale for this is that Lustre is a strict superset of LNet functionality so there shouldn't be any LNet code that depends on Lustre headers, and Lustre headers should not override or otherwise conflict with kernel headers.  Sorting alphabetically makes it easier to avoid duplicate header include and reduces the chance of patch conflicts if new headers are added at the end in separate patches.
 
[[Category: Development]]

Latest revision as of 16:30, 12 December 2016

When adding new #include files

Includes should be ordered in the following way:

  1. linux headers
  2. libcfs headers
  3. lnet headers
  4. lustre/include/lustre/* headers
  5. lustre/include/*
  6. *_internal.h and any other "local.h" files

Each group shall be sorted alphanumerically.

The rationale for this is that Lustre is a strict superset of LNet functionality so there shouldn't be any LNet code that depends on Lustre headers, and Lustre headers should not override or otherwise conflict with kernel headers. Sorting alphabetically makes it easier to avoid duplicate header include and reduces the chance of patch conflicts if new headers are added at the end in separate patches.