Header inclusion ordering style addition
Patrick Venture
venture at google.com
Wed Aug 29 03:35:57 AEST 2018
Each file in each daemon attempts to order the headers in a consistent
way. Typically, alphabetically. However, I propose here a more
explicit ordering:
https://gerrit.openbmc-project.xyz/12180
As an example of the ordering:
https://gerrit.openbmc-project.xyz/11421
I feel that having a consistent ordering that deals with the various
types of header files included provides a couple benefits:
* header files are included where you expect to find them
* easily can see that there are c-headers included in a clean group
* defines that the phosphor library headers are treated as normal cpp
libraries and included in that grouping making the headers come across
as system libraries and not something different.
Please take a look.
Patrick
More information about the openbmc
mailing list