<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jan 3, 2020, 10:53 AM Gedare Bloom <<a href="mailto:gedare@rtems.org">gedare@rtems.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello all,<br>
<br>
Vijay noted in another thread that:<br>
"For RTEMS, I don't see any preference mentioned in the docs for the<br>
order or includes:<br>
<a href="https://docs.rtems.org/branches/master/eng/coding-conventions.html" rel="noreferrer noreferrer" target="_blank">https://docs.rtems.org/branches/master/eng/coding-conventions.html</a><br>
<br>
In libbsd, however, the FreeBSD style guide is followed which has a<br>
preference for the order of header includes:<br>
<a href="https://www.freebsd.org/cgi/man.cgi?query=style&apropos=0&sektion=9" rel="noreferrer noreferrer" target="_blank">https://www.freebsd.org/cgi/man.cgi?query=style&apropos=0&sektion=9</a>"<br>
<br>
Should we consider any rules? I would suspect that at least ordering<br>
by API layering could be helpful. Lexical sorting is nice but probably<br>
there will be some exceptions based on dependencies.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Those look pretty good as rules. I try to alphabetize headers but you are right, sometimes it isn't possible. I'd be ok if this was added to our style.</div><div dir="auto"><br></div><div dir="auto">The linked page mentions how they do the beginning of a license block with /*- so a program they have recovnizes it as a license but indent doesn't like that so use /** when you need indent. They don't mention that /** turns it into a Doxygen comment which creates another problem. We may want to discuss doing something on these blocks. Personally I'd like to see licenses start to change and spdx tags added even if the license can't change.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Gedare<br>
_______________________________________________<br>
devel mailing list<br>
<a href="mailto:devel@rtems.org" target="_blank" rel="noreferrer">devel@rtems.org</a><br>
<a href="http://lists.rtems.org/mailman/listinfo/devel" rel="noreferrer noreferrer" target="_blank">http://lists.rtems.org/mailman/listinfo/devel</a><br>
</blockquote></div></div></div>