From 5b57ce06a8178e1d62ed9a3ebea417864e263fa6 Mon Sep 17 00:00:00 2001 From: Edward Welbourne Date: Mon, 26 Oct 2020 13:23:55 +0100 Subject: Resolve corelib's remaining ### Qt 6 comments No action taken at Qt 6, suggesting it shall never happen. Four removed, one converted to Qt 7, others converted to unversioned TODOs. Filed Jira tasks, and referenced in comments, for those retained. There remain two "once bootstrap builds are obsolete" comments and one other on which pending action may yet happen. Fixes: QTBUG-85700 Change-Id: Ib140a6a21c63370e51e4734cc591f67573a29d9a Reviewed-by: Volker Hilsheimer --- src/corelib/global/qglobal.h | 2 +- src/corelib/io/qfileinfo.cpp | 1 - src/corelib/io/qiodevice.h | 2 +- src/corelib/itemmodels/qabstractitemmodel.cpp | 2 +- src/corelib/kernel/qcoreapplication.cpp | 4 ++-- src/corelib/kernel/qmetaobject_p.h | 3 +-- src/corelib/thread/qthread.cpp | 2 -- src/corelib/time/qdatetime.h | 1 - src/corelib/tools/qsharedpointer_impl.h | 10 +++++----- 9 files changed, 11 insertions(+), 16 deletions(-) diff --git a/src/corelib/global/qglobal.h b/src/corelib/global/qglobal.h index d1f00234eb..5856ddfb01 100644 --- a/src/corelib/global/qglobal.h +++ b/src/corelib/global/qglobal.h @@ -1160,7 +1160,7 @@ template inline auto qGetPtrHelper(Ptr &ptr) noexcept -> decltype #define QT_TRANSLATE_NOOP3(scope, x, comment) {x, comment} #define QT_TRANSLATE_NOOP3_UTF8(scope, x, comment) {x, comment} -#ifndef QT_NO_TRANSLATION // ### Qt6: This should enclose the NOOPs above +#ifndef QT_NO_TRANSLATION #define QT_TR_N_NOOP(x) x #define QT_TRANSLATE_N_NOOP(scope, x) x diff --git a/src/corelib/io/qfileinfo.cpp b/src/corelib/io/qfileinfo.cpp index 7e1e62a775..8b80141a32 100644 --- a/src/corelib/io/qfileinfo.cpp +++ b/src/corelib/io/qfileinfo.cpp @@ -890,7 +890,6 @@ QString QFileInfo::suffix() const QDir QFileInfo::dir() const { Q_D(const QFileInfo); - // ### Qt 6: Maybe rename this to parentDirectory(), considering what it actually does? return QDir(d->fileEntry.path()); } diff --git a/src/corelib/io/qiodevice.h b/src/corelib/io/qiodevice.h index fa43efdb4d..e4388b8972 100644 --- a/src/corelib/io/qiodevice.h +++ b/src/corelib/io/qiodevice.h @@ -98,7 +98,7 @@ public: virtual bool open(OpenMode mode); virtual void close(); - // ### Qt 6: pos() and seek() should not be virtual, and + // ### Qt 7 - QTBUG-76492: pos() and seek() should not be virtual, and // ### seek() should call a virtual seekData() function. virtual qint64 pos() const; virtual qint64 size() const; diff --git a/src/corelib/itemmodels/qabstractitemmodel.cpp b/src/corelib/itemmodels/qabstractitemmodel.cpp index 521fd69fbc..7f842e29a6 100644 --- a/src/corelib/itemmodels/qabstractitemmodel.cpp +++ b/src/corelib/itemmodels/qabstractitemmodel.cpp @@ -2104,7 +2104,7 @@ bool QAbstractItemModel::clearItemData(const QModelIndex &index) */ bool QAbstractItemModel::setItemData(const QModelIndex &index, const QMap &roles) { - // ### Qt 6: Consider change the semantics of this function, + // ### TODO: Consider change the semantics of this function, // or deprecating/removing it altogether. // // For instance, it should try setting *all* the data diff --git a/src/corelib/kernel/qcoreapplication.cpp b/src/corelib/kernel/qcoreapplication.cpp index 44d8b8100a..38cc445e2a 100644 --- a/src/corelib/kernel/qcoreapplication.cpp +++ b/src/corelib/kernel/qcoreapplication.cpp @@ -1282,7 +1282,7 @@ void QCoreApplication::processEvents(QEventLoop::ProcessEventsFlags flags) */ void QCoreApplication::processEvents(QEventLoop::ProcessEventsFlags flags, int ms) { - // ### Qt 6: consider splitting this method into a public and a private + // ### TODO: consider splitting this method into a public and a private // one, so that a user-invoked processEvents can be detected // and handled properly. QThreadData *data = QThreadData::current(); @@ -1645,7 +1645,7 @@ bool QCoreApplication::compressEvent(QEvent *event, QObject *receiver, QPostEven */ void QCoreApplication::sendPostedEvents(QObject *receiver, int event_type) { - // ### Qt 6: consider splitting this method into a public and a private + // ### TODO: consider splitting this method into a public and a private // one, so that a user-invoked sendPostedEvents can be detected // and handled properly. QThreadData *data = QThreadData::current(); diff --git a/src/corelib/kernel/qmetaobject_p.h b/src/corelib/kernel/qmetaobject_p.h index 574acafd1f..f98185ac81 100644 --- a/src/corelib/kernel/qmetaobject_p.h +++ b/src/corelib/kernel/qmetaobject_p.h @@ -62,8 +62,7 @@ #include QT_BEGIN_NAMESPACE -// ### TODO Qt6: add a proper namespace with Q_NAMESPACE and use scoped enums -// A namespace and scoped are needed to avoid enum clashes +// ### TODO - QTBUG-87869: wrap in a proper Q_NAMESPACE and use scoped enums, to avoid name clashes enum PropertyFlags { Invalid = 0x00000000, diff --git a/src/corelib/thread/qthread.cpp b/src/corelib/thread/qthread.cpp index 5f80c73ba8..70ec71c26d 100644 --- a/src/corelib/thread/qthread.cpp +++ b/src/corelib/thread/qthread.cpp @@ -1025,8 +1025,6 @@ void QThread::requestInterruption() return; } Q_D(QThread); - // ### Qt 6: use std::atomic_flag, and document that - // requestInterruption/isInterruptionRequested do not synchronize with each other QMutexLocker locker(&d->mutex); if (!d->running || d->finished || d->isInFinish) return; diff --git a/src/corelib/time/qdatetime.h b/src/corelib/time/qdatetime.h index 91f63605f2..fc43a1b302 100644 --- a/src/corelib/time/qdatetime.h +++ b/src/corelib/time/qdatetime.h @@ -236,7 +236,6 @@ class QDateTimePrivate; class Q_CORE_EXPORT QDateTime { - // ### Qt 6: revisit the optimization struct ShortData { #if Q_BYTE_ORDER == Q_LITTLE_ENDIAN quintptr status : 8; diff --git a/src/corelib/tools/qsharedpointer_impl.h b/src/corelib/tools/qsharedpointer_impl.h index affd85d88d..53d9a34acb 100644 --- a/src/corelib/tools/qsharedpointer_impl.h +++ b/src/corelib/tools/qsharedpointer_impl.h @@ -1,6 +1,6 @@ /**************************************************************************** ** -** Copyright (C) 2019 The Qt Company Ltd. +** Copyright (C) 2020 The Qt Company Ltd. ** Copyright (C) 2020 Intel Corporation. ** Copyright (C) 2019 Klarälvdalens Datakonsult AB. ** Contact: https://www.qt.io/licensing/ @@ -692,8 +692,8 @@ public: value = actual; } - // ### Qt 6: remove users of this API; no one should ever access - // a weak pointer's data but the weak pointer itself + // ### TODO - QTBUG-88102: remove all users of this API; no one should ever + // access a weak pointer's data but the weak pointer itself inline T *internalData() const noexcept { return d == nullptr || d->strongref.loadRelaxed() == 0 ? nullptr : value; @@ -889,8 +889,8 @@ qobject_cast(const QWeakPointer &src) return qSharedPointerObjectCast::Type, T>(src); } -/// ### Qt 6: make this use toStrongRef() (once support for storing -/// non-managed QObjects in QWeakPointer is removed) +/// ### TODO - QTBUG-88102: make this use toStrongRef() (once support for +/// storing non-managed QObjects in QWeakPointer is removed) template QWeakPointer::Value, T>::type> qWeakPointerFromVariant(const QVariant &variant) -- cgit v1.2.3