From 29b55dcb830ad5f6f12a8918d267efca33d3af61 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Mon, 20 Mar 2023 10:51:56 +0100 Subject: QCryptographicHash: constexpr ALL of QSmallByteArray MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ... because we can. Pick-to: 6.5 Change-Id: I03872a69ac4625ca73b0a7f0310a2a951615b000 Reviewed-by: Qt CI Bot Reviewed-by: Giuseppe D'Angelo Reviewed-by: MÃ¥rten Nordheim --- src/corelib/tools/qcryptographichash.cpp | 38 ++++++++++++++++---------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/src/corelib/tools/qcryptographichash.cpp b/src/corelib/tools/qcryptographichash.cpp index dbb2448c1f..2c4a2b276b 100644 --- a/src/corelib/tools/qcryptographichash.cpp +++ b/src/corelib/tools/qcryptographichash.cpp @@ -132,19 +132,19 @@ public: QSmallByteArray() = default; // all compiler-generated SMFs are ok! template = true> // M == N is for copy ctor! - QSmallByteArray(const QSmallByteArray &other) noexcept + constexpr QSmallByteArray(const QSmallByteArray &other) noexcept { assign(other); } template = true> // M == N is for copy-assignment op! - QSmallByteArray &operator=(const QSmallByteArray &other) noexcept + constexpr QSmallByteArray &operator=(const QSmallByteArray &other) noexcept { assign(other); return *this; } template // ### underconstrained - void assign(const Container &c) + constexpr void assign(const Container &c) { const size_t otherSize = size_t(std::size(c)); Q_ASSERT(otherSize < N); @@ -152,43 +152,43 @@ public: m_size = quint8(otherSize); } - quint8 *data() noexcept { return m_data.data(); } - const quint8 *data() const noexcept { return m_data.data(); } - qsizetype size() const noexcept { return qsizetype{m_size}; } - quint8 &operator[](qsizetype n) + constexpr quint8 *data() noexcept { return m_data.data(); } + constexpr const quint8 *data() const noexcept { return m_data.data(); } + constexpr qsizetype size() const noexcept { return qsizetype{m_size}; } + constexpr quint8 &operator[](qsizetype n) { Q_ASSERT(n < size()); return data()[n]; } - const quint8 &operator[](qsizetype n) const + constexpr const quint8 &operator[](qsizetype n) const { Q_ASSERT(n < size()); return data()[n]; } - bool isEmpty() const noexcept { return size() == 0; } - void clear() noexcept { m_size = 0; } - void resizeForOverwrite(qsizetype s) + constexpr bool isEmpty() const noexcept { return size() == 0; } + constexpr void clear() noexcept { m_size = 0; } + constexpr void resizeForOverwrite(qsizetype s) { Q_ASSERT(s >= 0); Q_ASSERT(size_t(s) <= N); m_size = std::uint8_t(s); } - void resize(qsizetype s, quint8 v) + constexpr void resize(qsizetype s, quint8 v) { const auto oldSize = size(); resizeForOverwrite(s); if (s > oldSize) memset(data() + oldSize, v, size() - oldSize); } - QByteArrayView toByteArrayView() const noexcept + constexpr QByteArrayView toByteArrayView() const noexcept { return *this; } - auto begin() noexcept { return data(); } - auto begin() const noexcept { return data(); } - auto cbegin() const noexcept { return begin(); } - auto end() noexcept { return data() + size(); } - auto end() const noexcept { return data() + size(); } - auto cend() const noexcept { return end(); } + constexpr auto begin() noexcept { return data(); } + constexpr auto begin() const noexcept { return data(); } + constexpr auto cbegin() const noexcept { return begin(); } + constexpr auto end() noexcept { return data() + size(); } + constexpr auto end() const noexcept { return data() + size(); } + constexpr auto cend() const noexcept { return end(); } }; static constexpr int hashLengthInternal(QCryptographicHash::Algorithm method) noexcept -- cgit v1.2.3