Skip to content

Move to nested C++ namespaces wherever possible

Laurent Petre requested to merge cleanup/nested-namespaces into develop


Title say it all. More details in the commit message:

All namespaces have been converted to nested namespaces.

An exception applies to gem::hardware that must be compilable by an old
compiler without the C++17 features and syntax.

Due to the large changes introduced in this commit, other modifications
have been pushed:

* Unification of the files Doxygen directives:
  * Usage of @file without filename for all files to be included in the
  * @brief is used only when documentation was written.
  * Removal of the @author commands.
* Fix of the header include directives ordering.
* Specify the namespace corresponding to a closing brace.

Related Issue

Closes #66 (closed).

How Has This Been Tested?

This is just styling, code still compiles.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)


  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Merge request reports