From eb1f41b8c186614392cff5b7b51c86e5f5c4e2b6 Mon Sep 17 00:00:00 2001 From: "Bradley T. Hughes" Date: Mon, 12 Sep 2011 08:44:32 +0200 Subject: [PATCH] Don't call ref() in QByteArray::QByteArray() The call is unnecessary; it does nothing since the shared_null ref count is negative. Change-Id: I0d0c35a554e4fc5b734a25dab06f04bee7a9ae24 Reviewed-on: http://codereview.qt-project.org/4637 Reviewed-by: Qt Sanity Bot Reviewed-by: Oswald Buddenhagen --- src/corelib/tools/qbytearray.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/corelib/tools/qbytearray.h b/src/corelib/tools/qbytearray.h index b70dba4d55..fdd34b3935 100644 --- a/src/corelib/tools/qbytearray.h +++ b/src/corelib/tools/qbytearray.h @@ -402,7 +402,7 @@ public: inline DataPtr &data_ptr() { return d; } }; -inline QByteArray::QByteArray(): d(const_cast(&shared_null.ba)) { d->ref.ref(); } +inline QByteArray::QByteArray(): d(const_cast(&shared_null.ba)) { } inline QByteArray::~QByteArray() { if (!d->ref.deref()) qFree(d); } inline int QByteArray::size() const { return d->size; }