afrl-rq / OpenUxAS

Project for multi-UAV cooperative decision making
Other
53 stars 25 forks source link

Remove "using namespace" usage in header files, fix includes ordering. #92

Closed pwilliams12 closed 1 year ago

pwilliams12 commented 1 year ago
  1. "using namespace" should not be placed in header files as this can be dangerous. Only include at the source file level.
  2. Re-order includes to avoid implicit dependence on other headers, use following order:
    1. Header for this source file
    2. OpenUxAS header files
    3. Government headers (e.g. LMCP)
    4. 3rd party headers
    5. C++ STL and system headers