Skip to content

Commit

Permalink
don't emit comments to generated qfeatures.h
Browse files Browse the repository at this point in the history
the file is not meant to be human-readable and even less editable, so
there is no point in having comments in it.

also, it was completely inconsistent to start with: features without
dependencies were listed as "templates" in form of commented out
 #defines to disable them manually, while features with dependencies
had a respective #ifdef block to be automatically disabled, but no
"template" to disable them manually. now only the #ifdef blocks remain.
the actual configuration is done by configure via qconfig.h.

Change-Id: I8b9e56ba570908dad4cc6dfcd24bf0e1da8b290f
Reviewed-by: Tasuku Suzuki <[email protected]>
Reviewed-by: Joerg Bornemann <[email protected]>
  • Loading branch information
ossilator authored and The Qt Project committed Oct 29, 2013
1 parent 08a737d commit f37ed85
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions qtbase.pro
Original file line number Diff line number Diff line change
Expand Up @@ -127,17 +127,12 @@ features = $$sort_depends(features, features.)
features = $$reverse(features)
FEATURES_H = \
"/*" \
" * All features and their dependencies." \
" * All feature dependencies." \
" *" \
" * This list is generated by qmake from <qtbase>/src/corelib/global/qfeatures.txt" \
" */"
for (ft, features) {
FEATURES_H += \
"$$escape_expand(\\n)// $$eval(features.$${ft}.name)"
isEmpty(features.$${ft}.depends) {
FEATURES_H += \
"//$${LITERAL_HASH}define QT_NO_$$ft"
} else {
!isEmpty(features.$${ft}.depends) {
FEATURES_H += \
"$${LITERAL_HASH}if !defined(QT_NO_$$ft) && ($$join($$list($$split(features.$${ft}.depends)), ") || defined(QT_NO_", "defined(QT_NO_", ")"))" \
"$${LITERAL_HASH} define QT_NO_$$ft" \
Expand Down

0 comments on commit f37ed85

Please sign in to comment.