From 21a4a2daf878dfb39523e444e56c4a924a5bed0c Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Tue, 27 Feb 2024 14:49:42 +0100 Subject: [PATCH] QCborStreamReader: don't assume that clear() results in isNull() That QString and QByteArray::clear() shed all capacity() is a historical artifact we will fix at some point. Don't add more code that assumes a clear()ed string isNull(). Amends 8af346c1f66f813c3c8fe4d8b892ecfbe96eacfb. Amends 1d9137e13f9eb3f183c967e9e911c5b260f93dc0. Task-number: QTBUG-31283 Task-number: QTBUG-60745 Change-Id: Ib0d929325088d3e8e119fee3eafa964a783dc8e9 Reviewed-by: Ivan Solovev Reviewed-by: Thiago Macieira Reviewed-by: Juha Vuolle (cherry picked from commit 6c424dbcb0385a4b36835fc5103ca74c1044eccc) Reviewed-by: Qt Cherry-pick Bot --- src/corelib/serialization/qcborstreamreader.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/corelib/serialization/qcborstreamreader.h b/src/corelib/serialization/qcborstreamreader.h index f5e8fd689fa..f2bd2a614b1 100644 --- a/src/corelib/serialization/qcborstreamreader.h +++ b/src/corelib/serialization/qcborstreamreader.h @@ -150,21 +150,21 @@ public: { QString dst; if (!appendToString(dst)) - dst.clear(); + dst = QString{}; return dst; } QByteArray toUtf8String() { QByteArray dst; if (!appendToUtf8String(dst)) - dst.clear(); + dst = QByteArray{}; return dst; } QByteArray toByteArray() { QByteArray dst; if (!appendToByteArray(dst)) - dst.clear(); + dst = QByteArray{}; return dst; }