From 1ae7c76b2adb3b72f736b25cb8195cbefb4bef67 Mon Sep 17 00:00:00 2001 From: Giuseppe D'Angelo Date: Sun, 11 May 2014 16:56:21 +0200 Subject: Do not build or link to PCRE if QRegularExpression is disabled Per Oswald's suggestion, just don't touch PCRE if it's not needed. This can save ~500kB between text and data in QtCore. Change-Id: Ia10c819c7fff562dda84ab0b77194baffbc8904e Reviewed-by: Thiago Macieira Reviewed-by: Oswald Buddenhagen --- src/corelib/corelib.pro | 2 +- src/corelib/tools/tools.pri | 15 +++++++++------ 2 files changed, 10 insertions(+), 7 deletions(-) (limited to 'src/corelib') diff --git a/src/corelib/corelib.pro b/src/corelib/corelib.pro index 812aee3252..c690f49b08 100644 --- a/src/corelib/corelib.pro +++ b/src/corelib/corelib.pro @@ -31,6 +31,7 @@ ANDROID_PERMISSIONS = \ android.permission.WRITE_EXTERNAL_STORAGE load(qt_module) +load(qfeatures) include(animation/animation.pri) include(arch/arch.pri) @@ -84,7 +85,6 @@ cmake_umbrella_config_version_file.input = $$PWD/../../mkspecs/features/data/cma cmake_umbrella_config_version_file.output = $$DESTDIR/cmake/Qt5/Qt5ConfigVersion.cmake load(cmake_functions) -load(qfeatures) CMAKE_DISABLED_FEATURES = $$join(QT_DISABLED_FEATURES, "$$escape_expand(\\n) ") diff --git a/src/corelib/tools/tools.pri b/src/corelib/tools/tools.pri index 57a9591060..3d5111b049 100644 --- a/src/corelib/tools/tools.pri +++ b/src/corelib/tools/tools.pri @@ -43,7 +43,6 @@ HEADERS += \ tools/qqueue.h \ tools/qrect.h \ tools/qregexp.h \ - tools/qregularexpression.h \ tools/qringbuffer_p.h \ tools/qrefcount.h \ tools/qscopedpointer.h \ @@ -100,7 +99,6 @@ SOURCES += \ tools/qcontiguouscache.cpp \ tools/qrect.cpp \ tools/qregexp.cpp \ - tools/qregularexpression.cpp \ tools/qrefcount.cpp \ tools/qshareddata.cpp \ tools/qsharedpointer.cpp \ @@ -176,10 +174,15 @@ contains(QT_CONFIG,icu) { SOURCES += tools/qcollator_posix.cpp } -pcre { - include($$PWD/../../3rdparty/pcre.pri) -} else { - LIBS_PRIVATE += -lpcre16 +!contains(QT_DISABLED_FEATURES, regularexpression) { + HEADERS += tools/qregularexpression.h + SOURCES += tools/qregularexpression.cpp + + pcre { + include($$PWD/../../3rdparty/pcre.pri) + } else { + LIBS_PRIVATE += -lpcre16 + } } INCLUDEPATH += ../3rdparty/harfbuzz/src -- cgit v1.2.3