reserved identifier violation

[elfring]

I would like to point out that identifiers like “_AST_DECL_AST_DECL_HH” and “_COMM_MANAGER_H” do not fit to the expected naming convention of the C++ language standard. Would you like to adjust your selection for unique names?

[mitza-oci]

There are uses of preprocessor identifiers (and probably others) that do not conform to what the standard requires because they're reserved. I think a contribution to change this would be welcome.

[elfring]

:thought_balloon: Would you become interested to use a development tool like “clang-tidy” for corresponding source code adjustments?

[jwillemsen]

Feel free to use a tool to make a proposed pull request