diff --git a/qbs/modules/sqlite_sources/sqlite-sources.qbs b/qbs/modules/sqlite_sources/sqlite-sources.qbs index 559c3ec8652..ea80e58b1bd 100644 --- a/qbs/modules/sqlite_sources/sqlite-sources.qbs +++ b/qbs/modules/sqlite_sources/sqlite-sources.qbs @@ -9,16 +9,7 @@ Module { Depends { name: "cpp" } cpp.defines: [ - "SQLITE_THREADSAFE=2", "SQLITE_ENABLE_FTS5", "SQLITE_ENABLE_UNLOCK_NOTIFY", - "SQLITE_ENABLE_JSON1", "SQLITE_DEFAULT_FOREIGN_KEYS=1", "SQLITE_TEMP_STORE=2", - "SQLITE_DEFAULT_WAL_SYNCHRONOUS=1", "SQLITE_MAX_WORKER_THREADS", "SQLITE_DEFAULT_MEMSTATUS=0", - "SQLITE_OMIT_DEPRECATED", "SQLITE_OMIT_DECLTYPE", - "SQLITE_MAX_EXPR_DEPTH=0", "SQLITE_OMIT_SHARED_CACHE", "SQLITE_USE_ALLOCA", - "SQLITE_ENABLE_MEMORY_MANAGEMENT", "SQLITE_ENABLE_NULL_TRIM", "SQLITE_OMIT_EXPLAIN", - "SQLITE_OMIT_LOAD_EXTENSION", "SQLITE_OMIT_UTF16", "SQLITE_DQS=0", - "SQLITE_ENABLE_STAT4", "HAVE_ISNAN", "HAVE_FDATASYNC", "HAVE_MALLOC_USABLE_SIZE", - "SQLITE_DEFAULT_MMAP_SIZE=268435456", "SQLITE_CORE", "SQLITE_ENABLE_SESSION", "SQLITE_ENABLE_PREUPDATE_HOOK", - "SQLITE_LIKE_DOESNT_MATCH_BLOBS", + "_HAVE_SQLITE_CONFIG_H", "SQLITE_CORE" ].concat(buildSharedLib ? "BUILD_SQLITE_LIBRARY" : "BUILD_SQLITE_STATIC_LIBRARY") cpp.dynamicLibraries: base.concat((qbs.targetOS.contains("unix") && !qbs.targetOS.contains("bsd")) diff --git a/share/qtcreator/qml/qmlpuppet/qml2puppet/qml2puppetmain.cpp b/share/qtcreator/qml/qmlpuppet/qml2puppet/qml2puppetmain.cpp index 30aec5b0c99..51b265a593b 100644 --- a/share/qtcreator/qml/qmlpuppet/qml2puppet/qml2puppetmain.cpp +++ b/share/qtcreator/qml/qmlpuppet/qml2puppet/qml2puppetmain.cpp @@ -23,24 +23,25 @@ ** ****************************************************************************/ -#include +#include "iconrenderer/iconrenderer.h" +#include "import3d/import3d.h" +#include +#ifdef MULTILANGUAGE_TRANSLATIONPROVIDER +#include +#endif + +#include +#include +#include #include #include #include #include - #include #include -#include "iconrenderer/iconrenderer.h" -#include "import3d/import3d.h" -#include - -#include -#include - #ifdef ENABLE_QT_BREAKPAD #include #endif @@ -274,6 +275,10 @@ int main(int argc, char *argv[]) && qEnvironmentVariableIsSet("QT_QUICK_CONTROLS_STYLE") && qgetenv("QT_QUICK_CONTROLS_STYLE") != "Desktop"; +#ifdef MULTILANGUAGE_TRANSLATIONPROVIDER + Sqlite::LibraryInitializer::initialize(); +#endif + if (useGuiApplication) { QGuiApplication application(argc, argv); return internalMain(&application); diff --git a/share/qtcreator/qmldesigner/propertyEditorQmlSources/RegExpValidator.qml b/share/qtcreator/qmldesigner/propertyEditorQmlSources/RegExpValidator.qml new file mode 100644 index 00000000000..f9ace424d9e --- /dev/null +++ b/share/qtcreator/qmldesigner/propertyEditorQmlSources/RegExpValidator.qml @@ -0,0 +1,6 @@ +import QtQuick 2.15 + +RegularExpressionValidator { + id: root + property alias regExp: root.regularExpression +} diff --git a/share/qtcreator/qmldesigner/propertyEditorQmlSources/imports/StudioTheme/InternalConstants.qml b/share/qtcreator/qmldesigner/propertyEditorQmlSources/imports/StudioTheme/InternalConstants.qml index 54bbe343623..19e319be32c 100644 --- a/share/qtcreator/qmldesigner/propertyEditorQmlSources/imports/StudioTheme/InternalConstants.qml +++ b/share/qtcreator/qmldesigner/propertyEditorQmlSources/imports/StudioTheme/InternalConstants.qml @@ -28,10 +28,8 @@ import QtQuick 2.10 QtObject { readonly property int width: 1920 readonly property int height: 1080 - readonly property FontLoader mySystemFont: FontLoader { - name: "Arial" - } - readonly property FontLoader controlIcons: FontLoader { + + property FontLoader controlIcons: FontLoader { source: "icons.ttf" } @@ -150,12 +148,12 @@ QtObject { }) readonly property font font: Qt.font({ - "family": mySystemFont.name, + "family": "Arial", "pointSize": Qt.application.font.pixelSize }) readonly property font largeFont: Qt.font({ - "family": mySystemFont.name, + "family": "Arial", "pointSize": Qt.application.font.pixelSize * 1.6 }) } diff --git a/src/libs/3rdparty/sqlite/config.h b/src/libs/3rdparty/sqlite/config.h new file mode 100644 index 00000000000..19757d3f8e8 --- /dev/null +++ b/src/libs/3rdparty/sqlite/config.h @@ -0,0 +1,84 @@ +/**************************************************************************** +** +** Copyright (C) 2021 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of Qt Creator. +** +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU +** General Public License version 3 as published by the Free Software +** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT +** included in the packaging of this file. Please review the following +** information to ensure the GNU General Public License requirements will +** be met: https://www.gnu.org/licenses/gpl-3.0.html. +** +****************************************************************************/ + +#pragma once + +#include + +#if __has_include() +#include +#endif + +#if __has_include() +#define HAVE_UTIME 1 +#endif + +#if (_XOPEN_SOURCE >= 500) && !(_POSIX_C_SOURCE >= 200809L) || _DEFAULT_SOURCE || _BSD_SOURCE +#define HAVE_USLEEP 1 +#endif + +#ifdef _GNU_SOURCE +#define HAVE_STRCHRNUL 1 +#endif + +#if _POSIX_C_SOURCE >= 199309L || _XOPEN_SOURCE >= 500 +#define HAVE_FDATASYNC 1 +#endif + +#if _POSIX_C_SOURCE >= 1 || _BSD_SOURCE +#define HAVE_LOCALTIME_R 1 +#else +#define HAVE_LOCALTIME_S 1 +#endif + +#define HAVE_MALLOC_USABLE_SIZE 1 +#define HAVE_ISNAN 1 + +#define SQLITE_THREADSAFE 2 +#define SQLITE_ENABLE_FTS5 1 +#define SQLITE_ENABLE_UNLOCK_NOTIFY 1 +#define SQLITE_ENABLE_JSON1 1 +#define SQLITE_DEFAULT_FOREIGN_KEYS 1 +#define SQLITE_TEMP_STORE 2 +#define SQLITE_DEFAULT_WAL_SYNCHRONOUS 1 +#define SQLITE_MAX_WORKER_THREADS 1 +#define SQLITE_DEFAULT_MEMSTATUS 0 +#define SQLITE_OMIT_DEPRECATED 1 +#define SQLITE_OMIT_DECLTYPE 1 +#define SQLITE_MAX_EXPR_DEPTH 0 +#define SQLITE_OMIT_SHARED_CACHE 1 +#define SQLITE_USE_ALLOCA 1 +#define SQLITE_ENABLE_MEMORY_MANAGEMENT 1 +#define SQLITE_ENABLE_NULL_TRIM 1 +#define SQLITE_OMIT_EXPLAIN 1 +#define SQLITE_OMIT_LOAD_EXTENSION 1 +#define SQLITE_OMIT_UTF16 1 +#define SQLITE_DQS 0 +#define SQLITE_ENABLE_STAT4 1 +#define SQLITE_DEFAULT_MMAP_SIZE 268435456 +#define SQLITE_ENABLE_SESSION 1 +#define SQLITE_ENABLE_PREUPDATE_HOOK 1 +#define SQLITE_LIKE_DOESNT_MATCH_BLOBS 1 +#define SQLITE_OMIT_AUTOINIT 1 diff --git a/src/libs/3rdparty/sqlite/sqlite.h b/src/libs/3rdparty/sqlite/sqlite.h new file mode 100644 index 00000000000..ba06815b4d9 --- /dev/null +++ b/src/libs/3rdparty/sqlite/sqlite.h @@ -0,0 +1,29 @@ +/**************************************************************************** +** +** Copyright (C) 2021 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of Qt Creator. +** +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU +** General Public License version 3 as published by the Free Software +** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT +** included in the packaging of this file. Please review the following +** information to ensure the GNU General Public License requirements will +** be met: https://www.gnu.org/licenses/gpl-3.0.html. +** +****************************************************************************/ + +#pragma once + +#include "config.h" +#include "sqlite3ext.h" diff --git a/src/libs/3rdparty/sqlite/sqlite.pri b/src/libs/3rdparty/sqlite/sqlite.pri index ffc3a706e65..e869f4fc56e 100644 --- a/src/libs/3rdparty/sqlite/sqlite.pri +++ b/src/libs/3rdparty/sqlite/sqlite.pri @@ -1,6 +1,12 @@ INCLUDEPATH *= $$PWD +linux:DEFINES += _POSIX_C_SOURCE=200809L _GNU_SOURCE +osx:DEFINES += _BSD_SOURCE + + HEADERS += $$PWD/sqlite3.h \ + $$PWD/config.h \ + $$PWD/sqlite.h \ $$PWD/sqlite3ext.h SOURCES += $$PWD/sqlite3.c \ diff --git a/src/libs/qmljs/qmljscheck.cpp b/src/libs/qmljs/qmljscheck.cpp index 344b39d14e2..8e364fa6bec 100644 --- a/src/libs/qmljs/qmljscheck.cpp +++ b/src/libs/qmljs/qmljscheck.cpp @@ -1366,7 +1366,9 @@ bool Check::visit(Block *ast) && !cast(p) && !cast(p) && !cast(p) - && !cast(p)) { + && !isCaseOrDefault(p) + && !cast(p) + && hasVarStatement(ast)) { addMessage(WarnBlock, ast->lbraceToken); } if (!ast->statements @@ -1656,6 +1658,33 @@ bool Check::isQtQuick2Ui() const return _doc->language() == Dialect::QmlQtQuick2Ui; } +bool Check::isCaseOrDefault(Node *n) +{ + if (!cast(n)) + return false; + if (Node *p = parent(1)) + return p->kind == Node::Kind_CaseClause || p->kind == Node::Kind_DefaultClause; + return false; +} + +bool Check::hasVarStatement(AST::Block *b) const +{ + QTC_ASSERT(b, return false); + StatementList *s = b->statements; + while (s) { + if (auto var = cast(s->statement)) { + VariableDeclarationList *declList = var->declarations; + while (declList) { + if (declList->declaration && declList->declaration->scope == VariableScope::Var) + return true; + declList = declList->next; + } + } + s = s->next; + } + return false; +} + bool Check::visit(NewExpression *ast) { checkNewExpression(ast->expression); diff --git a/src/libs/qmljs/qmljscheck.h b/src/libs/qmljs/qmljscheck.h index e5e110c5f9b..ec208622485 100644 --- a/src/libs/qmljs/qmljscheck.h +++ b/src/libs/qmljs/qmljscheck.h @@ -122,6 +122,9 @@ private: bool isQtQuick2() const; bool isQtQuick2Ui() const; + bool isCaseOrDefault(AST::Node *n); + bool hasVarStatement(AST::Block *b) const; + AST::Node *parent(int distance = 0); Document::Ptr _doc; diff --git a/src/libs/sqlite/CMakeLists.txt b/src/libs/sqlite/CMakeLists.txt index 4bf3ae91df0..42de2ce9e99 100644 --- a/src/libs/sqlite/CMakeLists.txt +++ b/src/libs/sqlite/CMakeLists.txt @@ -1,16 +1,7 @@ add_qtc_library(Sqlite PUBLIC_DEFINES BUILD_SQLITE_LIBRARY - SQLITE_THREADSAFE=2 SQLITE_ENABLE_FTS5 SQLITE_ENABLE_UNLOCK_NOTIFY - SQLITE_ENABLE_JSON1 SQLITE_DEFAULT_FOREIGN_KEYS=1 SQLITE_TEMP_STORE=2 - SQLITE_DEFAULT_WAL_SYNCHRONOUS=1 SQLITE_MAX_WORKER_THREADS SQLITE_DEFAULT_MEMSTATUS=0 - SQLITE_OMIT_DEPRECATED SQLITE_OMIT_DECLTYPE - SQLITE_MAX_EXPR_DEPTH=0 SQLITE_OMIT_SHARED_CACHE SQLITE_USE_ALLOCA - SQLITE_ENABLE_MEMORY_MANAGEMENT SQLITE_ENABLE_NULL_TRIM SQLITE_OMIT_EXPLAIN - SQLITE_OMIT_LOAD_EXTENSION SQLITE_OMIT_UTF16 SQLITE_DQS=0 - SQLITE_ENABLE_STAT4 HAVE_ISNAN HAVE_FDATASYNC HAVE_MALLOC_USABLE_SIZE - SQLITE_DEFAULT_MMAP_SIZE=268435456 SQLITE_CORE SQLITE_ENABLE_SESSION SQLITE_ENABLE_PREUPDATE_HOOK - SQLITE_LIKE_DOESNT_MATCH_BLOBS + SQLITE_CORE DEPENDS Qt5::Core Threads::Threads ${CMAKE_DL_LIBS} PUBLIC_INCLUDES "${CMAKE_CURRENT_LIST_DIR}" @@ -19,6 +10,8 @@ add_qtc_library(Sqlite ../3rdparty/sqlite/sqlite3.c ../3rdparty/sqlite/sqlite3.h ../3rdparty/sqlite/carray.c + ../3rdparty/sqlite/config.h + ../3rdparty/sqlite/sqlite.h constraints.h createtablesqlstatementbuilder.cpp createtablesqlstatementbuilder.h lastchangedrowid.h @@ -46,4 +39,13 @@ add_qtc_library(Sqlite utf8stringvector.cpp utf8stringvector.h sqliteblob.h sqlitetimestamp.h + sqlitelibraryinitializer.cpp sqlitelibraryinitializer.h ) + +extend_qtc_library(Sqlite DEFINES _HAVE_SQLITE_CONFIG_H) + +if (APPLE) +extend_qtc_library(Sqlite DEFINES _BSD_SOURCE) +elseif (UNIX) +extend_qtc_library(Sqlite DEFINES _POSIX_C_SOURCE=200809L _GNU_SOURCE _DEFAULT_SOURCE) +endif() diff --git a/src/libs/sqlite/createtablesqlstatementbuilder.cpp b/src/libs/sqlite/createtablesqlstatementbuilder.cpp index 07008a9a1da..de77eec5769 100644 --- a/src/libs/sqlite/createtablesqlstatementbuilder.cpp +++ b/src/libs/sqlite/createtablesqlstatementbuilder.cpp @@ -248,7 +248,9 @@ void CreateTableSqlStatementBuilder::bindColumnDefinitionsAndTableConstraints() columnDefinitionStrings.reserve(m_columns.size()); for (const Column &column : m_columns) { - Utils::SmallString columnDefinitionString = {column.name, " ", column.typeString()}; + Utils::SmallString columnDefinitionString = {column.name, + SqlStatementBuilder::columnTypeToString( + column.type)}; ContraintsVisiter visiter{columnDefinitionString}; diff --git a/src/libs/sqlite/sqlite-lib.pri b/src/libs/sqlite/sqlite-lib.pri index 87b63eb388e..59d34a61c2f 100644 --- a/src/libs/sqlite/sqlite-lib.pri +++ b/src/libs/sqlite/sqlite-lib.pri @@ -15,6 +15,7 @@ SOURCES += \ $$PWD/sqlitedatabasebackend.cpp \ $$PWD/sqliteexception.cpp \ $$PWD/sqliteglobal.cpp \ + $$PWD/sqlitelibraryinitializer.cpp \ $$PWD/sqlitesessionchangeset.cpp \ $$PWD/sqlitesessions.cpp \ $$PWD/sqlstatementbuilder.cpp \ @@ -25,6 +26,7 @@ SOURCES += \ HEADERS += \ $$PWD/constraints.h \ $$PWD/sqliteblob.h \ + $$PWD/sqlitelibraryinitializer.h \ $$PWD/sqlitetimestamp.h \ $$PWD/tableconstraints.h \ $$PWD/createtablesqlstatementbuilder.h \ @@ -50,16 +52,7 @@ HEADERS += \ $$PWD/sqliteindex.h \ $$PWD/sqlitebasestatement.h -DEFINES += SQLITE_THREADSAFE=2 SQLITE_ENABLE_FTS5 SQLITE_ENABLE_UNLOCK_NOTIFY \ - SQLITE_ENABLE_JSON1 SQLITE_DEFAULT_FOREIGN_KEYS=1 SQLITE_TEMP_STORE=2 \ - SQLITE_DEFAULT_WAL_SYNCHRONOUS=1 SQLITE_MAX_WORKER_THREADS SQLITE_DEFAULT_MEMSTATUS=0 \ - SQLITE_OMIT_DEPRECATED SQLITE_OMIT_DECLTYPE \ - SQLITE_MAX_EXPR_DEPTH=0 SQLITE_OMIT_SHARED_CACHE SQLITE_USE_ALLOCA \ - SQLITE_ENABLE_MEMORY_MANAGEMENT SQLITE_ENABLE_NULL_TRIM SQLITE_OMIT_EXPLAIN \ - SQLITE_OMIT_LOAD_EXTENSION SQLITE_OMIT_UTF16 SQLITE_DQS=0 \ - SQLITE_ENABLE_STAT4 HAVE_ISNAN HAVE_FDATASYNC HAVE_MALLOC_USABLE_SIZE \ - SQLITE_DEFAULT_MMAP_SIZE=268435456 SQLITE_CORE SQLITE_ENABLE_SESSION SQLITE_ENABLE_PREUPDATE_HOOK \ - SQLITE_LIKE_DOESNT_MATCH_BLOBS +DEFINES += _HAVE_SQLITE_CONFIG_H SQLITE_CORE CONFIG(debug, debug|release): DEFINES += SQLITE_ENABLE_API_ARMOR diff --git a/src/libs/sqlite/sqlitebasestatement.cpp b/src/libs/sqlite/sqlitebasestatement.cpp index 6e60133511a..28ebbb8d2ea 100644 --- a/src/libs/sqlite/sqlitebasestatement.cpp +++ b/src/libs/sqlite/sqlitebasestatement.cpp @@ -29,7 +29,7 @@ #include "sqlitedatabasebackend.h" #include "sqliteexception.h" -#include "sqlite3.h" +#include "sqlite.h" #include #include diff --git a/src/libs/sqlite/sqlitebasestatement.h b/src/libs/sqlite/sqlitebasestatement.h index 019a7a44f3e..c96fb3faded 100644 --- a/src/libs/sqlite/sqlitebasestatement.h +++ b/src/libs/sqlite/sqlitebasestatement.h @@ -29,7 +29,6 @@ #include "sqliteblob.h" #include "sqliteexception.h" -#include "sqlitetransaction.h" #include "sqlitevalue.h" #include @@ -38,7 +37,6 @@ #include #include -#include #include #include #include @@ -93,7 +91,6 @@ public: void bind(int index, Utils::span values); void bind(int index, Utils::SmallStringView value); void bind(int index, const Value &value); - void bind(int index, ValueView value); void bind(int index, BlobView blobView); void bind(int index, uint value) { bind(index, static_cast(value)); } @@ -165,14 +162,13 @@ extern template SQLITE_EXPORT Utils::PathString BaseStatement::fetchValue class StatementImplementation : public BaseStatement { - struct Resetter; public: using BaseStatement::BaseStatement; void execute() { - Resetter resetter{this}; + Resetter resetter{*this}; BaseStatement::next(); resetter.reset(); } @@ -182,14 +178,15 @@ public: template void bindValues(const ValueType&... values) { - bindValuesByIndex(1, values...); + int index = 0; + (BaseStatement::bind(++index, values), ...); } template void write(const ValueType&... values) { - Resetter resetter{this}; - bindValuesByIndex(1, values...); + Resetter resetter{*this}; + bindValues(values...); BaseStatement::next(); resetter.reset(); } @@ -197,7 +194,7 @@ public: template std::vector values(std::size_t reserveSize) { - Resetter resetter{this}; + Resetter resetter{*this}; std::vector resultValues; resultValues.reserve(std::max(reserveSize, m_maximumResultCount)); @@ -214,7 +211,7 @@ public: template auto values(std::size_t reserveSize, const QueryTypes &...queryValues) { - Resetter resetter{this}; + Resetter resetter{*this}; std::vector resultValues; resultValues.reserve(std::max(reserveSize, m_maximumResultCount)); @@ -233,13 +230,13 @@ public: template auto value(const QueryTypes &...queryValues) { - Resetter resetter{this}; + Resetter resetter{*this}; Utils::optional resultValue; bindValues(queryValues...); if (BaseStatement::next()) - resultValue = createOptionalValue>(); + resultValue = assignValue>(); resetter.reset(); @@ -261,7 +258,7 @@ public: template void readCallback(Callable &&callable, const QueryTypes &...queryValues) { - Resetter resetter{this}; + Resetter resetter{*this}; bindValues(queryValues...); @@ -275,10 +272,10 @@ public: resetter.reset(); } - template + template void readTo(Container &container, const QueryTypes &...queryValues) { - Resetter resetter{this}; + Resetter resetter{*this}; bindValues(queryValues...); @@ -288,187 +285,39 @@ public: resetter.reset(); } - template - auto range(const QueryTypes &...queryValues) - { - return SqliteResultRange{*this, queryValues...}; - } - - template - auto rangeWithTransaction(const QueryTypes &...queryValues) - { - return SqliteResultRangeWithTransaction{*this, queryValues...}; - } - - template - class BaseSqliteResultRange - { - public: - class SqliteResultIteratator - { - public: - using iterator_category = std::input_iterator_tag; - using difference_type = int; - using value_type = ResultType; - using pointer = ResultType *; - using reference = ResultType &; - - SqliteResultIteratator(StatementImplementation &statement) - : m_statement{statement} - , m_hasNext{m_statement.next()} - {} - - SqliteResultIteratator(StatementImplementation &statement, bool hasNext) - : m_statement{statement} - , m_hasNext{hasNext} - {} - - SqliteResultIteratator &operator++() - { - m_hasNext = m_statement.next(); - return *this; - } - - void operator++(int) { m_hasNext = m_statement.next(); } - - friend bool operator==(const SqliteResultIteratator &first, - const SqliteResultIteratator &second) - { - return first.m_hasNext == second.m_hasNext; - } - - friend bool operator!=(const SqliteResultIteratator &first, - const SqliteResultIteratator &second) - { - return !(first == second); - } - - value_type operator*() const { return m_statement.createValue(); } - - private: - StatementImplementation &m_statement; - bool m_hasNext = false; - }; - - using value_type = ResultType; - using iterator = SqliteResultIteratator; - using const_iterator = iterator; - - template - BaseSqliteResultRange(StatementImplementation &statement, const QueryTypes &...queryValues) - : m_statement{statement} - { - statement.bindValues(queryValues...); - } - - BaseSqliteResultRange(BaseSqliteResultRange &) = delete; - BaseSqliteResultRange &operator=(BaseSqliteResultRange &) = delete; - - BaseSqliteResultRange(BaseSqliteResultRange &&other) - : m_statement{std::move(other.resetter)} - {} - BaseSqliteResultRange &operator=(BaseSqliteResultRange &&) = delete; - - iterator begin() & { return iterator{m_statement}; } - iterator end() & { return iterator{m_statement, false}; } - - const_iterator begin() const & { return iterator{m_statement}; } - const_iterator end() const & { return iterator{m_statement, false}; } - - private: - StatementImplementation &m_statement; - }; - - template - class SqliteResultRange : public BaseSqliteResultRange - { - public: - template - SqliteResultRange(StatementImplementation &statement, const QueryTypes &...queryValues) - : BaseSqliteResultRange{statement} - , resetter{&statement} - - { - statement.bindValues(queryValues...); - } - - ~SqliteResultRange() - { - if (!std::uncaught_exceptions()) - resetter.reset(); - } - - private: - Resetter resetter; - }; - - template - class SqliteResultRangeWithTransaction : public BaseSqliteResultRange - { - public: - template - SqliteResultRangeWithTransaction(StatementImplementation &statement, - const QueryTypes &...queryValues) - : BaseSqliteResultRange{statement} - , m_transaction{statement.database()} - , resetter{&statement} - { - statement.bindValues(queryValues...); - } - - ~SqliteResultRangeWithTransaction() - { - if (!std::uncaught_exceptions()) { - resetter.reset(); - m_transaction.commit(); - } - } - - private: - DeferredTransaction m_transaction; - Resetter resetter; - }; - protected: ~StatementImplementation() = default; private: struct Resetter { - Resetter(StatementImplementation *statement) + Resetter(StatementImplementation &statement) : statement(statement) {} - Resetter(Resetter &) = delete; - Resetter &operator=(Resetter &) = delete; - - Resetter(Resetter &&other) - : statement{std::exchange(other.statement, nullptr)} - {} - void reset() { try { - if (statement) - statement->reset(); + statement.reset(); } catch (...) { - statement = nullptr; + shouldReset = false; throw; } - statement = nullptr; + shouldReset = false; } ~Resetter() noexcept { try { - if (statement) - statement->reset(); + if (shouldReset) + statement.reset(); } catch (...) { } } - StatementImplementation *statement; + StatementImplementation &statement; + bool shouldReset = true; }; struct ValueGetter @@ -490,11 +339,6 @@ private: int column; }; - constexpr int resultCount(int localResultCount) const - { - return ResultCount < 0 ? localResultCount : ResultCount; - } - template void emplaceBackValues(ContainerType &container, std::integer_sequence) { @@ -507,28 +351,17 @@ private: emplaceBackValues(container, std::make_integer_sequence{}); } - template - ResultOptionalType createOptionalValue(std::integer_sequence) + template + ResultOptionalType assignValue(std::integer_sequence) { return ResultOptionalType(Utils::in_place, ValueGetter(*this, ColumnIndices)...); } template - ResultOptionalType createOptionalValue() + ResultOptionalType assignValue() { - return createOptionalValue(std::make_integer_sequence{}); - } - - template - ResultType createValue(std::integer_sequence) - { - return ResultType{ValueGetter(*this, ColumnIndices)...}; - } - - template - ResultType createValue() - { - return createValue(std::make_integer_sequence{}); + return assignValue(std::make_integer_sequence{}); } template @@ -543,19 +376,6 @@ private: return callCallable(callable, std::make_integer_sequence{}); } - template - void bindValuesByIndex(int index, const ValueType &value) - { - BaseStatement::bind(index, value); - } - - template - void bindValuesByIndex(int index, const ValueType &value, const ValueTypes &...values) - { - BaseStatement::bind(index, value); - bindValuesByIndex(index + 1, values...); - } - void setMaximumResultCount(std::size_t count) { m_maximumResultCount = std::max(m_maximumResultCount, count); diff --git a/src/libs/sqlite/sqlitecolumn.h b/src/libs/sqlite/sqlitecolumn.h index 1775a683075..35a5e0f25c9 100644 --- a/src/libs/sqlite/sqlitecolumn.h +++ b/src/libs/sqlite/sqlitecolumn.h @@ -39,7 +39,7 @@ public: Column(Utils::SmallStringView tableName, Utils::SmallStringView name, - ColumnType type, + ColumnType type = ColumnType::None, Constraints &&constraints = {}) : constraints(std::move(constraints)) , name(name) diff --git a/src/libs/sqlite/sqlitedatabasebackend.cpp b/src/libs/sqlite/sqlitedatabasebackend.cpp index 0101e826260..9c096364cb5 100644 --- a/src/libs/sqlite/sqlitedatabasebackend.cpp +++ b/src/libs/sqlite/sqlitedatabasebackend.cpp @@ -35,7 +35,7 @@ #include #include -#include "sqlite3.h" +#include "sqlite.h" #include #include diff --git a/src/libs/sqlite/sqliteglobal.h b/src/libs/sqlite/sqliteglobal.h index 0bdb3c2aef9..cefdebd8d01 100644 --- a/src/libs/sqlite/sqliteglobal.h +++ b/src/libs/sqlite/sqliteglobal.h @@ -39,7 +39,7 @@ namespace Sqlite { -enum class ColumnType : char { Numeric, Integer, Real, Text, Blob, None }; +enum class ColumnType : char { None, Numeric, Integer, Real, Text, Blob }; enum class ConstraintType : char { NoConstraint, PrimaryKey, Unique, ForeignKey }; diff --git a/src/libs/sqlite/sqlitelibraryinitializer.cpp b/src/libs/sqlite/sqlitelibraryinitializer.cpp new file mode 100644 index 00000000000..c42d1269661 --- /dev/null +++ b/src/libs/sqlite/sqlitelibraryinitializer.cpp @@ -0,0 +1,47 @@ +/**************************************************************************** +** +** Copyright (C) 2021 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of Qt Creator. +** +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU +** General Public License version 3 as published by the Free Software +** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT +** included in the packaging of this file. Please review the following +** information to ensure the GNU General Public License requirements will +** be met: https://www.gnu.org/licenses/gpl-3.0.html. +** +****************************************************************************/ + +#include "sqlitelibraryinitializer.h" + +#include "sqlitedatabasebackend.h" + +namespace Sqlite { + +void LibraryInitializer::initialize() +{ + static LibraryInitializer initializer; +} + +LibraryInitializer::LibraryInitializer() +{ + DatabaseBackend::initializeSqliteLibrary(); +} + +LibraryInitializer::~LibraryInitializer() +{ + DatabaseBackend::shutdownSqliteLibrary(); +} + +} // namespace Sqlite diff --git a/src/libs/sqlite/sqlitelibraryinitializer.h b/src/libs/sqlite/sqlitelibraryinitializer.h new file mode 100644 index 00000000000..6df8b19a22e --- /dev/null +++ b/src/libs/sqlite/sqlitelibraryinitializer.h @@ -0,0 +1,42 @@ +/**************************************************************************** +** +** Copyright (C) 2021 The Qt Company Ltd. +** Contact: https://www.qt.io/licensing/ +** +** This file is part of Qt Creator. +** +** Commercial License Usage +** Licensees holding valid commercial Qt licenses may use this file in +** accordance with the commercial license agreement provided with the +** Software or, alternatively, in accordance with the terms contained in +** a written agreement between you and The Qt Company. For licensing terms +** and conditions see https://www.qt.io/terms-conditions. For further +** information use the contact form at https://www.qt.io/contact-us. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU +** General Public License version 3 as published by the Free Software +** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT +** included in the packaging of this file. Please review the following +** information to ensure the GNU General Public License requirements will +** be met: https://www.gnu.org/licenses/gpl-3.0.html. +** +****************************************************************************/ + +#pragma once + +#include "sqliteglobal.h" + +namespace Sqlite { + +class LibraryInitializer +{ +public: + SQLITE_EXPORT static void initialize(); + +private: + LibraryInitializer(); + ~LibraryInitializer(); +}; + +} // namespace Sqlite diff --git a/src/libs/sqlite/sqlitesessionchangeset.cpp b/src/libs/sqlite/sqlitesessionchangeset.cpp index 76fc088f92f..eef98bc5d57 100644 --- a/src/libs/sqlite/sqlitesessionchangeset.cpp +++ b/src/libs/sqlite/sqlitesessionchangeset.cpp @@ -29,7 +29,7 @@ #include -#include +#include namespace Sqlite { diff --git a/src/libs/sqlite/sqlitesessions.cpp b/src/libs/sqlite/sqlitesessions.cpp index df668fdc593..5162a9fa593 100644 --- a/src/libs/sqlite/sqlitesessions.cpp +++ b/src/libs/sqlite/sqlitesessions.cpp @@ -28,7 +28,7 @@ #include "sqlitesessionchangeset.h" #include "sqlitetable.h" -#include +#include #include diff --git a/src/libs/sqlite/sqlitetable.h b/src/libs/sqlite/sqlitetable.h index 9883415a9f6..35453945f21 100644 --- a/src/libs/sqlite/sqlitetable.h +++ b/src/libs/sqlite/sqlitetable.h @@ -72,7 +72,7 @@ public: } Column &addColumn(Utils::SmallStringView name, - ColumnType type = ColumnType::Numeric, + ColumnType type = ColumnType::None, Constraints &&constraints = {}) { m_sqliteColumns.emplace_back(m_tableName, name, type, std::move(constraints)); diff --git a/src/libs/sqlite/sqlstatementbuilder.cpp b/src/libs/sqlite/sqlstatementbuilder.cpp index ecef170a346..7a6cc1a81a3 100644 --- a/src/libs/sqlite/sqlstatementbuilder.cpp +++ b/src/libs/sqlite/sqlstatementbuilder.cpp @@ -179,12 +179,18 @@ bool SqlStatementBuilder::isBuild() const Utils::SmallString SqlStatementBuilder::columnTypeToString(ColumnType columnType) { switch (columnType) { - case ColumnType::Numeric: return "NUMERIC"; - case ColumnType::Integer: return "INTEGER"; - case ColumnType::Real: return "REAL"; - case ColumnType::Text: return "TEXT"; - case ColumnType::Blob: return "BLOB"; - case ColumnType::None: return {}; + case ColumnType::Numeric: + return " NUMERIC"; + case ColumnType::Integer: + return " INTEGER"; + case ColumnType::Real: + return " REAL"; + case ColumnType::Text: + return " TEXT"; + case ColumnType::Blob: + return " BLOB"; + case ColumnType::None: + return {}; } Q_UNREACHABLE(); diff --git a/src/plugins/projectexplorer/project.cpp b/src/plugins/projectexplorer/project.cpp index ec30204f91f..c90ee9f2704 100644 --- a/src/plugins/projectexplorer/project.cpp +++ b/src/plugins/projectexplorer/project.cpp @@ -49,6 +49,7 @@ #include #include #include +#include #include #include @@ -1073,6 +1074,23 @@ QStringList Project::availableQmlPreviewTranslations(QString *errorMessage) }); } +QList Project::modifiedDocuments() const +{ + QList modifiedProjectDocuments; + + for (Core::IDocument *doc : Core::DocumentModel::openedDocuments()) { + if (doc->isModified() && isKnownFile(doc->filePath())) + modifiedProjectDocuments.append(doc); + } + + return modifiedProjectDocuments; +} + +bool Project::isModified() const +{ + return !modifiedDocuments().isEmpty(); +} + #if defined(WITH_TESTS) } // namespace ProjectExplorer diff --git a/src/plugins/projectexplorer/project.h b/src/plugins/projectexplorer/project.h index c16af959466..31cb6e12b98 100644 --- a/src/plugins/projectexplorer/project.h +++ b/src/plugins/projectexplorer/project.h @@ -185,6 +185,9 @@ public: QStringList availableQmlPreviewTranslations(QString *errorMessage); + QList modifiedDocuments() const; + bool isModified() const; + signals: void projectFileIsDirty(const Utils::FilePath &path); diff --git a/src/plugins/qmldesigner/components/propertyeditor/quick2propertyeditorview.cpp b/src/plugins/qmldesigner/components/propertyeditor/quick2propertyeditorview.cpp index 7032a26ef76..92ac7fdf8f1 100644 --- a/src/plugins/qmldesigner/components/propertyeditor/quick2propertyeditorview.cpp +++ b/src/plugins/qmldesigner/components/propertyeditor/quick2propertyeditorview.cpp @@ -25,22 +25,23 @@ #include "quick2propertyeditorview.h" -#include "propertyeditorvalue.h" +#include "aligndistribute.h" +#include "annotationeditor/annotationeditor.h" +#include "bindingeditor/actioneditor.h" +#include "bindingeditor/bindingeditor.h" #include "fileresourcesmodel.h" #include "gradientmodel.h" -#include "gradientpresetdefaultlistmodel.h" #include "gradientpresetcustomlistmodel.h" +#include "gradientpresetdefaultlistmodel.h" #include "itemfiltermodel.h" -#include "simplecolorpalettemodel.h" -#include "bindingeditor/bindingeditor.h" -#include "bindingeditor/actioneditor.h" -#include "annotationeditor/annotationeditor.h" -#include "qmlanchorbindingproxy.h" -#include "theme.h" -#include "aligndistribute.h" #include "propertyeditorcontextobject.h" -#include "tooltip.h" +#include "propertyeditorqmlbackend.h" +#include "propertyeditorvalue.h" +#include "qmlanchorbindingproxy.h" #include "richtexteditor/richtexteditorproxy.h" +#include "simplecolorpalettemodel.h" +#include "theme.h" +#include "tooltip.h" namespace QmlDesigner { @@ -71,6 +72,11 @@ void Quick2PropertyEditorView::registerQmlTypes() Tooltip::registerDeclarativeType(); EasingCurveEditor::registerDeclarativeType(); RichTextEditorProxy::registerDeclarativeType(); + QUrl regExpUrl = QUrl::fromLocalFile(PropertyEditorQmlBackend::propertyEditorResourcesPath() + + "/RegExpValidator.qml"); +#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0) + qmlRegisterType(regExpUrl, "HelperWidgets", 2, 0, "RegExpValidator"); +#endif } } diff --git a/src/plugins/qmldesigner/designercore/include/propertycontainer.h b/src/plugins/qmldesigner/designercore/include/propertycontainer.h index 5eebf01fbd4..ae38f9aafce 100644 --- a/src/plugins/qmldesigner/designercore/include/propertycontainer.h +++ b/src/plugins/qmldesigner/designercore/include/propertycontainer.h @@ -66,8 +66,4 @@ private: mutable QVariant m_value; }; -QMLDESIGNERCORE_EXPORT QDataStream &operator<<(QDataStream &stream, const QList &propertyContainerList); -QMLDESIGNERCORE_EXPORT QDataStream &operator>>(QDataStream &stream, QList &propertyContainerList); -QMLDESIGNERCORE_EXPORT QDebug operator<<(QDebug debug, QList &propertyContainerList); - } //namespace QmlDesigner diff --git a/src/plugins/qmldesigner/designercore/model/modelnode.cpp b/src/plugins/qmldesigner/designercore/model/modelnode.cpp index f196f5ca406..0e40c8edabe 100644 --- a/src/plugins/qmldesigner/designercore/model/modelnode.cpp +++ b/src/plugins/qmldesigner/designercore/model/modelnode.cpp @@ -203,7 +203,7 @@ static bool isIdToAvoid(const QString& id) static bool idContainsWrongLetter(const QString& id) { - static QRegularExpression idExpr(QStringLiteral("^[a-z_][a-zA-Z0-9_]+$")); + static QRegularExpression idExpr(QStringLiteral("^[a-z_][a-zA-Z0-9_]*$")); return !id.contains(idExpr); } diff --git a/src/plugins/qmldesigner/designercore/model/propertycontainer.cpp b/src/plugins/qmldesigner/designercore/model/propertycontainer.cpp index ae62b4d0e6c..440b91ffa08 100644 --- a/src/plugins/qmldesigner/designercore/model/propertycontainer.cpp +++ b/src/plugins/qmldesigner/designercore/model/propertycontainer.cpp @@ -113,37 +113,5 @@ QDebug operator<<(QDebug debug, const PropertyContainer &propertyContainer) return debug.space(); } -QDataStream &operator<<(QDataStream &stream, const QList &propertyContainerList) -{ - stream << propertyContainerList.count(); - foreach (const PropertyContainer &propertyContainer, propertyContainerList) - stream << propertyContainer; - - return stream; -} - -QDataStream &operator>>(QDataStream &stream, QList &propertyContainerList) -{ - int count; - stream >> count; - Q_ASSERT(count >= 0); - for ( int i = 0; i < count; i++) { - PropertyContainer propertyContainer; - stream >> propertyContainer; - propertyContainerList.append(propertyContainer); - } - - return stream; -} - -QDebug operator<<(QDebug debug, QList &propertyContainerList) -{ - foreach (const PropertyContainer &propertyContainer, propertyContainerList) - debug << propertyContainer; - - return debug.space(); -} - - } //namespace QmlDesigner diff --git a/src/plugins/qmldesigner/qmldesignerplugin.cpp b/src/plugins/qmldesigner/qmldesignerplugin.cpp index 91d760339be..489d7951c13 100644 --- a/src/plugins/qmldesigner/qmldesignerplugin.cpp +++ b/src/plugins/qmldesigner/qmldesignerplugin.cpp @@ -55,18 +55,19 @@ #include #include #include +#include #include #include #include #include -#include #include #include -#include -#include #include -#include +#include #include +#include +#include +#include #include #include @@ -211,7 +212,9 @@ QmlDesignerPlugin::~QmlDesignerPlugin() //////////////////////////////////////////////////// bool QmlDesignerPlugin::initialize(const QStringList & /*arguments*/, QString *errorMessage/* = 0*/) { - QDir{}.mkpath(Core::ICore::cacheResourcePath().toString()); + Sqlite::LibraryInitializer::initialize(); + + QDir{}.mkpath(Core::ICore::cacheResourcePath()); if (!Utils::HostOsInfo::canCreateOpenGLContext(errorMessage)) return false; @@ -219,10 +222,8 @@ bool QmlDesignerPlugin::initialize(const QStringList & /*arguments*/, QString *e if (DesignerSettings::getValue(DesignerSettingsKey::STANDALONE_MODE).toBool()) GenerateResource::generateMenuEntry(); - const QString fontPath - = Core::ICore::resourcePath( - "qmldesigner/propertyEditorQmlSources/imports/StudioTheme/icons.ttf") - .toString(); + QString fontPath = Core::ICore::resourcePath() + + QStringLiteral("/qmldesigner/propertyEditorQmlSources/imports/StudioTheme/icons.ttf"); if (QFontDatabase::addApplicationFont(fontPath) < 0) qCWarning(qmldesignerLog) << "Could not add font " << fontPath << "to font database"; diff --git a/src/plugins/qmlpreview/qmldebugtranslationclient.cpp b/src/plugins/qmlpreview/qmldebugtranslationclient.cpp index 3ef4351d498..e383784ef59 100644 --- a/src/plugins/qmlpreview/qmldebugtranslationclient.cpp +++ b/src/plugins/qmlpreview/qmldebugtranslationclient.cpp @@ -26,6 +26,8 @@ #include "qmldebugtranslationclient.h" #include +#include + #ifdef FOUND_QML_DEBUG_TRANSLATION_PROTOCOL #include #endif diff --git a/src/plugins/texteditor/codeassist/codeassistant.cpp b/src/plugins/texteditor/codeassist/codeassistant.cpp index 1c499956187..7439980df31 100644 --- a/src/plugins/texteditor/codeassist/codeassistant.cpp +++ b/src/plugins/texteditor/codeassist/codeassistant.cpp @@ -407,7 +407,7 @@ void CodeAssistantPrivate::finalizeProposal() bool CodeAssistantPrivate::isDisplayingProposal() const { - return m_proposalWidget != nullptr && m_proposalWidget->isVisible(); + return m_proposalWidget != nullptr && m_proposalWidget->proposalIsVisible(); } bool CodeAssistantPrivate::isWaitingForProposal() const diff --git a/src/plugins/texteditor/codeassist/functionhintproposalwidget.cpp b/src/plugins/texteditor/codeassist/functionhintproposalwidget.cpp index 149fba33fa3..e96071cec5a 100644 --- a/src/plugins/texteditor/codeassist/functionhintproposalwidget.cpp +++ b/src/plugins/texteditor/codeassist/functionhintproposalwidget.cpp @@ -219,10 +219,15 @@ void FunctionHintProposalWidget::closeProposal() abort(); } +bool FunctionHintProposalWidget::proposalIsVisible() const +{ + return d->m_popupFrame->isVisible(); +} + void FunctionHintProposalWidget::abort() { qApp->removeEventFilter(this); - if (d->m_popupFrame->isVisible()) + if (proposalIsVisible()) d->m_popupFrame->close(); deleteLater(); } diff --git a/src/plugins/texteditor/codeassist/functionhintproposalwidget.h b/src/plugins/texteditor/codeassist/functionhintproposalwidget.h index 99def1676a1..a37905bbad5 100644 --- a/src/plugins/texteditor/codeassist/functionhintproposalwidget.h +++ b/src/plugins/texteditor/codeassist/functionhintproposalwidget.h @@ -52,6 +52,8 @@ public: void updateProposal(const QString &prefix) override; void closeProposal() override; + bool proposalIsVisible() const override; + protected: bool eventFilter(QObject *o, QEvent *e) override; diff --git a/src/plugins/texteditor/codeassist/iassistproposalwidget.h b/src/plugins/texteditor/codeassist/iassistproposalwidget.h index 0b7fdc0dddf..d178fc4b5ed 100644 --- a/src/plugins/texteditor/codeassist/iassistproposalwidget.h +++ b/src/plugins/texteditor/codeassist/iassistproposalwidget.h @@ -57,6 +57,8 @@ public: virtual void updateProposal(const QString &prefix) = 0; virtual void closeProposal() = 0; + virtual bool proposalIsVisible() const { return isVisible(); } + int basePosition() const; void setBasePosition(int basePosition); diff --git a/tests/unit/unittest/CMakeLists.txt b/tests/unit/unittest/CMakeLists.txt index 5a2c21c03cc..49050ae2f37 100644 --- a/tests/unit/unittest/CMakeLists.txt +++ b/tests/unit/unittest/CMakeLists.txt @@ -225,6 +225,14 @@ endforeach() extend_qtc_test_with_target_sources(Sqlite) +extend_qtc_test(unittest DEFINES _HAVE_SQLITE_CONFIG_H) + +if (APPLE) +extend_qtc_test(unittest DEFINES _BSD_SOURCE) +elseif (UNIX) +extend_qtc_test(unittest DEFINES _POSIX_C_SOURCE=200809L _GNU_SOURCE _DEFAULT_SOURCE) +endif() + # Do not work on the source directory data add_custom_command(TARGET unittest POST_BUILD COMMAND "${CMAKE_COMMAND}" -E copy_directory diff --git a/tests/unit/unittest/gtest-creator-printing.cpp b/tests/unit/unittest/gtest-creator-printing.cpp index f0f6a04ad76..af8fc6c0eb4 100644 --- a/tests/unit/unittest/gtest-creator-printing.cpp +++ b/tests/unit/unittest/gtest-creator-printing.cpp @@ -72,7 +72,7 @@ #include #include -#include +#include namespace { ClangBackEnd::FilePathCaching *filePathCache = nullptr; diff --git a/tests/unit/unittest/sqlitedatabasebackend-test.cpp b/tests/unit/unittest/sqlitedatabasebackend-test.cpp index 2390bd4a3c6..f6c6a62bb2a 100644 --- a/tests/unit/unittest/sqlitedatabasebackend-test.cpp +++ b/tests/unit/unittest/sqlitedatabasebackend-test.cpp @@ -30,7 +30,7 @@ #include #include -#include +#include #include diff --git a/tests/unit/unittest/sqlitetable-test.cpp b/tests/unit/unittest/sqlitetable-test.cpp index b916ad5bc89..2faebd844cd 100644 --- a/tests/unit/unittest/sqlitetable-test.cpp +++ b/tests/unit/unittest/sqlitetable-test.cpp @@ -93,7 +93,9 @@ TEST_F(SqliteTable, InitializeTable) table.addColumn("name"); table.addColumn("value"); - EXPECT_CALL(databaseMock, execute(Eq("CREATE TEMPORARY TABLE IF NOT EXISTS testTable(name NUMERIC, value NUMERIC) WITHOUT ROWID"))); + EXPECT_CALL(databaseMock, + execute(Eq( + "CREATE TEMPORARY TABLE IF NOT EXISTS testTable(name, value) WITHOUT ROWID"))); table.initialize(databaseMock); } @@ -107,7 +109,7 @@ TEST_F(SqliteTable, InitializeTableWithIndex) table.addIndex({column}); table.addIndex({column2}); - EXPECT_CALL(databaseMock, execute(Eq("CREATE TABLE testTable(name NUMERIC, value NUMERIC)"))); + EXPECT_CALL(databaseMock, execute(Eq("CREATE TABLE testTable(name, value)"))); EXPECT_CALL(databaseMock, execute(Eq("CREATE INDEX IF NOT EXISTS index_testTable_name ON testTable(name)"))); EXPECT_CALL(databaseMock, execute(Eq("CREATE INDEX IF NOT EXISTS index_testTable_value ON testTable(value)"))); @@ -299,9 +301,7 @@ TEST_F(SqliteTable, AddPrimaryTableContraint) const auto &nameColumn = table.addColumn("name"); table.addPrimaryKeyContraint({idColumn, nameColumn}); - EXPECT_CALL(databaseMock, - execute( - Eq("CREATE TABLE testTable(id NUMERIC, name NUMERIC, PRIMARY KEY(id, name))"))); + EXPECT_CALL(databaseMock, execute(Eq("CREATE TABLE testTable(id, name, PRIMARY KEY(id, name))"))); table.initialize(databaseMock); } diff --git a/tests/unit/unittest/sqlstatementbuilder-test.cpp b/tests/unit/unittest/sqlstatementbuilder-test.cpp index 654439fbcd0..1e24faeded1 100644 --- a/tests/unit/unittest/sqlstatementbuilder-test.cpp +++ b/tests/unit/unittest/sqlstatementbuilder-test.cpp @@ -137,11 +137,11 @@ TEST(SqlStatementBuilder, ClearBinding) TEST(SqlStatementBuilder, ColumnType) { - ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Numeric), "NUMERIC"); - ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Integer), "INTEGER"); - ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Real), "REAL"); - ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Text), "TEXT"); - ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Blob), "BLOB"); + ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Numeric), " NUMERIC"); + ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Integer), " INTEGER"); + ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Real), " REAL"); + ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Text), " TEXT"); + ASSERT_THAT(SqlStatementBuilder::columnTypeToString(ColumnType::Blob), " BLOB"); ASSERT_TRUE(SqlStatementBuilder::columnTypeToString(ColumnType::None).isEmpty()); } diff --git a/tests/unit/unittest/unittest.pro b/tests/unit/unittest/unittest.pro index 0f6c0d3bcee..d00920fdb5e 100644 --- a/tests/unit/unittest/unittest.pro +++ b/tests/unit/unittest/unittest.pro @@ -37,6 +37,9 @@ QMAKE_LFLAGS += -fno-merge-debug-strings -fuse-ld=gold CONFIG(release, debug|release):QMAKE_LFLAGS += -Wl,--strip-debug } +CONFIG(debug, debug|release): DEFINES += SQLITE_DEBUG + + gcc:!clang: QMAKE_CXXFLAGS += -Wno-noexcept-type msvc{ QMAKE_CXXFLAGS += /bigobj /wd4267 /wd4141 /wd4146 /wd4624 diff --git a/tests/unit/unittest/unittests-main.cpp b/tests/unit/unittest/unittests-main.cpp index 26993dd9ded..5f5146de794 100644 --- a/tests/unit/unittest/unittests-main.cpp +++ b/tests/unit/unittest/unittests-main.cpp @@ -26,6 +26,8 @@ #include "googletest.h" #include +#include + #include #include @@ -52,6 +54,7 @@ public: int main(int argc, char *argv[]) { + Sqlite::LibraryInitializer::initialize(); Sqlite::Database::activateLogging(); QGuiApplication application(argc, argv);