D5866: ecm_qt_declare_logging_category(): more unique include guard for header
Friedrich W. H. Kossebau
noreply at phabricator.kde.org
Mon May 15 14:52:03 UTC 2017
kossebau created this revision.
Restricted Application added projects: Frameworks, Build System.
REVISION SUMMARY
The old guard was created just from the identifier + _H, which runs the
chance to clash in projects which use an identifier matching the project
name and which also have a class or central file header which is named
by the project and then has an include guard matching the filename.
Example:
project ABC -> abc.h with ABC_H guard
identifier ABC, header debug.h -> debug.h with ABC_H guard
any.cpp including both abc.h and debug.h will see only one content
Using both the header file name and identifier for the guard name
and prefixing it additionally with a macro specific term should make
the guard both follow the usual pattern for guards matching the
file name and also add some namespacing to allow for similar named
header files in bigger projects (e.g. "debug.h") which could be
included in the same include tree.
REPOSITORY
R240 Extra CMake Modules
BRANCH
improveqtlogdeclareheaderguard
REVISION DETAIL
https://phabricator.kde.org/D5866
AFFECTED FILES
modules/ECMQtDeclareLoggingCategory.cmake
To: kossebau, #frameworks, #build_system
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170515/d6dc4eee/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list