search

stephen-hqxu / superterrainplus

SuperTerrain+: A real-time procedural 3D infinite terrain engine with geographical features and photorealistic rendering.
MIT License
12 stars 1 forks source link

Namespace refactoring #25

Closed stephen-hqxu closed 3 years ago

stephen-hqxu commented 3 years ago

Current state

Notice namespaces in all headers are declared with documentation, which is unnecessary and makes maintenance much harder if the documentation needs to be changed in the future.

Also it's not needed to declare nested namespaces lile this:

namespace A {
   namespace B {
      }
}

Suggestion

Consider removing all documentations from the namespace declaration.

As for C++17, it's possible to declare nested namespaces like:

namespace A::B::C {

}
stephen-hqxu commented 3 years ago

Done

We have removed all documentations for namespaces, and merge nested namespaces when necessary.