nasa / CF

The Core Flight System (cFS) CFDP application.
Apache License 2.0
76 stars 45 forks source link

Superfluous header #includes #360

Closed thnkslprpt closed 1 year ago

thnkslprpt commented 1 year ago

Checklist

Describe the bug In a recent PR (in cFE) it was noted that some includes were not necessary either because the relevant files would have to be included separately anyway, or because they came 'for free' via nested includes in other headers that pull in the same files.

A quick run through of CF shows a few that might be able to be removed here as well (this branch still builds fine, at least the CI workflows).

Code snips https://github.com/nasa/CF/compare/main...thnkslprpt:CF:cppclean-headers

Reporter Info Avi Weiss @thnkslprpt