Sorry for the long PR. This basically sticks a namespace { ... } // anonymous namespace around all sections that had static functions and removes the static keyword. In some cases I moved some functions around to consolidate several anon namespace sections into one. This is not consistently done though, in some case I thought it to be better to leave the anon section in the context of the code around it.
This is recommended in C++ Core Guidelines SF.22 https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#Rs-unnamed2
Sorry for the long PR. This basically sticks a
namespace { ... } // anonymous namespace
around all sections that had static functions and removes thestatic
keyword. In some cases I moved some functions around to consolidate several anon namespace sections into one. This is not consistently done though, in some case I thought it to be better to leave the anon section in the context of the code around it.