diff options
Diffstat (limited to 'kde-modules/KDEFrameworkCompilerSettings.cmake')
-rw-r--r-- | kde-modules/KDEFrameworkCompilerSettings.cmake | 15 |
1 files changed, 13 insertions, 2 deletions
diff --git a/kde-modules/KDEFrameworkCompilerSettings.cmake b/kde-modules/KDEFrameworkCompilerSettings.cmake index 0799a8d7..d0bf4dd7 100644 --- a/kde-modules/KDEFrameworkCompilerSettings.cmake +++ b/kde-modules/KDEFrameworkCompilerSettings.cmake @@ -1,3 +1,14 @@ +# Set stricter compile and link flags for KDE Frameworks modules +# +# The KDECompilerSettings module is included and, in addition, various +# defines that affect the Qt libraries are set to enforce certain +# conventions. +# +# For example, constructions like QString("foo") are prohibited, instead +# forcing the use of QLatin1String or QStringLiteral, and some +# Qt-defined keywords like signals and slots will not be defined. +# + include(KDECompilerSettings) add_definitions(-DQT_NO_CAST_TO_ASCII @@ -10,8 +21,8 @@ add_definitions(-DQT_NO_CAST_TO_ASCII ) if(NOT MSVC) # QT_STRICT_ITERATORS breaks MSVC: it tries to link to QTypedArrayData symbols - # when using foreach. However these symbols don't actually exist - # not having QT_STRICT_ITERATORS defined fixes this issue. + # when using foreach. However these symbols don't actually exist. + # Not having QT_STRICT_ITERATORS defined fixes this issue. # This is fixed by https://codereview.qt-project.org/#change,76311 # TODO: set QT_STRICT_ITERATORS on all platforms once we depend on Qt 5.3 add_definitions(-DQT_STRICT_ITERATORS) |