From 02dfec920951afc1c512aa41fd10b025ae1fbbb2 Mon Sep 17 00:00:00 2001 From: Lars Knoll Date: Wed, 29 Jul 2020 15:04:23 +0200 Subject: Remove QVariant(QDataStream) constructor The constructor wasn't constructing a QVariant holding a QDataStream, bug loading the variant from the datastream. This is opposed to how QDataStream works in all other places, and also not in line with all other constructors for QVariant. [ChangeLog][QtCore][QVariant] Removed the QVariant(QDataStream) constructor. Use QVariant().load(datastream) instead. Change-Id: I6ac54e12166b9383c09c3940eef2516f896fd359 Reviewed-by: Qt CI Bot Reviewed-by: Thiago Macieira --- src/corelib/kernel/qvariant.h | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/corelib/kernel/qvariant.h') diff --git a/src/corelib/kernel/qvariant.h b/src/corelib/kernel/qvariant.h index ae4c26ed7a..e2394fc469 100644 --- a/src/corelib/kernel/qvariant.h +++ b/src/corelib/kernel/qvariant.h @@ -177,10 +177,6 @@ class Q_CORE_EXPORT QVariant explicit QVariant(QMetaType type, const void *copy = nullptr); QVariant(const QVariant &other); -#ifndef QT_NO_DATASTREAM - explicit QVariant(QDataStream &s); -#endif - QVariant(int i); QVariant(uint ui); QVariant(qlonglong ll); -- cgit v1.2.3