From 0475460102411979ab51c365cb2bec2c05b69cb4 Mon Sep 17 00:00:00 2001 From: Jarek Kobus Date: Mon, 22 Jun 2020 11:06:12 +0200 Subject: Use QList instead of QVector in corelib Applied to headers only. Source file to be changed separately. Omitted statemachine for now to avoid conflicts. Omitted qmetatype.h for now - to be handled later. Task-number: QTBUG-84469 Change-Id: I317376037a62467c313467d92955ad0b7473aa97 Reviewed-by: Sona Kurazyan --- src/corelib/tools/qversionnumber.h | 30 +++++++++++++----------------- 1 file changed, 13 insertions(+), 17 deletions(-) (limited to 'src/corelib/tools/qversionnumber.h') diff --git a/src/corelib/tools/qversionnumber.h b/src/corelib/tools/qversionnumber.h index f4e384c8dc..f8a2d8eae8 100644 --- a/src/corelib/tools/qversionnumber.h +++ b/src/corelib/tools/qversionnumber.h @@ -42,10 +42,10 @@ #ifndef QVERSIONNUMBER_H #define QVERSIONNUMBER_H +#include +#include #include #include -#include -#include #include #include @@ -84,24 +84,24 @@ class QVersionNumber union { quintptr dummy; qint8 inline_segments[sizeof(void*)]; - QVector *pointer_segments; + QList *pointer_segments; }; // set the InlineSegmentMarker and set length to zero SegmentStorage() noexcept : dummy(1) {} - SegmentStorage(const QVector &seg) + SegmentStorage(const QList &seg) { if (dataFitsInline(seg.begin(), seg.size())) setInlineData(seg.begin(), seg.size()); else - pointer_segments = new QVector(seg); + pointer_segments = new QList(seg); } SegmentStorage(const SegmentStorage &other) { if (other.isUsingPointer()) - pointer_segments = new QVector(*other.pointer_segments); + pointer_segments = new QList(*other.pointer_segments); else dummy = other.dummy; } @@ -111,7 +111,7 @@ class QVersionNumber if (isUsingPointer() && other.isUsingPointer()) { *pointer_segments = *other.pointer_segments; } else if (other.isUsingPointer()) { - pointer_segments = new QVector(*other.pointer_segments); + pointer_segments = new QList(*other.pointer_segments); } else { if (isUsingPointer()) delete pointer_segments; @@ -132,19 +132,19 @@ class QVersionNumber return *this; } - explicit SegmentStorage(QVector &&seg) + explicit SegmentStorage(QList &&seg) { if (dataFitsInline(seg.begin(), seg.size())) setInlineData(seg.begin(), seg.size()); else - pointer_segments = new QVector(std::move(seg)); + pointer_segments = new QList(std::move(seg)); } SegmentStorage(std::initializer_list args) { if (dataFitsInline(args.begin(), int(args.size()))) { setInlineData(args.begin(), int(args.size())); } else { - pointer_segments = new QVector(args); + pointer_segments = new QList(args); } } @@ -218,15 +218,11 @@ public: inline QVersionNumber() noexcept : m_segments() {} - inline explicit QVersionNumber(const QVector &seg) - : m_segments(seg) - {} + inline explicit QVersionNumber(const QList &seg) : m_segments(seg) { } // compiler-generated copy/move ctor/assignment operators and the destructor are ok - explicit QVersionNumber(QVector &&seg) - : m_segments(std::move(seg)) - {} + explicit QVersionNumber(QList &&seg) : m_segments(std::move(seg)) { } inline QVersionNumber(std::initializer_list args) : m_segments(args) @@ -258,7 +254,7 @@ public: Q_REQUIRED_RESULT Q_CORE_EXPORT QVersionNumber normalized() const; - Q_REQUIRED_RESULT Q_CORE_EXPORT QVector segments() const; + Q_REQUIRED_RESULT Q_CORE_EXPORT QList segments() const; Q_REQUIRED_RESULT inline int segmentAt(int index) const noexcept { return (m_segments.size() > index) ? m_segments.at(index) : 0; } -- cgit v1.2.3