I would like to point out that identifiers like "_WEBMODULES_H" and "_SQL" do not fit to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?
I would like to point out that identifiers like "[`_WEBMODULES_H`](https://github.com/znc/znc/blob/3e56f093f2c21cd9ea72452773f9fc6a8fdba481/include/znc/WebModules.h#L17)" and "[`_SQL`](https://github.com/znc/znc/blob/b2dcad5fd49e6c0d604de38c311db0e08ed061a7/include/znc/ZNCString.h#L27)" [do not fit](https://www.securecoding.cert.org/confluence/display/cplusplus/DCL32-CPP.+Do+not+declare+or+define+a+reserved+identifier#DCL32-CPP.Donotdeclareordefineareservedidentifier-NoncompliantCodeExample%28HeaderGuard%29) to the expected naming convention of the C++ language standard.
Would you like to adjust your selection for unique names?
##
<bountysource-plugin>
---
There is a **[$5 open bounty](https://www.bountysource.com/issues/1331675-reserved-identifier-violation?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github).
</bountysource-plugin>
<!-- github-sync-id:23150087 -->
What about #pragma once? While not standard, just about every compiler supports it. And it's faster than preprocessor directives.
What about [#pragma once](https://en.wikipedia.org/wiki/Pragma_once)? While not standard, just about every compiler supports it. And it's faster than preprocessor directives.
<!-- github-comment-id:460903003 -->
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
I would like to point out that identifiers like "
--- There is a **[$5 open bounty](https://www.bountysource.com/issues/1331675-reserved-identifier-violation?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)** on this issue. Add to the bounty at [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F1759&utm_medium=issues&utm_source=github)._WEBMODULES_H" and "_SQL" do not fit to the expected naming convention of the C++ language standard.Would you like to adjust your selection for unique names?
Thanks for pointing these issues out, and pull requests are welcome :)
What about #pragma once? While not standard, just about every compiler supports it. And it's faster than preprocessor directives.