import qt5-qtdeclarative-5.15.3-1.el9
This commit is contained in:
parent
34b48d1cc0
commit
e8cb50e584
2
.gitignore
vendored
2
.gitignore
vendored
@ -1 +1 @@
|
|||||||
SOURCES/qtdeclarative-everywhere-src-5.15.2.tar.xz
|
SOURCES/qtdeclarative-everywhere-opensource-src-5.15.3.tar.xz
|
||||||
|
@ -1 +1 @@
|
|||||||
cde54101a922dc92028a26e8b07149a83b2eaae2 SOURCES/qtdeclarative-everywhere-src-5.15.2.tar.xz
|
0384e6be3138fecbe858adc4d9fdb83970c033e6 SOURCES/qtdeclarative-everywhere-opensource-src-5.15.3.tar.xz
|
||||||
|
@ -1,7 +1,7 @@
|
|||||||
From 6d7dfed857ab9e83ab2a917a929ce3b25342d90a Mon Sep 17 00:00:00 2001
|
From a6ec2f93dba2a3e4ac07332c5adf4456348bf4a1 Mon Sep 17 00:00:00 2001
|
||||||
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
||||||
Date: Fri, 21 May 2021 13:30:41 +0200
|
Date: Fri, 21 May 2021 13:30:41 +0200
|
||||||
Subject: [PATCH 25/28] Give a warning when StyledText encounters a non
|
Subject: [PATCH 01/20] Give a warning when StyledText encounters a non
|
||||||
supported entity
|
supported entity
|
||||||
|
|
||||||
Pick-to: 6.1 5.15
|
Pick-to: 6.1 5.15
|
||||||
@ -35,5 +35,5 @@ index 660852ba83..d531fc9205 100644
|
|||||||
} else if (*ch == QLatin1Char(' ')) {
|
} else if (*ch == QLatin1Char(' ')) {
|
||||||
QStringRef entity(&textIn, entityStart - 1, entityLength + 1);
|
QStringRef entity(&textIn, entityStart - 1, entityLength + 1);
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 4f08a2da5b0da675cf6a75683a43a106f5a1e7b8 Mon Sep 17 00:00:00 2001
|
From 0eb5ff2e97713e12318c00bab9f3605abb8592c2 Mon Sep 17 00:00:00 2001
|
||||||
From: Antonio Rojas <arojas@archlinux.org>
|
From: Antonio Rojas <arojas@archlinux.org>
|
||||||
Date: Sun, 23 May 2021 14:32:46 +0200
|
Date: Sun, 23 May 2021 14:32:46 +0200
|
||||||
Subject: [PATCH 26/28] Add missing limits include to fix build with GCC 11
|
Subject: [PATCH 02/20] Add missing limits include to fix build with GCC 11
|
||||||
|
|
||||||
This is not required for Qt 6, since it is indirectly pulled via
|
This is not required for Qt 6, since it is indirectly pulled via
|
||||||
qanystrigview.h, but it is for Qt 5 (where qanystrigview does
|
qanystrigview.h, but it is for Qt 5 (where qanystrigview does
|
||||||
@ -29,5 +29,5 @@ index a7e37d1964..01b2f58f16 100644
|
|||||||
|
|
||||||
//
|
//
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From ebc0daeea59af400601e6207cd9939c746fccdc7 Mon Sep 17 00:00:00 2001
|
From 45621c00cae6b108b412e63388588be8bc4ccc44 Mon Sep 17 00:00:00 2001
|
||||||
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
||||||
Date: Fri, 21 May 2021 13:17:15 +0200
|
Date: Fri, 21 May 2021 13:17:15 +0200
|
||||||
Subject: [PATCH 27/28] Document that StyledText also supports and
|
Subject: [PATCH 03/20] Document that StyledText also supports and
|
||||||
"
|
"
|
||||||
|
|
||||||
Change-Id: I1715f8ae8ec8d0fbaf6dbe2b8663cc169da663cd
|
Change-Id: I1715f8ae8ec8d0fbaf6dbe2b8663cc169da663cd
|
||||||
@ -12,7 +12,7 @@ Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquicktext.cpp b/src/quick/items/qquicktext.cpp
|
diff --git a/src/quick/items/qquicktext.cpp b/src/quick/items/qquicktext.cpp
|
||||||
index 581ab9f76a..26840b99e3 100644
|
index b18d03a20a..43c29d1f58 100644
|
||||||
--- a/src/quick/items/qquicktext.cpp
|
--- a/src/quick/items/qquicktext.cpp
|
||||||
+++ b/src/quick/items/qquicktext.cpp
|
+++ b/src/quick/items/qquicktext.cpp
|
||||||
@@ -2168,7 +2168,7 @@ void QQuickText::resetMaximumLineCount()
|
@@ -2168,7 +2168,7 @@ void QQuickText::resetMaximumLineCount()
|
||||||
@ -25,5 +25,5 @@ index 581ab9f76a..26840b99e3 100644
|
|||||||
|
|
||||||
\c Text.StyledText parser is strict, requiring tags to be correctly nested.
|
\c Text.StyledText parser is strict, requiring tags to be correctly nested.
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 0dda47d9f1a22567ad8f1266be2f0cd8a9226c7f Mon Sep 17 00:00:00 2001
|
From bac7823a97ab901abc8da392ba23747feac4c933 Mon Sep 17 00:00:00 2001
|
||||||
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
||||||
Date: Fri, 21 May 2021 13:42:35 +0200
|
Date: Fri, 21 May 2021 13:42:35 +0200
|
||||||
Subject: [PATCH 28/28] Support ' in styled text
|
Subject: [PATCH 04/20] Support ' in styled text
|
||||||
|
|
||||||
Pick-to: 6.1 5.15
|
Pick-to: 6.1 5.15
|
||||||
Change-Id: I4a8db963e52a7899ab1796f9a560e8029cc1c929
|
Change-Id: I4a8db963e52a7899ab1796f9a560e8029cc1c929
|
||||||
@ -14,7 +14,7 @@ Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
|
|||||||
2 files changed, 3 insertions(+), 1 deletion(-)
|
2 files changed, 3 insertions(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquicktext.cpp b/src/quick/items/qquicktext.cpp
|
diff --git a/src/quick/items/qquicktext.cpp b/src/quick/items/qquicktext.cpp
|
||||||
index 26840b99e3..4ddd2a41bc 100644
|
index 43c29d1f58..459dea8f74 100644
|
||||||
--- a/src/quick/items/qquicktext.cpp
|
--- a/src/quick/items/qquicktext.cpp
|
||||||
+++ b/src/quick/items/qquicktext.cpp
|
+++ b/src/quick/items/qquicktext.cpp
|
||||||
@@ -2168,7 +2168,7 @@ void QQuickText::resetMaximumLineCount()
|
@@ -2168,7 +2168,7 @@ void QQuickText::resetMaximumLineCount()
|
||||||
@ -40,5 +40,5 @@ index d531fc9205..a25af90414 100644
|
|||||||
textOut += QChar(34);
|
textOut += QChar(34);
|
||||||
else if (entity == QLatin1String("nbsp"))
|
else if (entity == QLatin1String("nbsp"))
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,51 +0,0 @@
|
|||||||
From 4ee1462686dc12eb463f5ba6b378d43a39aed074 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Date: Thu, 12 Nov 2020 12:11:29 +0100
|
|
||||||
Subject: [PATCH 05/28] QQuickView docs: show correct usage of
|
|
||||||
setInitialProperties
|
|
||||||
|
|
||||||
Change-Id: If63f4c59f18bc0754ce2e68e424f6efd0f512d30
|
|
||||||
Reviewed-by: Mitch Curtis <mitch.curtis@qt.io>
|
|
||||||
(cherry picked from commit 54d4f8f526f9c9a1af702b14925e1d34ee8b2134)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/quick/doc/snippets/qquickview-ex.cpp | 9 +++++++++
|
|
||||||
src/quick/items/qquickview.cpp | 4 ++++
|
|
||||||
2 files changed, 13 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/quick/doc/snippets/qquickview-ex.cpp b/src/quick/doc/snippets/qquickview-ex.cpp
|
|
||||||
index 32406f8f2f..5f93dfdbe8 100644
|
|
||||||
--- a/src/quick/doc/snippets/qquickview-ex.cpp
|
|
||||||
+++ b/src/quick/doc/snippets/qquickview-ex.cpp
|
|
||||||
@@ -59,3 +59,12 @@ int main(int argc, char *argv[])
|
|
||||||
return app.exec();
|
|
||||||
}
|
|
||||||
//![0]
|
|
||||||
+
|
|
||||||
+void makeDocTeamHappyByKeepingExampleCompilable() {
|
|
||||||
+//![1]
|
|
||||||
+ QScopedPointer<QQuickView> view { new QQuickView };
|
|
||||||
+ view->setInitialProperties({"x, 100"}, {"width", 50});
|
|
||||||
+ view->setSource(QUrl::fromLocalFile("myqmlfile.qml"));
|
|
||||||
+ view->show();
|
|
||||||
+//![1]
|
|
||||||
+}
|
|
||||||
diff --git a/src/quick/items/qquickview.cpp b/src/quick/items/qquickview.cpp
|
|
||||||
index 97f6689d8a..b3a5270e9b 100644
|
|
||||||
--- a/src/quick/items/qquickview.cpp
|
|
||||||
+++ b/src/quick/items/qquickview.cpp
|
|
||||||
@@ -240,7 +240,11 @@ void QQuickView::setSource(const QUrl& url)
|
|
||||||
Sets the initial properties \a initialProperties with which the QML
|
|
||||||
component gets initialized after calling \l QQuickView::setSource().
|
|
||||||
|
|
||||||
+ \snippet qquickview-ex.cpp 1
|
|
||||||
+
|
|
||||||
\note You can only use this function to initialize top-level properties.
|
|
||||||
+ \note This function should always be called before setSource, as it has
|
|
||||||
+ no effect once the component has become \c Ready.
|
|
||||||
|
|
||||||
\sa QQmlComponent::createWithInitialProperties()
|
|
||||||
\since 5.14
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 6683c414c5cc6ab46197c41bb1361c518ca84d3e Mon Sep 17 00:00:00 2001
|
From d7d355175aa1e625e5b59b4494e37aeac9c849d5 Mon Sep 17 00:00:00 2001
|
||||||
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
From: Albert Astals Cid <albert.astals.cid@kdab.com>
|
||||||
Date: Thu, 17 Jun 2021 16:32:28 +0200
|
Date: Thu, 17 Jun 2021 16:32:28 +0200
|
||||||
Subject: [PATCH 29/29] Remove unused QPointer<QQuickPointerMask>
|
Subject: [PATCH 05/20] Remove unused QPointer<QQuickPointerMask>
|
||||||
|
|
||||||
Change-Id: I009fa6bbd8599dc3bb2e810176fe20e70ed50851
|
Change-Id: I009fa6bbd8599dc3bb2e810176fe20e70ed50851
|
||||||
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
||||||
@ -31,5 +31,5 @@ index fba383e268..0d63618622 100644
|
|||||||
QPointF targetStartPos;
|
QPointF targetStartPos;
|
||||||
QPointF lastPos;
|
QPointF lastPos;
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 92225b72b9ca6b1efc9bc7bb0c12dd7487e900a7 Mon Sep 17 00:00:00 2001
|
From 3c42d4d3dce95b67d65541c5612384eab0c3e27b Mon Sep 17 00:00:00 2001
|
||||||
From: Dmitry Shachnev <mitya57@gmail.com>
|
From: Dmitry Shachnev <mitya57@gmail.com>
|
||||||
Date: Wed, 18 Aug 2021 22:50:29 +0300
|
Date: Wed, 18 Aug 2021 22:50:29 +0300
|
||||||
Subject: [PATCH 30/36] Include <limits> in Yarr.h to fix build with GCC 11
|
Subject: [PATCH 06/20] Include <limits> in Yarr.h to fix build with GCC 11
|
||||||
MIME-Version: 1.0
|
MIME-Version: 1.0
|
||||||
Content-Type: text/plain; charset=UTF-8
|
Content-Type: text/plain; charset=UTF-8
|
||||||
Content-Transfer-Encoding: 8bit
|
Content-Transfer-Encoding: 8bit
|
||||||
@ -39,5 +39,5 @@ index ccf78f9880..2955ea7e72 100644
|
|||||||
|
|
||||||
namespace JSC { namespace Yarr {
|
namespace JSC { namespace Yarr {
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,43 +0,0 @@
|
|||||||
From 81238e0ff8453f4fb78436ac9bec8452584680ae Mon Sep 17 00:00:00 2001
|
|
||||||
From: Bartlomiej Moskal <bartlomiej.moskal@siili.com>
|
|
||||||
Date: Thu, 5 Nov 2020 10:12:29 +0100
|
|
||||||
Subject: [PATCH 06/28] QQuickWindow: Check if QQuickItem was not deleted
|
|
||||||
|
|
||||||
Added check into deliverMatchingPointsToItem method for Android device.
|
|
||||||
|
|
||||||
In QT_VERSION below 6.0.0 touchEnabled for QtQuickItems is set by default to true
|
|
||||||
It causes delivering touch events to Items which are not interested
|
|
||||||
In some cases it may cause a crash. For example using Material Style in Android.
|
|
||||||
QQuickShaderEffectSource may be deleted and then try to handle touch
|
|
||||||
|
|
||||||
Fixes: QTBUG-85379
|
|
||||||
Change-Id: Ia2c4e016db57ef9c86fcc31d4cfba6154068a546
|
|
||||||
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
|
||||||
(cherry picked from commit a14e4fcdf94d26774490b26a4ef77981594f583f)
|
|
||||||
Reviewed-by: Bartlomiej Moskal <bartlomiej.moskal@siili.com>
|
|
||||||
---
|
|
||||||
src/quick/items/qquickwindow.cpp | 8 ++++++++
|
|
||||||
1 file changed, 8 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquickwindow.cpp b/src/quick/items/qquickwindow.cpp
|
|
||||||
index d0c9ad5454..9ff91eb9a0 100644
|
|
||||||
--- a/src/quick/items/qquickwindow.cpp
|
|
||||||
+++ b/src/quick/items/qquickwindow.cpp
|
|
||||||
@@ -2864,6 +2864,14 @@ void QQuickWindowPrivate::deliverMatchingPointsToItem(QQuickItem *item, QQuickPo
|
|
||||||
{
|
|
||||||
Q_Q(QQuickWindow);
|
|
||||||
QQuickItemPrivate *itemPrivate = QQuickItemPrivate::get(item);
|
|
||||||
+#if defined(Q_OS_ANDROID) && QT_VERSION < QT_VERSION_CHECK(6, 0, 0)
|
|
||||||
+ // QTBUG-85379
|
|
||||||
+ // In QT_VERSION below 6.0.0 touchEnabled for QtQuickItems is set by default to true
|
|
||||||
+ // It causes delivering touch events to Items which are not interested
|
|
||||||
+ // In some cases (like using Material Style in Android) it may cause a crash
|
|
||||||
+ if (itemPrivate->wasDeleted)
|
|
||||||
+ return;
|
|
||||||
+#endif
|
|
||||||
pointerEvent->localize(item);
|
|
||||||
|
|
||||||
// Let the Item's handlers (if any) have the event first.
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,271 +0,0 @@
|
|||||||
From e749605ecbe76c392552d0e3a75f47b31bed9ba1 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Kimmo Ollila <kimmo.ollila@qt.io>
|
|
||||||
Date: Wed, 11 Nov 2020 12:23:53 +0200
|
|
||||||
Subject: [PATCH 07/28] Avoid GHS linker to optimize away QML type
|
|
||||||
registrations
|
|
||||||
|
|
||||||
GHS linker optimizes away QML type registrations despite volatile.
|
|
||||||
To prevent this we add #pragma ghs reference(s) to avoid such linker
|
|
||||||
optimization.
|
|
||||||
|
|
||||||
Fixes: QTBUG-88033
|
|
||||||
Change-Id: I7c8983506360710185c37028873234b1464847d5
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
---
|
|
||||||
src/imports/folderlistmodel/plugin.cpp | 1 +
|
|
||||||
src/imports/labsanimation/plugin.cpp | 1 +
|
|
||||||
src/imports/labsmodels/plugin.cpp | 1 +
|
|
||||||
src/imports/layouts/plugin.cpp | 1 +
|
|
||||||
src/imports/localstorage/plugin.cpp | 1 +
|
|
||||||
src/imports/settings/plugin.cpp | 1 +
|
|
||||||
src/imports/sharedimage/plugin.cpp | 1 +
|
|
||||||
src/imports/statemachine/plugin.cpp | 1 +
|
|
||||||
src/imports/testlib/main.cpp | 1 +
|
|
||||||
src/imports/wavefrontmesh/plugin.cpp | 1 +
|
|
||||||
src/imports/window/plugin.cpp | 1 +
|
|
||||||
src/particles/qtquickparticlesglobal_p.h | 1 +
|
|
||||||
src/qml/qml/qqmlextensionplugin.h | 7 +++++++
|
|
||||||
src/qml/qtqmlglobal_p.h | 2 ++
|
|
||||||
src/qmlmodels/qtqmlmodelsglobal_p.h | 1 +
|
|
||||||
src/qmlworkerscript/qtqmlworkerscriptglobal_p.h | 1 +
|
|
||||||
src/quick/qtquickglobal_p.h | 1 +
|
|
||||||
src/quickshapes/qquickshapesglobal_p.h | 2 ++
|
|
||||||
18 files changed, 26 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/imports/folderlistmodel/plugin.cpp b/src/imports/folderlistmodel/plugin.cpp
|
|
||||||
index 7a38769b77..7206df6664 100644
|
|
||||||
--- a/src/imports/folderlistmodel/plugin.cpp
|
|
||||||
+++ b/src/imports/folderlistmodel/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include "qquickfolderlistmodel.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_folderlistmodel();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_folderlistmodel);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/labsanimation/plugin.cpp b/src/imports/labsanimation/plugin.cpp
|
|
||||||
index 9c985f0dcf..c35be764f9 100644
|
|
||||||
--- a/src/imports/labsanimation/plugin.cpp
|
|
||||||
+++ b/src/imports/labsanimation/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include "qquickboundaryrule_p.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_animation();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_animation);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/labsmodels/plugin.cpp b/src/imports/labsmodels/plugin.cpp
|
|
||||||
index ab5e0023a6..b06491e663 100644
|
|
||||||
--- a/src/imports/labsmodels/plugin.cpp
|
|
||||||
+++ b/src/imports/labsmodels/plugin.cpp
|
|
||||||
@@ -51,6 +51,7 @@
|
|
||||||
#endif
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_qmlmodels();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_qmlmodels);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/layouts/plugin.cpp b/src/imports/layouts/plugin.cpp
|
|
||||||
index af270c1732..b6ae516eee 100644
|
|
||||||
--- a/src/imports/layouts/plugin.cpp
|
|
||||||
+++ b/src/imports/layouts/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include "qquickstacklayout_p.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_QtQuick_Layouts();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick_Layouts);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/localstorage/plugin.cpp b/src/imports/localstorage/plugin.cpp
|
|
||||||
index e488b3d43c..0291ed4715 100644
|
|
||||||
--- a/src/imports/localstorage/plugin.cpp
|
|
||||||
+++ b/src/imports/localstorage/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include <QtQml/qqml.h>
|
|
||||||
|
|
||||||
extern void qml_register_types_QtQuick_LocalStorage();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick_LocalStorage);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/settings/plugin.cpp b/src/imports/settings/plugin.cpp
|
|
||||||
index e8e640412b..e83147f612 100644
|
|
||||||
--- a/src/imports/settings/plugin.cpp
|
|
||||||
+++ b/src/imports/settings/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include "qqmlsettings_p.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_settings();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_settings);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/sharedimage/plugin.cpp b/src/imports/sharedimage/plugin.cpp
|
|
||||||
index d7c2ef8d17..79168d933b 100644
|
|
||||||
--- a/src/imports/sharedimage/plugin.cpp
|
|
||||||
+++ b/src/imports/sharedimage/plugin.cpp
|
|
||||||
@@ -100,6 +100,7 @@
|
|
||||||
*/
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_sharedimage();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_sharedimage);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/statemachine/plugin.cpp b/src/imports/statemachine/plugin.cpp
|
|
||||||
index c370504029..abb238965e 100644
|
|
||||||
--- a/src/imports/statemachine/plugin.cpp
|
|
||||||
+++ b/src/imports/statemachine/plugin.cpp
|
|
||||||
@@ -49,6 +49,7 @@
|
|
||||||
#include <qqml.h>
|
|
||||||
|
|
||||||
extern void qml_register_types_QtQml_StateMachine();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQml_StateMachine);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/testlib/main.cpp b/src/imports/testlib/main.cpp
|
|
||||||
index 83fc150e6c..1da251c49b 100644
|
|
||||||
--- a/src/imports/testlib/main.cpp
|
|
||||||
+++ b/src/imports/testlib/main.cpp
|
|
||||||
@@ -51,6 +51,7 @@ QML_DECLARE_TYPE(QuickTestEvent)
|
|
||||||
QML_DECLARE_TYPE(QuickTestUtil)
|
|
||||||
|
|
||||||
extern void qml_register_types_QtTest();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtTest);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/wavefrontmesh/plugin.cpp b/src/imports/wavefrontmesh/plugin.cpp
|
|
||||||
index eea0db19db..edd4d1dba5 100644
|
|
||||||
--- a/src/imports/wavefrontmesh/plugin.cpp
|
|
||||||
+++ b/src/imports/wavefrontmesh/plugin.cpp
|
|
||||||
@@ -43,6 +43,7 @@
|
|
||||||
#include "qwavefrontmesh.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_Qt_labs_wavefrontmesh();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_Qt_labs_wavefrontmesh);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/imports/window/plugin.cpp b/src/imports/window/plugin.cpp
|
|
||||||
index 5152fa02ec..ff2f10fde3 100644
|
|
||||||
--- a/src/imports/window/plugin.cpp
|
|
||||||
+++ b/src/imports/window/plugin.cpp
|
|
||||||
@@ -42,6 +42,7 @@
|
|
||||||
#include "plugin.h"
|
|
||||||
|
|
||||||
extern void qml_register_types_QtQuick_Window();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick_Window);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/particles/qtquickparticlesglobal_p.h b/src/particles/qtquickparticlesglobal_p.h
|
|
||||||
index 927bc29050..91c2764060 100644
|
|
||||||
--- a/src/particles/qtquickparticlesglobal_p.h
|
|
||||||
+++ b/src/particles/qtquickparticlesglobal_p.h
|
|
||||||
@@ -66,5 +66,6 @@
|
|
||||||
#endif
|
|
||||||
|
|
||||||
void Q_QUICKPARTICLES_PRIVATE_EXPORT qml_register_types_QtQuick_Particles();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick_Particles);
|
|
||||||
|
|
||||||
#endif // QTQUICKPARTICLESGLOBAL_P_H
|
|
||||||
diff --git a/src/qml/qml/qqmlextensionplugin.h b/src/qml/qml/qqmlextensionplugin.h
|
|
||||||
index ef7ff422cd..afb3f99c4a 100644
|
|
||||||
--- a/src/qml/qml/qqmlextensionplugin.h
|
|
||||||
+++ b/src/qml/qml/qqmlextensionplugin.h
|
|
||||||
@@ -44,6 +44,13 @@
|
|
||||||
#include <QtCore/QUrl>
|
|
||||||
#include <QtQml/qqmlextensioninterface.h>
|
|
||||||
|
|
||||||
+#if defined(Q_CC_GHS)
|
|
||||||
+# define GHS_PRAGMA(S) _Pragma(#S)
|
|
||||||
+# define GHS_KEEP_REFERENCE(S) GHS_PRAGMA(ghs reference S ##__Fv)
|
|
||||||
+#else
|
|
||||||
+# define GHS_KEEP_REFERENCE(S)
|
|
||||||
+#endif
|
|
||||||
+
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
class QQmlEngine;
|
|
||||||
diff --git a/src/qml/qtqmlglobal_p.h b/src/qml/qtqmlglobal_p.h
|
|
||||||
index a729729b67..7b0910fa13 100644
|
|
||||||
--- a/src/qml/qtqmlglobal_p.h
|
|
||||||
+++ b/src/qml/qtqmlglobal_p.h
|
|
||||||
@@ -53,6 +53,7 @@
|
|
||||||
|
|
||||||
#include <QtCore/private/qglobal_p.h>
|
|
||||||
#include <QtQml/qtqmlglobal.h>
|
|
||||||
+#include <QtQml/qqmlextensionplugin.h>
|
|
||||||
#ifndef QT_QML_BOOTSTRAPPED
|
|
||||||
# include <QtQml/private/qtqml-config_p.h>
|
|
||||||
#endif
|
|
||||||
@@ -61,6 +62,7 @@
|
|
||||||
#define Q_QML_PRIVATE_EXPORT Q_QML_EXPORT
|
|
||||||
|
|
||||||
void Q_QML_PRIVATE_EXPORT qml_register_types_QtQml();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQml);
|
|
||||||
|
|
||||||
#if !defined(QT_QMLDEVTOOLS_LIB) && !defined(QT_BUILD_QMLDEVTOOLS_LIB)
|
|
||||||
# define Q_QML_AUTOTEST_EXPORT Q_AUTOTEST_EXPORT
|
|
||||||
diff --git a/src/qmlmodels/qtqmlmodelsglobal_p.h b/src/qmlmodels/qtqmlmodelsglobal_p.h
|
|
||||||
index 1a1157138d..24df6ef7b3 100644
|
|
||||||
--- a/src/qmlmodels/qtqmlmodelsglobal_p.h
|
|
||||||
+++ b/src/qmlmodels/qtqmlmodelsglobal_p.h
|
|
||||||
@@ -59,5 +59,6 @@
|
|
||||||
#define Q_QMLMODELS_AUTOTEST_EXPORT Q_AUTOTEST_EXPORT
|
|
||||||
|
|
||||||
void Q_QMLMODELS_PRIVATE_EXPORT qml_register_types_QtQml_Models();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQml_Models);
|
|
||||||
|
|
||||||
#endif // QTQMLMODELSGLOBAL_P_H
|
|
||||||
diff --git a/src/qmlworkerscript/qtqmlworkerscriptglobal_p.h b/src/qmlworkerscript/qtqmlworkerscriptglobal_p.h
|
|
||||||
index c75d5f3129..6452567f6b 100644
|
|
||||||
--- a/src/qmlworkerscript/qtqmlworkerscriptglobal_p.h
|
|
||||||
+++ b/src/qmlworkerscript/qtqmlworkerscriptglobal_p.h
|
|
||||||
@@ -58,5 +58,6 @@
|
|
||||||
#define Q_QMLWORKERSCRIPT_AUTOTEST_EXPORT Q_AUTOTEST_EXPORT
|
|
||||||
|
|
||||||
void Q_QMLWORKERSCRIPT_PRIVATE_EXPORT qml_register_types_QtQml_WorkerScript();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQml_WorkerScript);
|
|
||||||
|
|
||||||
#endif // QTQMLWORKERSCRIPTGLOBAL_P_H
|
|
||||||
diff --git a/src/quick/qtquickglobal_p.h b/src/quick/qtquickglobal_p.h
|
|
||||||
index 80e59563c7..97680569e7 100644
|
|
||||||
--- a/src/quick/qtquickglobal_p.h
|
|
||||||
+++ b/src/quick/qtquickglobal_p.h
|
|
||||||
@@ -62,6 +62,7 @@
|
|
||||||
#define Q_QUICK_PRIVATE_EXPORT Q_QUICK_EXPORT
|
|
||||||
|
|
||||||
void Q_QUICK_PRIVATE_EXPORT qml_register_types_QtQuick();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick);
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
|
|
||||||
diff --git a/src/quickshapes/qquickshapesglobal_p.h b/src/quickshapes/qquickshapesglobal_p.h
|
|
||||||
index 40f6cfbdcf..37386c23b2 100644
|
|
||||||
--- a/src/quickshapes/qquickshapesglobal_p.h
|
|
||||||
+++ b/src/quickshapes/qquickshapesglobal_p.h
|
|
||||||
@@ -51,6 +51,7 @@
|
|
||||||
// We mean it.
|
|
||||||
//
|
|
||||||
|
|
||||||
+#include <QtQml/qqmlextensionplugin.h>
|
|
||||||
#include "qquickshapesglobal.h"
|
|
||||||
|
|
||||||
QT_BEGIN_NAMESPACE
|
|
||||||
@@ -60,5 +61,6 @@ QT_BEGIN_NAMESPACE
|
|
||||||
QT_END_NAMESPACE
|
|
||||||
|
|
||||||
void Q_QUICKSHAPES_PRIVATE_EXPORT qml_register_types_QtQuick_Shapes();
|
|
||||||
+GHS_KEEP_REFERENCE(qml_register_types_QtQuick_Shapes);
|
|
||||||
|
|
||||||
#endif // QQUICKSHAPESGLOBAL_P_H
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 1c33a9d045897ce755a818ebff7ddecae97885d3 Mon Sep 17 00:00:00 2001
|
From 31179b578276f10841fcb60a876ea757f9ca82ff Mon Sep 17 00:00:00 2001
|
||||||
From: Aleix Pol <aleixpol@kde.org>
|
From: Aleix Pol <aleixpol@kde.org>
|
||||||
Date: Tue, 21 Sep 2021 00:10:26 +0200
|
Date: Tue, 21 Sep 2021 00:10:26 +0200
|
||||||
Subject: [PATCH 31/36] QQuickLoader: Do not incubate if the source arrives
|
Subject: [PATCH 07/20] QQuickLoader: Do not incubate if the source arrives
|
||||||
after setActive(false)
|
after setActive(false)
|
||||||
|
|
||||||
Otherwise we end up in the crazy place of active being false but item
|
Otherwise we end up in the crazy place of active being false but item
|
||||||
@ -21,10 +21,10 @@ Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|||||||
create mode 100644 tests/auto/quick/qquickloader/data/loader-async-race.qml
|
create mode 100644 tests/auto/quick/qquickloader/data/loader-async-race.qml
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquickloader.cpp b/src/quick/items/qquickloader.cpp
|
diff --git a/src/quick/items/qquickloader.cpp b/src/quick/items/qquickloader.cpp
|
||||||
index 8cd63a4236..a6f7009946 100644
|
index cb4f79a3c2..7fbe66fdda 100644
|
||||||
--- a/src/quick/items/qquickloader.cpp
|
--- a/src/quick/items/qquickloader.cpp
|
||||||
+++ b/src/quick/items/qquickloader.cpp
|
+++ b/src/quick/items/qquickloader.cpp
|
||||||
@@ -738,6 +738,9 @@ void QQuickLoaderPrivate::_q_sourceLoaded()
|
@@ -737,6 +737,9 @@ void QQuickLoaderPrivate::_q_sourceLoaded()
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -71,7 +71,7 @@ index 0000000000..8ba625c5c1
|
|||||||
+ }
|
+ }
|
||||||
+}
|
+}
|
||||||
diff --git a/tests/auto/quick/qquickloader/tst_qquickloader.cpp b/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
diff --git a/tests/auto/quick/qquickloader/tst_qquickloader.cpp b/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
||||||
index f4b682f3f4..fe2d71b037 100644
|
index 0f6c811adb..dddacbaa0b 100644
|
||||||
--- a/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
--- a/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
||||||
+++ b/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
+++ b/tests/auto/quick/qquickloader/tst_qquickloader.cpp
|
||||||
@@ -132,6 +132,7 @@ private slots:
|
@@ -132,6 +132,7 @@ private slots:
|
||||||
@ -82,7 +82,7 @@ index f4b682f3f4..fe2d71b037 100644
|
|||||||
};
|
};
|
||||||
|
|
||||||
Q_DECLARE_METATYPE(QList<QQmlError>)
|
Q_DECLARE_METATYPE(QList<QQmlError>)
|
||||||
@@ -1491,6 +1492,24 @@ void tst_QQuickLoader::setSourceAndCheckStatus()
|
@@ -1496,6 +1497,24 @@ void tst_QQuickLoader::setSourceAndCheckStatus()
|
||||||
QCOMPARE(loader->status(), QQuickLoader::Null);
|
QCOMPARE(loader->status(), QQuickLoader::Null);
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -108,5 +108,5 @@ index f4b682f3f4..fe2d71b037 100644
|
|||||||
|
|
||||||
#include "tst_qquickloader.moc"
|
#include "tst_qquickloader.moc"
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,83 +0,0 @@
|
|||||||
From f4afe4934819a7f0fb9754b9bb08fb1acb818058 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Shinichi Okada <shinichi.okada@qt.io>
|
|
||||||
Date: Tue, 17 Nov 2020 14:15:50 +0900
|
|
||||||
Subject: [PATCH 08/28] QML Text doesn't reset lineCount when text is empty
|
|
||||||
|
|
||||||
lineCount is not reset when replacing a multi-line QML Text 'text'
|
|
||||||
property with an "" empty string. Also, the lineCountChanged signal is
|
|
||||||
not emitted
|
|
||||||
|
|
||||||
Task-number: QTBUG-84458
|
|
||||||
Change-Id: Ic3c02e6a90e6675eadbaafc6af6ab0356ee98123
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
(cherry picked from commit ae1b9c6d94001411efeef600e22638906e0fa990)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/quick/items/qquicktext.cpp | 2 +
|
|
||||||
.../auto/quick/qquicktext/tst_qquicktext.cpp | 37 +++++++++++++++++++
|
|
||||||
2 files changed, 39 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquicktext.cpp b/src/quick/items/qquicktext.cpp
|
|
||||||
index 90469ee82b..581ab9f76a 100644
|
|
||||||
--- a/src/quick/items/qquicktext.cpp
|
|
||||||
+++ b/src/quick/items/qquicktext.cpp
|
|
||||||
@@ -398,6 +398,8 @@ void QQuickTextPrivate::updateSize()
|
|
||||||
layedOutTextRect = QRectF(0, 0, 0, fontHeight);
|
|
||||||
advance = QSizeF();
|
|
||||||
signalSizeChange(previousSize);
|
|
||||||
+ lineCount = 1;
|
|
||||||
+ emit q->lineCountChanged();
|
|
||||||
updateType = UpdatePaintNode;
|
|
||||||
q->update();
|
|
||||||
return;
|
|
||||||
diff --git a/tests/auto/quick/qquicktext/tst_qquicktext.cpp b/tests/auto/quick/qquicktext/tst_qquicktext.cpp
|
|
||||||
index 42fdbea58d..308c6b5091 100644
|
|
||||||
--- a/tests/auto/quick/qquicktext/tst_qquicktext.cpp
|
|
||||||
+++ b/tests/auto/quick/qquicktext/tst_qquicktext.cpp
|
|
||||||
@@ -2243,6 +2243,43 @@ void tst_qquicktext::lineCount()
|
|
||||||
QCOMPARE(myText->lineCount(), 2);
|
|
||||||
QCOMPARE(myText->truncated(), true);
|
|
||||||
QCOMPARE(myText->maximumLineCount(), 2);
|
|
||||||
+
|
|
||||||
+ // QTBUG-84458
|
|
||||||
+ myText->resetMaximumLineCount();
|
|
||||||
+ myText->setText("qqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 2);
|
|
||||||
+ myText->setText("qqqqq\nqqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 3);
|
|
||||||
+ myText->setText("");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 1);
|
|
||||||
+
|
|
||||||
+ myText->setText("qqqqq\nqqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 3);
|
|
||||||
+ myText->setFontSizeMode(QQuickText::HorizontalFit);
|
|
||||||
+ myText->setText("");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 1);
|
|
||||||
+
|
|
||||||
+ myText->setText("qqqqq\nqqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 3);
|
|
||||||
+ myText->setFontSizeMode(QQuickText::VerticalFit);
|
|
||||||
+ myText->setText("");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 1);
|
|
||||||
+
|
|
||||||
+ myText->setText("qqqqq\nqqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 3);
|
|
||||||
+ myText->setFontSizeMode(QQuickText::Fit);
|
|
||||||
+ myText->setText("");
|
|
||||||
+ QCOMPARE(myText->lineCount(), 1);
|
|
||||||
+
|
|
||||||
+ QScopedPointer<QQuickView> layoutWindow(createView(testFile("lineLayoutHAlign.qml")));
|
|
||||||
+ QQuickText *lineLaidOut = layoutWindow->rootObject()->findChild<QQuickText*>("myText");
|
|
||||||
+ QVERIFY(lineLaidOut != nullptr);
|
|
||||||
+
|
|
||||||
+ lineLaidOut->setText("qqqqq\nqqqqq\nqqqqq");
|
|
||||||
+ QCOMPARE(lineLaidOut->lineCount(), 3);
|
|
||||||
+ lineLaidOut->setFontSizeMode(QQuickText::FixedSize);
|
|
||||||
+ lineLaidOut->setText("");
|
|
||||||
+ QCOMPARE(lineLaidOut->lineCount(), 1);
|
|
||||||
}
|
|
||||||
|
|
||||||
void tst_qquicktext::lineHeight()
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From e0271324f05fee2c8670a73d1e7e89aef51532a3 Mon Sep 17 00:00:00 2001
|
From 4fd8f31beb6ce07207cc8e934c7f4caa6abd8a54 Mon Sep 17 00:00:00 2001
|
||||||
From: Aleix Pol <aleixpol@kde.org>
|
From: Aleix Pol <aleixpol@kde.org>
|
||||||
Date: Thu, 23 Sep 2021 03:43:04 +0200
|
Date: Thu, 23 Sep 2021 03:43:04 +0200
|
||||||
Subject: [PATCH 32/36] QQmlDelegateModel: Refresh the view when a column is
|
Subject: [PATCH 08/20] QQmlDelegateModel: Refresh the view when a column is
|
||||||
added at 0
|
added at 0
|
||||||
|
|
||||||
It can happen that a model reports n>0 rows but columns=0 (See
|
It can happen that a model reports n>0 rows but columns=0 (See
|
||||||
@ -23,7 +23,7 @@ Signed-off-by: Aleix Pol <aleixpol@kde.org>
|
|||||||
create mode 100644 tests/auto/qml/qqmldelegatemodel/data/redrawUponColumnChange.qml
|
create mode 100644 tests/auto/qml/qqmldelegatemodel/data/redrawUponColumnChange.qml
|
||||||
|
|
||||||
diff --git a/src/qmlmodels/qqmldelegatemodel.cpp b/src/qmlmodels/qqmldelegatemodel.cpp
|
diff --git a/src/qmlmodels/qqmldelegatemodel.cpp b/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
index 12c3d11937..8ce3da1cf1 100644
|
index e6aff955fe..381726291d 100644
|
||||||
--- a/src/qmlmodels/qqmldelegatemodel.cpp
|
--- a/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
+++ b/src/qmlmodels/qqmldelegatemodel.cpp
|
+++ b/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
@@ -389,6 +389,12 @@ void QQmlDelegateModelPrivate::connectToAbstractItemModel()
|
@@ -389,6 +389,12 @@ void QQmlDelegateModelPrivate::connectToAbstractItemModel()
|
||||||
@ -52,7 +52,7 @@ index 12c3d11937..8ce3da1cf1 100644
|
|||||||
QObject::disconnect(aim, SIGNAL(dataChanged(QModelIndex,QModelIndex,QVector<int>)),
|
QObject::disconnect(aim, SIGNAL(dataChanged(QModelIndex,QModelIndex,QVector<int>)),
|
||||||
q, SLOT(_q_dataChanged(QModelIndex,QModelIndex,QVector<int>)));
|
q, SLOT(_q_dataChanged(QModelIndex,QModelIndex,QVector<int>)));
|
||||||
QObject::disconnect(aim, SIGNAL(rowsMoved(QModelIndex,int,int,QModelIndex,int)),
|
QObject::disconnect(aim, SIGNAL(rowsMoved(QModelIndex,int,int,QModelIndex,int)),
|
||||||
@@ -1953,6 +1965,38 @@ void QQmlDelegateModel::_q_rowsMoved(
|
@@ -1958,6 +1970,38 @@ void QQmlDelegateModel::_q_rowsMoved(
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -192,5 +192,5 @@ index 87f42c0c8a..1d338ac330 100644
|
|||||||
|
|
||||||
#include "tst_qqmldelegatemodel.moc"
|
#include "tst_qqmldelegatemodel.moc"
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,38 +0,0 @@
|
|||||||
From 4d6078e4b556e83f55ffed55f14203c3ed880c62 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Mitch Curtis <mitch.curtis@qt.io>
|
|
||||||
Date: Thu, 19 Nov 2020 16:39:23 +0100
|
|
||||||
Subject: [PATCH 09/28] Doc: mention that INCLUDEPATH must be set in some cases
|
|
||||||
|
|
||||||
Otherwise the generated type registrations .cpp file will not be able
|
|
||||||
to include the sources.
|
|
||||||
|
|
||||||
Change-Id: I7821c7674b4341546da2fc49e584bf10cc60b46f
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit 06c31f386f8664343debd219a522a8897df0f3ec)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/qml/doc/src/cppintegration/definetypes.qdoc | 8 ++++++++
|
|
||||||
1 file changed, 8 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/qml/doc/src/cppintegration/definetypes.qdoc b/src/qml/doc/src/cppintegration/definetypes.qdoc
|
|
||||||
index cbbbd9ba58..ece2fd5fd7 100644
|
|
||||||
--- a/src/qml/doc/src/cppintegration/definetypes.qdoc
|
|
||||||
+++ b/src/qml/doc/src/cppintegration/definetypes.qdoc
|
|
||||||
@@ -117,6 +117,14 @@ QML_IMPORT_NAME = com.mycompany.messaging
|
|
||||||
QML_IMPORT_MAJOR_VERSION = 1
|
|
||||||
\endcode
|
|
||||||
|
|
||||||
+If the header the class is declared in is not accessible from your project's
|
|
||||||
+include path, you may have to amend the include path so that the generated
|
|
||||||
+registration code can be compiled:
|
|
||||||
+
|
|
||||||
+\code
|
|
||||||
+INCLUDEPATH += com/mycompany/messaging
|
|
||||||
+\endcode
|
|
||||||
+
|
|
||||||
The type can be used in an \l{qtqml-syntax-basics.html#object-declarations}
|
|
||||||
{object declaration} from QML, and its properties can be read and written to,
|
|
||||||
as per the example below:
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 9c0030bc8e828ecfbd8a4c8dd6bbfcbd3655b71c Mon Sep 17 00:00:00 2001
|
From 00dd37619f35db3cddb2ac59198d3bde7ce73cd7 Mon Sep 17 00:00:00 2001
|
||||||
From: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
|
From: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
|
||||||
Date: Sun, 10 Oct 2021 21:04:21 +0300
|
Date: Sun, 10 Oct 2021 21:04:21 +0300
|
||||||
Subject: [PATCH 33/36] Fix sweep step for tainted QObject JavaScript wrappers
|
Subject: [PATCH 09/20] Fix sweep step for tainted QObject JavaScript wrappers
|
||||||
|
|
||||||
Currently, whenever the garbage collector runs, it will destroy all
|
Currently, whenever the garbage collector runs, it will destroy all
|
||||||
valid tainted wrappers.
|
valid tainted wrappers.
|
||||||
@ -115,5 +115,5 @@ index 5d635aa63b..824fd89e5b 100644
|
|||||||
// engine2 doesn't own the object as engine1 was the first to wrap it above.
|
// engine2 doesn't own the object as engine1 was the first to wrap it above.
|
||||||
// Therefore, no effect here.
|
// Therefore, no effect here.
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From c1ddd97c87c729c7d05c7a74f82d41cfc5bd9cf8 Mon Sep 17 00:00:00 2001
|
From 55f07163f84badcc9aa0805f1523ef43a7225778 Mon Sep 17 00:00:00 2001
|
||||||
From: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
|
From: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
|
||||||
Date: Tue, 12 Oct 2021 13:13:01 +0200
|
Date: Tue, 12 Oct 2021 13:13:01 +0200
|
||||||
Subject: [PATCH 34/36] Fix distorted text with subpixel matrix translation
|
Subject: [PATCH 10/20] Fix distorted text with subpixel matrix translation
|
||||||
|
|
||||||
We would pixel-align native text *before* applying the
|
We would pixel-align native text *before* applying the
|
||||||
model-view matrix, which would cause GL_NEAREST artifacts to
|
model-view matrix, which would cause GL_NEAREST artifacts to
|
||||||
@ -479,5 +479,5 @@ index 0000000000..c60fc4d8b0
|
|||||||
+ }
|
+ }
|
||||||
+}
|
+}
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,47 +0,0 @@
|
|||||||
From 65851a450926befad65d9cffcaa217875d1936c6 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maximilian Goldstein <max.goldstein@qt.io>
|
|
||||||
Date: Mon, 23 Nov 2020 13:17:40 +0100
|
|
||||||
Subject: [PATCH 10/28] qmlfunctions.qdoc: Add clarification to QML_FOREIGN
|
|
||||||
|
|
||||||
Fixes: QTBUG-87150
|
|
||||||
Change-Id: If99a06a07892bdfef7b6b1e8fa737480750992fe
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
(cherry picked from commit 56f428c360191230b571969a2651e85380030afa)
|
|
||||||
---
|
|
||||||
examples/qml/doc/src/qml-extending.qdoc | 4 ++++
|
|
||||||
src/qml/doc/src/qmlfunctions.qdoc | 4 ++++
|
|
||||||
2 files changed, 8 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/examples/qml/doc/src/qml-extending.qdoc b/examples/qml/doc/src/qml-extending.qdoc
|
|
||||||
index 723e470d45..c9922ebd45 100644
|
|
||||||
--- a/examples/qml/doc/src/qml-extending.qdoc
|
|
||||||
+++ b/examples/qml/doc/src/qml-extending.qdoc
|
|
||||||
@@ -79,6 +79,10 @@ Qt's internal QLineEdit class.
|
|
||||||
|
|
||||||
\snippet referenceexamples/extended/lineedit.h 0
|
|
||||||
|
|
||||||
+Note the usage of \l QML_NAMED_ELEMENT() instead of \l QML_ELEMENT.
|
|
||||||
+QML_ELEMENT uses the name of the containing type by default, "LineEditExtension" in this case.
|
|
||||||
+As the class being an extension class is an implementation detail, we choose the more natural name "LineEdit" instead
|
|
||||||
+
|
|
||||||
The QML engine then instantiates a \l QLineEdit:
|
|
||||||
|
|
||||||
\snippet referenceexamples/extended/main.cpp 1
|
|
||||||
diff --git a/src/qml/doc/src/qmlfunctions.qdoc b/src/qml/doc/src/qmlfunctions.qdoc
|
|
||||||
index 12b7efb159..4e531ceb61 100644
|
|
||||||
--- a/src/qml/doc/src/qmlfunctions.qdoc
|
|
||||||
+++ b/src/qml/doc/src/qmlfunctions.qdoc
|
|
||||||
@@ -250,6 +250,10 @@
|
|
||||||
This is useful for registering types that cannot be amended to add the macros,
|
|
||||||
for example because they belong to 3rdparty libraries.
|
|
||||||
|
|
||||||
+ \b{NOTE:} You may want to use \l QML_NAMED_ELEMENT() instead of \l QML_ELEMENT due to the fact that
|
|
||||||
+ the element will be named like the struct it is contained in, not the foreign type.
|
|
||||||
+ See \l {Extending QML - Extension Objects Example} for an example.
|
|
||||||
+
|
|
||||||
\sa QML_ELEMENT, QML_NAMED_ELEMENT()
|
|
||||||
*/
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,214 +0,0 @@
|
|||||||
From b2862e2bd84d651c1f4fb2ced96ee6f40ea1f3e4 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Andrei Golubev <andrei.golubev@qt.io>
|
|
||||||
Date: Fri, 20 Nov 2020 10:44:44 +0100
|
|
||||||
Subject: [PATCH 11/28] Fix QML property cache leaks of delegate items
|
|
||||||
|
|
||||||
The delegate items are destroyed through an event loop by a call to a
|
|
||||||
deleteLater(). This, however, doesn't work when the application is
|
|
||||||
in the process of exiting and the event loop is already closed (i.e.
|
|
||||||
we're in a stack unwinding part that starts after app.exec())
|
|
||||||
|
|
||||||
Combat this situation by setting a parent of the to-be-deleted object
|
|
||||||
to some QObject that will be destroyed e.g. QCoreApplication::instance()
|
|
||||||
before the program finishes. As QObjects clean their children on
|
|
||||||
destruction, this will make sure that we cleanup the previously leaking
|
|
||||||
thing regardless of the event loop
|
|
||||||
|
|
||||||
Added a test to check that delegates are destroyed (as a separate binary
|
|
||||||
due to differences in main() function)
|
|
||||||
|
|
||||||
Fixes: QTBUG-87228
|
|
||||||
Change-Id: I59066603b77497fe4fd8d051798c3e4b47c119f0
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
(cherry picked from commit 3a5617dc45e281552b9c1f7a04f0561b8fa14d94)
|
|
||||||
---
|
|
||||||
src/qmlmodels/qqmldelegatemodel.cpp | 11 ++-
|
|
||||||
.../qquickview_extra/data/qtbug_87228.qml | 30 ++++++++
|
|
||||||
.../qquickview_extra/qquickview_extra.pro | 12 +++
|
|
||||||
.../qquickview_extra/tst_qquickview_extra.cpp | 77 +++++++++++++++++++
|
|
||||||
tests/auto/quick/quick.pro | 1 +
|
|
||||||
5 files changed, 130 insertions(+), 1 deletion(-)
|
|
||||||
create mode 100644 tests/auto/quick/qquickview_extra/data/qtbug_87228.qml
|
|
||||||
create mode 100644 tests/auto/quick/qquickview_extra/qquickview_extra.pro
|
|
||||||
create mode 100644 tests/auto/quick/qquickview_extra/tst_qquickview_extra.cpp
|
|
||||||
|
|
||||||
diff --git a/src/qmlmodels/qqmldelegatemodel.cpp b/src/qmlmodels/qqmldelegatemodel.cpp
|
|
||||||
index 725b9e8bc3..12c3d11937 100644
|
|
||||||
--- a/src/qmlmodels/qqmldelegatemodel.cpp
|
|
||||||
+++ b/src/qmlmodels/qqmldelegatemodel.cpp
|
|
||||||
@@ -1,6 +1,6 @@
|
|
||||||
/****************************************************************************
|
|
||||||
**
|
|
||||||
-** Copyright (C) 2016 The Qt Company Ltd.
|
|
||||||
+** Copyright (C) 2020 The Qt Company Ltd.
|
|
||||||
** Contact: https://www.qt.io/licensing/
|
|
||||||
**
|
|
||||||
** This file is part of the QtQml module of the Qt Toolkit.
|
|
||||||
@@ -2379,6 +2379,15 @@ void QQmlDelegateModelItem::destroyObject()
|
|
||||||
data->ownContext = nullptr;
|
|
||||||
data->context = nullptr;
|
|
||||||
}
|
|
||||||
+ /* QTBUG-87228: when destroying object at the application exit, the deferred
|
|
||||||
+ * parent by setting it to QCoreApplication instance if it's nullptr, so
|
|
||||||
+ * deletion won't work. Not to leak memory, make sure our object has a that
|
|
||||||
+ * the parent claims the object at the end of the lifetime. When not at the
|
|
||||||
+ * application exit, normal event loop will handle the deferred deletion
|
|
||||||
+ * earlier.
|
|
||||||
+ */
|
|
||||||
+ if (object->parent() == nullptr)
|
|
||||||
+ object->setParent(QCoreApplication::instance());
|
|
||||||
object->deleteLater();
|
|
||||||
|
|
||||||
if (attached) {
|
|
||||||
diff --git a/tests/auto/quick/qquickview_extra/data/qtbug_87228.qml b/tests/auto/quick/qquickview_extra/data/qtbug_87228.qml
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..ff10eba23d
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/quick/qquickview_extra/data/qtbug_87228.qml
|
|
||||||
@@ -0,0 +1,30 @@
|
|
||||||
+import QtQml 2.12
|
|
||||||
+import QtQml.Models 2.12
|
|
||||||
+import QtQuick 2.12
|
|
||||||
+
|
|
||||||
+Item {
|
|
||||||
+ height: 480
|
|
||||||
+ width: 320
|
|
||||||
+ Rectangle {
|
|
||||||
+ id: rootRect
|
|
||||||
+
|
|
||||||
+ function addItem(desc) {
|
|
||||||
+ myModel.append({"desc": desc});
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ Rectangle {
|
|
||||||
+ ListView {
|
|
||||||
+ objectName: "listView"
|
|
||||||
+ delegate: Text {
|
|
||||||
+ required property string desc
|
|
||||||
+ text: desc
|
|
||||||
+ }
|
|
||||||
+ model: ListModel { id: myModel }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ Component.onCompleted: {
|
|
||||||
+ addItem("Test creation of a delegate with a property");
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
diff --git a/tests/auto/quick/qquickview_extra/qquickview_extra.pro b/tests/auto/quick/qquickview_extra/qquickview_extra.pro
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..b40af0ce19
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/quick/qquickview_extra/qquickview_extra.pro
|
|
||||||
@@ -0,0 +1,12 @@
|
|
||||||
+CONFIG += testcase
|
|
||||||
+TARGET = tst_qquickview_extra
|
|
||||||
+macx:CONFIG -= app_bundle
|
|
||||||
+
|
|
||||||
+SOURCES += tst_qquickview_extra.cpp
|
|
||||||
+
|
|
||||||
+include (../../shared/util.pri)
|
|
||||||
+include (../shared/util.pri)
|
|
||||||
+
|
|
||||||
+TESTDATA = data/*
|
|
||||||
+
|
|
||||||
+QT += core-private gui-private qml-private quick-private testlib
|
|
||||||
diff --git a/tests/auto/quick/qquickview_extra/tst_qquickview_extra.cpp b/tests/auto/quick/qquickview_extra/tst_qquickview_extra.cpp
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..f697a438bd
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/quick/qquickview_extra/tst_qquickview_extra.cpp
|
|
||||||
@@ -0,0 +1,77 @@
|
|
||||||
+/****************************************************************************
|
|
||||||
+**
|
|
||||||
+** Copyright (C) 2020 The Qt Company Ltd.
|
|
||||||
+** Contact: https://www.qt.io/licensing/
|
|
||||||
+**
|
|
||||||
+** This file is part of the test suite of the Qt Toolkit.
|
|
||||||
+**
|
|
||||||
+** $QT_BEGIN_LICENSE:GPL-EXCEPT$
|
|
||||||
+** Commercial License Usage
|
|
||||||
+** Licensees holding valid commercial Qt licenses may use this file in
|
|
||||||
+** accordance with the commercial license agreement provided with the
|
|
||||||
+** Software or, alternatively, in accordance with the terms contained in
|
|
||||||
+** a written agreement between you and The Qt Company. For licensing terms
|
|
||||||
+** and conditions see https://www.qt.io/terms-conditions. For further
|
|
||||||
+** information use the contact form at https://www.qt.io/contact-us.
|
|
||||||
+**
|
|
||||||
+** GNU General Public License Usage
|
|
||||||
+** Alternatively, this file may be used under the terms of the GNU
|
|
||||||
+** General Public License version 3 as published by the Free Software
|
|
||||||
+** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
|
|
||||||
+** included in the packaging of this file. Please review the following
|
|
||||||
+** information to ensure the GNU General Public License requirements will
|
|
||||||
+** be met: https://www.gnu.org/licenses/gpl-3.0.html.
|
|
||||||
+**
|
|
||||||
+** $QT_END_LICENSE$
|
|
||||||
+**
|
|
||||||
+****************************************************************************/
|
|
||||||
+#include <qtest.h>
|
|
||||||
+#include <QtTest/QSignalSpy>
|
|
||||||
+#include <QtQuick/qquickview.h>
|
|
||||||
+#include <QtQuick/qquickitem.h>
|
|
||||||
+#include <QtQml/qqmlengine.h>
|
|
||||||
+#include "../../shared/util.h"
|
|
||||||
+#include <QtCore/QDebug>
|
|
||||||
+#include <QtCore/QTimer>
|
|
||||||
+
|
|
||||||
+// Extra app-less tests
|
|
||||||
+class tst_QQuickViewExtra : public QQmlDataTest
|
|
||||||
+{
|
|
||||||
+ Q_OBJECT
|
|
||||||
+public:
|
|
||||||
+ tst_QQuickViewExtra();
|
|
||||||
+
|
|
||||||
+private slots:
|
|
||||||
+ void qtbug_87228();
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
+tst_QQuickViewExtra::tst_QQuickViewExtra() { }
|
|
||||||
+
|
|
||||||
+void tst_QQuickViewExtra::qtbug_87228()
|
|
||||||
+{
|
|
||||||
+ QScopedPointer<QSignalSpy> deletionSpy;
|
|
||||||
+ {
|
|
||||||
+ int argc = 0;
|
|
||||||
+ QGuiApplication app(argc, nullptr);
|
|
||||||
+ QQuickView view;
|
|
||||||
+
|
|
||||||
+ view.setSource(testFileUrl("qtbug_87228.qml"));
|
|
||||||
+ view.show();
|
|
||||||
+ QTimer::singleShot(500, &app, QCoreApplication::quit);
|
|
||||||
+ app.exec();
|
|
||||||
+
|
|
||||||
+ QObject *listView = view.findChild<QObject *>("listView");
|
|
||||||
+ QVERIFY(listView);
|
|
||||||
+ QQuickItem *contentItem = listView->property("contentItem").value<QQuickItem *>();
|
|
||||||
+ QVERIFY(contentItem);
|
|
||||||
+ auto children = contentItem->childItems();
|
|
||||||
+ QVERIFY(children.size() > 0);
|
|
||||||
+ // for the sake of this test, any child would be suitable, so pick first
|
|
||||||
+ deletionSpy.reset(new QSignalSpy(children[0], SIGNAL(destroyed(QObject *))));
|
|
||||||
+ }
|
|
||||||
+ QCOMPARE(deletionSpy->count(), 1);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+QTEST_APPLESS_MAIN(tst_QQuickViewExtra)
|
|
||||||
+
|
|
||||||
+#include "tst_qquickview_extra.moc"
|
|
||||||
diff --git a/tests/auto/quick/quick.pro b/tests/auto/quick/quick.pro
|
|
||||||
index 541bfdd527..45bcf8a9ce 100644
|
|
||||||
--- a/tests/auto/quick/quick.pro
|
|
||||||
+++ b/tests/auto/quick/quick.pro
|
|
||||||
@@ -85,6 +85,7 @@ QUICKTESTS += \
|
|
||||||
qquicktextinput \
|
|
||||||
qquickvisualdatamodel \
|
|
||||||
qquickview \
|
|
||||||
+ qquickview_extra \
|
|
||||||
qquickcanvasitem \
|
|
||||||
qquickdesignersupport \
|
|
||||||
qquickscreen \
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 9dde7b66a30a33074fc2c2c682b53b18791dba0d Mon Sep 17 00:00:00 2001
|
From ffa77919e72951c7accf84ad09dc669d166aedf8 Mon Sep 17 00:00:00 2001
|
||||||
From: Laszlo Agocs <laszlo.agocs@qt.io>
|
From: Laszlo Agocs <laszlo.agocs@qt.io>
|
||||||
Date: Mon, 11 Oct 2021 15:37:33 +0200
|
Date: Mon, 11 Oct 2021 15:37:33 +0200
|
||||||
Subject: [PATCH 35/36] Revert "Fix for possible crash in
|
Subject: [PATCH 11/20] Revert "Fix for possible crash in
|
||||||
QSGDefaultLayer::grab"
|
QSGDefaultLayer::grab"
|
||||||
MIME-Version: 1.0
|
MIME-Version: 1.0
|
||||||
Content-Type: text/plain; charset=UTF-8
|
Content-Type: text/plain; charset=UTF-8
|
||||||
@ -67,5 +67,5 @@ index 4deb6c70a3..c0a1ccab78 100644
|
|||||||
protected:
|
protected:
|
||||||
void releaseResources() override;
|
void releaseResources() override;
|
||||||
--
|
--
|
||||||
2.31.1
|
2.35.1
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From 8e85afd011ea25eec0aa710481e2474dbaaee869 Mon Sep 17 00:00:00 2001
|
From c3804f86e0a75c0c8402127a5b01bc7a858e6492 Mon Sep 17 00:00:00 2001
|
||||||
From: Volker Hilsheimer <volker.hilsheimer@qt.io>
|
From: Volker Hilsheimer <volker.hilsheimer@qt.io>
|
||||||
Date: Wed, 17 Mar 2021 16:52:21 +0100
|
Date: Wed, 17 Mar 2021 16:52:21 +0100
|
||||||
Subject: [PATCH 37/41] QQuickItemAnimation: close potential memory leak
|
Subject: [PATCH 12/20] QQuickItemAnimation: close potential memory leak
|
||||||
|
|
||||||
Fix static analyzer warning bff6cb4333f531d5a72f7bf6dc1485f6.
|
Fix static analyzer warning bff6cb4333f531d5a72f7bf6dc1485f6.
|
||||||
|
|
||||||
@ -54,5 +54,5 @@ index 23694e2de3..dfb56ccc00 100644
|
|||||||
return nullptr;
|
return nullptr;
|
||||||
}
|
}
|
||||||
--
|
--
|
||||||
2.33.1
|
2.35.1
|
||||||
|
|
@ -1,66 +0,0 @@
|
|||||||
From 89ea9f1f9468aa47718cbb398317c63a9479adf2 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Date: Tue, 24 Nov 2020 13:23:23 +0100
|
|
||||||
Subject: [PATCH 12/28] QQuickTextInput: Store mask data in std::unique_ptr
|
|
||||||
|
|
||||||
This ensures that the memory is freed reliably
|
|
||||||
|
|
||||||
Fixes: QTBUG-88807
|
|
||||||
Change-Id: I841a5a2b226a69ce50975d95702a948857d1b54f
|
|
||||||
Reviewed-by: Eskil Abrahamsen Blomfeldt <eskil.abrahamsen-blomfeldt@qt.io>
|
|
||||||
(cherry picked from commit d2d8e90e9f218103d60737e1273ab5322834d9ec)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/quick/items/qquicktextinput.cpp | 6 ++----
|
|
||||||
src/quick/items/qquicktextinput_p_p.h | 4 +++-
|
|
||||||
2 files changed, 5 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquicktextinput.cpp b/src/quick/items/qquicktextinput.cpp
|
|
||||||
index 6275b298ed..bb78ead0e8 100644
|
|
||||||
--- a/src/quick/items/qquicktextinput.cpp
|
|
||||||
+++ b/src/quick/items/qquicktextinput.cpp
|
|
||||||
@@ -3831,8 +3831,7 @@ void QQuickTextInputPrivate::parseInputMask(const QString &maskFields)
|
|
||||||
int delimiter = maskFields.indexOf(QLatin1Char(';'));
|
|
||||||
if (maskFields.isEmpty() || delimiter == 0) {
|
|
||||||
if (m_maskData) {
|
|
||||||
- delete [] m_maskData;
|
|
||||||
- m_maskData = nullptr;
|
|
||||||
+ m_maskData.reset(nullptr);
|
|
||||||
m_maxLength = 32767;
|
|
||||||
internalSetText(QString());
|
|
||||||
}
|
|
||||||
@@ -3863,8 +3862,7 @@ void QQuickTextInputPrivate::parseInputMask(const QString &maskFields)
|
|
||||||
m_maxLength++;
|
|
||||||
}
|
|
||||||
|
|
||||||
- delete [] m_maskData;
|
|
||||||
- m_maskData = new MaskInputData[m_maxLength];
|
|
||||||
+ m_maskData.reset(new MaskInputData[m_maxLength]);
|
|
||||||
|
|
||||||
MaskInputData::Casemode m = MaskInputData::NoCaseMode;
|
|
||||||
c = 0;
|
|
||||||
diff --git a/src/quick/items/qquicktextinput_p_p.h b/src/quick/items/qquicktextinput_p_p.h
|
|
||||||
index 7965f3d3f4..7fbba49405 100644
|
|
||||||
--- a/src/quick/items/qquicktextinput_p_p.h
|
|
||||||
+++ b/src/quick/items/qquicktextinput_p_p.h
|
|
||||||
@@ -58,6 +58,8 @@
|
|
||||||
|
|
||||||
#include "qplatformdefs.h"
|
|
||||||
|
|
||||||
+#include <memory>
|
|
||||||
+
|
|
||||||
//
|
|
||||||
// W A R N I N G
|
|
||||||
// -------------
|
|
||||||
@@ -230,7 +232,7 @@ public:
|
|
||||||
|
|
||||||
QQuickItem *cursorItem;
|
|
||||||
QQuickTextNode *textNode;
|
|
||||||
- MaskInputData *m_maskData;
|
|
||||||
+ std::unique_ptr<MaskInputData[]> m_maskData;
|
|
||||||
QInputControl *m_inputControl;
|
|
||||||
|
|
||||||
QList<int> m_transactions;
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,100 +0,0 @@
|
|||||||
From 810a0afe1e9bd14e4393a73bf6c299b25745dbc5 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Richard Weickelt <richard@weickelt.de>
|
|
||||||
Date: Tue, 24 Nov 2020 01:14:28 +0100
|
|
||||||
Subject: [PATCH 13/28] Fix crash when calling hasOwnProperty() on proxy object
|
|
||||||
|
|
||||||
Property pointer p needs to be checked for nullptr value in
|
|
||||||
QV4::ProxyObject::virtualGetOwnProperty(). This can happen when calling
|
|
||||||
hasOwnProperty() or propertyIsEnumerable().
|
|
||||||
|
|
||||||
Fixes: QTBUG-88786
|
|
||||||
Change-Id: I43da58fed4d8656f9187213f7317f17398739e34
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit 9b321a34490cd17c0eb043b69bd7c9d8d8f513d5)
|
|
||||||
Reviewed-by: Richard Weickelt <richard@weickelt.de>
|
|
||||||
---
|
|
||||||
src/qml/jsruntime/qv4proxy.cpp | 10 +++---
|
|
||||||
.../qml/qqmlecmascript/tst_qqmlecmascript.cpp | 31 ++++++++++++++++++-
|
|
||||||
2 files changed, 36 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/qml/jsruntime/qv4proxy.cpp b/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
index 24676ffd00..1505eae426 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
+++ b/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
@@ -265,9 +265,9 @@ PropertyAttributes ProxyObject::virtualGetOwnProperty(const Managed *m, Property
|
|
||||||
ScopedProperty targetDesc(scope);
|
|
||||||
PropertyAttributes targetAttributes = target->getOwnProperty(id, targetDesc);
|
|
||||||
if (trapResult->isUndefined()) {
|
|
||||||
- p->value = Encode::undefined();
|
|
||||||
- if (targetAttributes == Attr_Invalid) {
|
|
||||||
+ if (p)
|
|
||||||
p->value = Encode::undefined();
|
|
||||||
+ if (targetAttributes == Attr_Invalid) {
|
|
||||||
return Attr_Invalid;
|
|
||||||
}
|
|
||||||
if (!targetAttributes.isConfigurable() || !target->isExtensible()) {
|
|
||||||
@@ -295,8 +295,10 @@ PropertyAttributes ProxyObject::virtualGetOwnProperty(const Managed *m, Property
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
- p->value = resultDesc->value;
|
|
||||||
- p->set = resultDesc->set;
|
|
||||||
+ if (p) {
|
|
||||||
+ p->value = resultDesc->value;
|
|
||||||
+ p->set = resultDesc->set;
|
|
||||||
+ }
|
|
||||||
return resultAttributes;
|
|
||||||
}
|
|
||||||
|
|
||||||
diff --git a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
index 1e10841430..3a9d1bfb4c 100644
|
|
||||||
--- a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
+++ b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
@@ -382,7 +382,7 @@ private slots:
|
|
||||||
void semicolonAfterProperty();
|
|
||||||
void hugeStack();
|
|
||||||
void variantConversionMethod();
|
|
||||||
-
|
|
||||||
+ void proxyHandlerTraps();
|
|
||||||
void gcCrashRegressionTest();
|
|
||||||
|
|
||||||
private:
|
|
||||||
@@ -9306,6 +9306,35 @@ void tst_qqmlecmascript::variantConversionMethod()
|
|
||||||
QCOMPARE(obj.funcCalled, QLatin1String("QModelIndex"));
|
|
||||||
}
|
|
||||||
|
|
||||||
+void tst_qqmlecmascript::proxyHandlerTraps()
|
|
||||||
+{
|
|
||||||
+ const QString expression = QStringLiteral(R"SNIPPET(
|
|
||||||
+ (function(){
|
|
||||||
+ const target = {
|
|
||||||
+ prop: 47
|
|
||||||
+ };
|
|
||||||
+ const handler = {
|
|
||||||
+ getOwnPropertyDescriptor(target, prop) {
|
|
||||||
+ return { configurable: true, enumerable: true, value: 47 };
|
|
||||||
+ }
|
|
||||||
+ };
|
|
||||||
+ const proxy = new Proxy(target, handler);
|
|
||||||
+
|
|
||||||
+ // QTBUG-88786
|
|
||||||
+ if (!proxy.propertyIsEnumerable("prop"))
|
|
||||||
+ throw Error("FAIL: propertyisEnumerable");
|
|
||||||
+ if (!proxy.hasOwnProperty("prop"))
|
|
||||||
+ throw Error("FAIL: hasOwnProperty");
|
|
||||||
+
|
|
||||||
+ return "SUCCESS";
|
|
||||||
+ })()
|
|
||||||
+ )SNIPPET");
|
|
||||||
+
|
|
||||||
+ QJSEngine engine;
|
|
||||||
+ QJSValue value = engine.evaluate(expression);
|
|
||||||
+ QVERIFY(value.isString() && value.toString() == QStringLiteral("SUCCESS"));
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
QTEST_MAIN(tst_qqmlecmascript)
|
|
||||||
|
|
||||||
#include "tst_qqmlecmascript.moc"
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From fe8c06d78a1b2f261806797c6f15f313cc3fc490 Mon Sep 17 00:00:00 2001
|
From 44ac7f39d874202163926f505a46179c084154b9 Mon Sep 17 00:00:00 2001
|
||||||
From: Maximilian Goldstein <max.goldstein@qt.io>
|
From: Maximilian Goldstein <max.goldstein@qt.io>
|
||||||
Date: Tue, 23 Feb 2021 16:10:44 +0100
|
Date: Tue, 23 Feb 2021 16:10:44 +0100
|
||||||
Subject: [PATCH 38/41] qqmldelegatemodel: Fix out of bounds cache removal
|
Subject: [PATCH 13/20] qqmldelegatemodel: Fix out of bounds cache removal
|
||||||
|
|
||||||
Pick-to: 5.15 6.0 6.1
|
Pick-to: 5.15 6.0 6.1
|
||||||
Task-number: QTBUG-91276
|
Task-number: QTBUG-91276
|
||||||
@ -13,7 +13,7 @@ Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|||||||
1 file changed, 1 insertion(+), 1 deletion(-)
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
diff --git a/src/qmlmodels/qqmldelegatemodel.cpp b/src/qmlmodels/qqmldelegatemodel.cpp
|
diff --git a/src/qmlmodels/qqmldelegatemodel.cpp b/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
index 8ce3da1cf1..8a74a854f4 100644
|
index 381726291d..53e511303e 100644
|
||||||
--- a/src/qmlmodels/qqmldelegatemodel.cpp
|
--- a/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
+++ b/src/qmlmodels/qqmldelegatemodel.cpp
|
+++ b/src/qmlmodels/qqmldelegatemodel.cpp
|
||||||
@@ -1621,7 +1621,7 @@ void QQmlDelegateModelPrivate::itemsRemoved(
|
@@ -1621,7 +1621,7 @@ void QQmlDelegateModelPrivate::itemsRemoved(
|
||||||
@ -26,5 +26,5 @@ index 8ce3da1cf1..8a74a854f4 100644
|
|||||||
|
|
||||||
for (int i = 1; i < m_groupCount; ++i) {
|
for (int i = 1; i < m_groupCount; ++i) {
|
||||||
--
|
--
|
||||||
2.33.1
|
2.35.1
|
||||||
|
|
@ -1,44 +0,0 @@
|
|||||||
From 514a69659a56fda410f4ab955f03c0d2a38b52f9 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Piotr Mikolajczyk <piotr.mikolajczyk@qt.io>
|
|
||||||
Date: Tue, 10 Nov 2020 14:58:12 +0100
|
|
||||||
Subject: [PATCH 14/28] Accessibility event is sent on item's geometry change
|
|
||||||
MIME-Version: 1.0
|
|
||||||
Content-Type: text/plain; charset=UTF-8
|
|
||||||
Content-Transfer-Encoding: 8bit
|
|
||||||
|
|
||||||
In case of enabled accessibility, whenever the geometry
|
|
||||||
of a QQuickItem changes, accessibility module is notified
|
|
||||||
by a LocationChange event. This enables responding to this
|
|
||||||
by for example moving the accessibility frame on the screen.
|
|
||||||
|
|
||||||
Task-number: QTBUG-79611
|
|
||||||
Change-Id: I808e835384ef42bba2e9aabecf4be3cda07859fe
|
|
||||||
Reviewed-by: Jan Arve Sæther <jan-arve.saether@qt.io>
|
|
||||||
(cherry picked from commit def81070668f101e1e2cbb46d586bbab64c8e00f)
|
|
||||||
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
|
|
||||||
---
|
|
||||||
src/quick/items/qquickitem.cpp | 8 ++++++++
|
|
||||||
1 file changed, 8 insertions(+)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquickitem.cpp b/src/quick/items/qquickitem.cpp
|
|
||||||
index 67c4611d9e..ddd67522b9 100644
|
|
||||||
--- a/src/quick/items/qquickitem.cpp
|
|
||||||
+++ b/src/quick/items/qquickitem.cpp
|
|
||||||
@@ -3753,6 +3753,14 @@ void QQuickItem::geometryChanged(const QRectF &newGeometry, const QRectF &oldGeo
|
|
||||||
emit widthChanged();
|
|
||||||
if (change.heightChange())
|
|
||||||
emit heightChanged();
|
|
||||||
+#if QT_CONFIG(accessibility)
|
|
||||||
+ if (QAccessible::isActive()) {
|
|
||||||
+ if (QObject *acc = QQuickAccessibleAttached::findAccessible(this)) {
|
|
||||||
+ QAccessibleEvent ev(acc, QAccessible::LocationChanged);
|
|
||||||
+ QAccessible::updateAccessibility(&ev);
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+#endif
|
|
||||||
}
|
|
||||||
|
|
||||||
/*!
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,7 +1,7 @@
|
|||||||
From d7ede3462d7e12e4fa60a6efa1b83e2157be4b00 Mon Sep 17 00:00:00 2001
|
From f8f6b9d084decbad8ee90880493c413ac90c2911 Mon Sep 17 00:00:00 2001
|
||||||
From: Andreas Hartmetz <andreas@ixgreen.de>
|
From: Andreas Hartmetz <andreas@ixgreen.de>
|
||||||
Date: Fri, 5 Mar 2021 12:41:06 +0100
|
Date: Fri, 5 Mar 2021 12:41:06 +0100
|
||||||
Subject: [PATCH 39/41] QQuickWindow: don't leak old screenChanged connections
|
Subject: [PATCH 14/20] QQuickWindow: don't leak old screenChanged connections
|
||||||
|
|
||||||
Connections could accumulate. Because the newest one was invoked
|
Connections could accumulate. Because the newest one was invoked
|
||||||
last due to how signal-slot invocations are ordered, rendering
|
last due to how signal-slot invocations are ordered, rendering
|
||||||
@ -23,7 +23,7 @@ Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
|||||||
1 file changed, 3 insertions(+), 4 deletions(-)
|
1 file changed, 3 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquickwindow.cpp b/src/quick/items/qquickwindow.cpp
|
diff --git a/src/quick/items/qquickwindow.cpp b/src/quick/items/qquickwindow.cpp
|
||||||
index 9ff91eb9a0..fbb807ca96 100644
|
index eea1e93f32..c956c85091 100644
|
||||||
--- a/src/quick/items/qquickwindow.cpp
|
--- a/src/quick/items/qquickwindow.cpp
|
||||||
+++ b/src/quick/items/qquickwindow.cpp
|
+++ b/src/quick/items/qquickwindow.cpp
|
||||||
@@ -450,15 +450,14 @@ void QQuickWindow::physicalDpiChanged()
|
@@ -450,15 +450,14 @@ void QQuickWindow::physicalDpiChanged()
|
||||||
@ -46,5 +46,5 @@ index 9ff91eb9a0..fbb807ca96 100644
|
|||||||
|
|
||||||
d->forcePolish();
|
d->forcePolish();
|
||||||
--
|
--
|
||||||
2.33.1
|
2.35.1
|
||||||
|
|
@ -0,0 +1,73 @@
|
|||||||
|
From 743ae2a4d59eccc4720390c4c757b081eb2c6bfa Mon Sep 17 00:00:00 2001
|
||||||
|
From: =?UTF-8?q?Jan=20Arve=20S=C3=A6ther?= <jan-arve.saether@qt.io>
|
||||||
|
Date: Thu, 3 Sep 2020 10:51:01 +0200
|
||||||
|
Subject: [PATCH 15/20] Fix TapHandler so that it actually registers a tap
|
||||||
|
|
||||||
|
This bug caused all quick examples that used the
|
||||||
|
shared\LauncherList.qml to be broken.
|
||||||
|
|
||||||
|
In QtGui, QSinglePointEvent will construct itself with a point id of 0
|
||||||
|
if there is a valid point, and with a point id of -1 if the point is
|
||||||
|
invalid (the default constructor does the latter).
|
||||||
|
However, QQuickSinglePointHandler::wantsPointerEvent() did not agree
|
||||||
|
with that, because it assumed that a point id of 0 meant
|
||||||
|
uninitialized/invalid point.
|
||||||
|
The fix is to change QQuickSinglePointHandler::wantsPointerEvent() and
|
||||||
|
QQuickHandlerPoint so that it assumes that the id -1 is now an invalid
|
||||||
|
point, (instead of 0)
|
||||||
|
|
||||||
|
Change-Id: I8c9683dfe06ebb77c5342a26f08174b67e7cbd90
|
||||||
|
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
||||||
|
(cherry picked from commit 8d3a91016506fd0afedb0be535f7c34a4ca762f6)
|
||||||
|
---
|
||||||
|
src/quick/handlers/qquickhandlerpoint.cpp | 4 ++--
|
||||||
|
src/quick/handlers/qquicksinglepointhandler.cpp | 4 ++--
|
||||||
|
2 files changed, 4 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/quick/handlers/qquickhandlerpoint.cpp b/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
index 72efdfd0f4..6aef3545dd 100644
|
||||||
|
--- a/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
+++ b/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
@@ -82,7 +82,7 @@ void QQuickHandlerPoint::localize(QQuickItem *item)
|
||||||
|
|
||||||
|
void QQuickHandlerPoint::reset()
|
||||||
|
{
|
||||||
|
- m_id = 0;
|
||||||
|
+ m_id = -1;
|
||||||
|
m_uniqueId = QPointingDeviceUniqueId();
|
||||||
|
m_position = QPointF();
|
||||||
|
m_scenePosition = QPointF();
|
||||||
|
@@ -165,7 +165,7 @@ void QQuickHandlerPoint::reset(const QVector<QQuickHandlerPoint> &points)
|
||||||
|
pressureSum += point.pressure();
|
||||||
|
ellipseDiameterSum += point.ellipseDiameters();
|
||||||
|
}
|
||||||
|
- m_id = 0;
|
||||||
|
+ m_id = -1;
|
||||||
|
m_uniqueId = QPointingDeviceUniqueId();
|
||||||
|
// all points are required to be from the same event, so pressed buttons and modifiers should be the same
|
||||||
|
m_pressedButtons = points.first().pressedButtons();
|
||||||
|
diff --git a/src/quick/handlers/qquicksinglepointhandler.cpp b/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
index b51f53b74f..89081b4e84 100644
|
||||||
|
--- a/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
+++ b/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
@@ -75,7 +75,7 @@ bool QQuickSinglePointHandler::wantsPointerEvent(QQuickPointerEvent *event)
|
||||||
|
if (!QQuickPointerDeviceHandler::wantsPointerEvent(event))
|
||||||
|
return false;
|
||||||
|
|
||||||
|
- if (d->pointInfo.id()) {
|
||||||
|
+ if (d->pointInfo.id() != -1) {
|
||||||
|
// We already know which one we want, so check whether it's there.
|
||||||
|
// It's expected to be an update or a release.
|
||||||
|
// If we no longer want it, cancel the grab.
|
||||||
|
@@ -125,7 +125,7 @@ bool QQuickSinglePointHandler::wantsPointerEvent(QQuickPointerEvent *event)
|
||||||
|
chosen->setAccepted();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
- return d->pointInfo.id();
|
||||||
|
+ return d->pointInfo.id() != -1;
|
||||||
|
}
|
||||||
|
|
||||||
|
void QQuickSinglePointHandler::handlePointerEventImpl(QQuickPointerEvent *event)
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,36 +0,0 @@
|
|||||||
From 3c075bfd348306cd553caddb9f8bf3f596666636 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Alessandro Portale <alessandro.portale@qt.io>
|
|
||||||
Date: Wed, 25 Nov 2020 23:43:03 +0100
|
|
||||||
Subject: [PATCH 15/28] qmltypes.prf: Take abi into account for
|
|
||||||
*_metatypes.json file names
|
|
||||||
|
|
||||||
The lib/metatypes/*_metatypes.json file names contain the ABI. When
|
|
||||||
constructing the qmltyperegistrar command, the right file names
|
|
||||||
with that ABI part need to be passed as "foreign-types".
|
|
||||||
|
|
||||||
Fixes: QTBUG-85888
|
|
||||||
Fixes: QTBUG-87117
|
|
||||||
Change-Id: I20daac1b6b9a27c5ac48b3c2c685e2fed301e213
|
|
||||||
Reviewed-by: Assam Boudjelthia <assam.boudjelthia@qt.io>
|
|
||||||
(cherry picked from commit acc5e48a90d0daeccb28175b80ab6b52cac5d84a)
|
|
||||||
---
|
|
||||||
src/qmltyperegistrar/qmltypes.prf | 3 ++-
|
|
||||||
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/src/qmltyperegistrar/qmltypes.prf b/src/qmltyperegistrar/qmltypes.prf
|
|
||||||
index 354fa1736f..2cc0027b7e 100644
|
|
||||||
--- a/src/qmltyperegistrar/qmltypes.prf
|
|
||||||
+++ b/src/qmltyperegistrar/qmltypes.prf
|
|
||||||
@@ -44,7 +44,8 @@ qt_module_deps = $$replace(qt_module_deps, _private$, '')
|
|
||||||
qt_module_deps = $$unique(qt_module_deps)
|
|
||||||
|
|
||||||
for(dep, qt_module_deps) {
|
|
||||||
- METATYPES_FILENAME = $$lower($$eval(QT.$${dep}.module))_metatypes.json
|
|
||||||
+ android:ABI = _$${ANDROID_TARGET_ARCH}
|
|
||||||
+ METATYPES_FILENAME = $$lower($$eval(QT.$${dep}.module))$${ABI}_metatypes.json
|
|
||||||
INSTALLED_METATYPES = $$[QT_INSTALL_LIBS]/metatypes/$$METATYPES_FILENAME
|
|
||||||
isEmpty(MODULE_BASE_OUTDIR) {
|
|
||||||
QML_FOREIGN_METATYPES += $$INSTALLED_METATYPES
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -0,0 +1,61 @@
|
|||||||
|
From 84e0c1e0973ca7467089dc0227a2b9ac4aef52fa Mon Sep 17 00:00:00 2001
|
||||||
|
From: Albert Astals Cid <aacid@kde.org>
|
||||||
|
Date: Tue, 16 Nov 2021 22:43:37 +0100
|
||||||
|
Subject: [PATCH 16/20] Revert "Fix TapHandler so that it actually registers a
|
||||||
|
tap"
|
||||||
|
|
||||||
|
This reverts commit 36e8ccd434f948e4f11a8f9d59139ec072e41ff5.
|
||||||
|
|
||||||
|
It's causing regresions
|
||||||
|
---
|
||||||
|
src/quick/handlers/qquickhandlerpoint.cpp | 4 ++--
|
||||||
|
src/quick/handlers/qquicksinglepointhandler.cpp | 4 ++--
|
||||||
|
2 files changed, 4 insertions(+), 4 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/quick/handlers/qquickhandlerpoint.cpp b/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
index 6aef3545dd..72efdfd0f4 100644
|
||||||
|
--- a/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
+++ b/src/quick/handlers/qquickhandlerpoint.cpp
|
||||||
|
@@ -82,7 +82,7 @@ void QQuickHandlerPoint::localize(QQuickItem *item)
|
||||||
|
|
||||||
|
void QQuickHandlerPoint::reset()
|
||||||
|
{
|
||||||
|
- m_id = -1;
|
||||||
|
+ m_id = 0;
|
||||||
|
m_uniqueId = QPointingDeviceUniqueId();
|
||||||
|
m_position = QPointF();
|
||||||
|
m_scenePosition = QPointF();
|
||||||
|
@@ -165,7 +165,7 @@ void QQuickHandlerPoint::reset(const QVector<QQuickHandlerPoint> &points)
|
||||||
|
pressureSum += point.pressure();
|
||||||
|
ellipseDiameterSum += point.ellipseDiameters();
|
||||||
|
}
|
||||||
|
- m_id = -1;
|
||||||
|
+ m_id = 0;
|
||||||
|
m_uniqueId = QPointingDeviceUniqueId();
|
||||||
|
// all points are required to be from the same event, so pressed buttons and modifiers should be the same
|
||||||
|
m_pressedButtons = points.first().pressedButtons();
|
||||||
|
diff --git a/src/quick/handlers/qquicksinglepointhandler.cpp b/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
index 89081b4e84..b51f53b74f 100644
|
||||||
|
--- a/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
+++ b/src/quick/handlers/qquicksinglepointhandler.cpp
|
||||||
|
@@ -75,7 +75,7 @@ bool QQuickSinglePointHandler::wantsPointerEvent(QQuickPointerEvent *event)
|
||||||
|
if (!QQuickPointerDeviceHandler::wantsPointerEvent(event))
|
||||||
|
return false;
|
||||||
|
|
||||||
|
- if (d->pointInfo.id() != -1) {
|
||||||
|
+ if (d->pointInfo.id()) {
|
||||||
|
// We already know which one we want, so check whether it's there.
|
||||||
|
// It's expected to be an update or a release.
|
||||||
|
// If we no longer want it, cancel the grab.
|
||||||
|
@@ -125,7 +125,7 @@ bool QQuickSinglePointHandler::wantsPointerEvent(QQuickPointerEvent *event)
|
||||||
|
chosen->setAccepted();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
- return d->pointInfo.id() != -1;
|
||||||
|
+ return d->pointInfo.id();
|
||||||
|
}
|
||||||
|
|
||||||
|
void QQuickSinglePointHandler::handlePointerEventImpl(QQuickPointerEvent *event)
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,68 +0,0 @@
|
|||||||
From 37c290c5b8659256ff574a06a3df2c363ae446b5 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maximilian Goldstein <max.goldstein@qt.io>
|
|
||||||
Date: Wed, 2 Dec 2020 13:08:57 +0100
|
|
||||||
Subject: [PATCH 16/28] qv4qmlcontext: Fix bounded signal expressions when
|
|
||||||
debugging
|
|
||||||
|
|
||||||
Fixes: QTBUG-83599
|
|
||||||
Change-Id: I8909f0b2d3eca909512b99c172c8dc5e93e48482
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit bad85119bf35468292cfd80ecc934b66515f0c68)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/qml/jsruntime/qv4qmlcontext.cpp | 4 ++--
|
|
||||||
.../qml/debugger/qv4debugger/tst_qv4debugger.cpp | 12 +++++++++---
|
|
||||||
2 files changed, 11 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/qml/jsruntime/qv4qmlcontext.cpp b/src/qml/jsruntime/qv4qmlcontext.cpp
|
|
||||||
index e2d3b98ff6..6eece147a6 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4qmlcontext.cpp
|
|
||||||
+++ b/src/qml/jsruntime/qv4qmlcontext.cpp
|
|
||||||
@@ -466,9 +466,9 @@ ReturnedValue QQmlContextWrapper::resolveQmlContextPropertyLookupGetter(Lookup *
|
|
||||||
return static_cast<Heap::CallContext *>(ctx)->locals[index].asReturnedValue();
|
|
||||||
}
|
|
||||||
|
|
||||||
- // Skip only block contexts within the current call context.
|
|
||||||
+ // Skip only block and call contexts.
|
|
||||||
// Other contexts need a regular QML property lookup. See below.
|
|
||||||
- if (ctx->type != Heap::ExecutionContext::Type_BlockContext)
|
|
||||||
+ if (ctx->type != Heap::ExecutionContext::Type_BlockContext && ctx->type != Heap::ExecutionContext::Type_CallContext)
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
|
|
||||||
diff --git a/tests/auto/qml/debugger/qv4debugger/tst_qv4debugger.cpp b/tests/auto/qml/debugger/qv4debugger/tst_qv4debugger.cpp
|
|
||||||
index 84f5eebd10..e3cbeb9891 100644
|
|
||||||
--- a/tests/auto/qml/debugger/qv4debugger/tst_qv4debugger.cpp
|
|
||||||
+++ b/tests/auto/qml/debugger/qv4debugger/tst_qv4debugger.cpp
|
|
||||||
@@ -910,19 +910,25 @@ void tst_qv4debugger::signalParameters()
|
|
||||||
component.setData("import QtQml 2.12\n"
|
|
||||||
"QtObject {\n"
|
|
||||||
" id: root\n"
|
|
||||||
- " property string result\n"
|
|
||||||
+ " property string result: 'unset'\n"
|
|
||||||
+ " property string resultCallbackInternal: 'unset'\n"
|
|
||||||
+ " property string resultCallbackExternal: 'unset'\n"
|
|
||||||
" signal signalWithArg(string textArg)\n"
|
|
||||||
+ " function call(callback) { callback(); }\n"
|
|
||||||
+ " function externalCallback() { root.resultCallbackExternal = textArg; }\n"
|
|
||||||
" property Connections connections : Connections {\n"
|
|
||||||
" target: root\n"
|
|
||||||
- " onSignalWithArg: { root.result = textArg; }\n"
|
|
||||||
+ " onSignalWithArg: { root.result = textArg; call(function() { root.resultCallbackInternal = textArg; }); call(externalCallback); }\n"
|
|
||||||
" }\n"
|
|
||||||
" Component.onCompleted: signalWithArg('something')\n"
|
|
||||||
"}", QUrl("test.qml"));
|
|
||||||
|
|
||||||
- QVERIFY(component.isReady());
|
|
||||||
+ QVERIFY2(component.isReady(), qPrintable(component.errorString()));
|
|
||||||
QScopedPointer<QObject> obj(component.create());
|
|
||||||
QVERIFY(obj);
|
|
||||||
QCOMPARE(obj->property("result").toString(), QLatin1String("something"));
|
|
||||||
+ QCOMPARE(obj->property("resultCallbackInternal").toString(), QLatin1String("something"));
|
|
||||||
+ QCOMPARE(obj->property("resultCallbackExternal").toString(), QLatin1String("unset"));
|
|
||||||
}
|
|
||||||
|
|
||||||
QTEST_MAIN(tst_qv4debugger)
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -0,0 +1,36 @@
|
|||||||
|
From c6e595e7fbbe80c8db7ae33d8af05a4fd946a2f5 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Marc Mutz <marc.mutz@qt.io>
|
||||||
|
Date: Tue, 21 Dec 2021 09:20:17 +0100
|
||||||
|
Subject: [PATCH 17/20] QQmlJs::FixedPoolArray: fix UB (precondition violation)
|
||||||
|
in allocate()
|
||||||
|
|
||||||
|
Says ubsan:
|
||||||
|
|
||||||
|
qqmljsfixedpoolarray_p.h:90:19: runtime error: null pointer passed as argument 2, which is declared to never be null
|
||||||
|
|
||||||
|
Fix, like in so many other places, by a size check.
|
||||||
|
|
||||||
|
Pick-to: 6.3 6.2 5.15
|
||||||
|
Change-Id: I9181d6ecb467c2dc726978ce7f93b35a6bf2f944
|
||||||
|
Reviewed-by: Lars Knoll <lars.knoll@qt.io>
|
||||||
|
(cherry picked from commit d74e931f3fc2587ac6d1e2930acbbe54ea5be2b5)
|
||||||
|
---
|
||||||
|
src/qml/common/qqmljsfixedpoolarray_p.h | 2 +-
|
||||||
|
1 file changed, 1 insertion(+), 1 deletion(-)
|
||||||
|
|
||||||
|
diff --git a/src/qml/common/qqmljsfixedpoolarray_p.h b/src/qml/common/qqmljsfixedpoolarray_p.h
|
||||||
|
index b65b994d6c..15a8cd6878 100644
|
||||||
|
--- a/src/qml/common/qqmljsfixedpoolarray_p.h
|
||||||
|
+++ b/src/qml/common/qqmljsfixedpoolarray_p.h
|
||||||
|
@@ -86,7 +86,7 @@ public:
|
||||||
|
if (QTypeInfo<T>::isComplex) {
|
||||||
|
for (int i = 0; i < count; ++i)
|
||||||
|
new (data + i) T(vector.at(i));
|
||||||
|
- } else {
|
||||||
|
+ } else if (count) {
|
||||||
|
memcpy(data, static_cast<const void*>(vector.constData()), count * sizeof(T));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,32 +0,0 @@
|
|||||||
From 30c7a6c6a874264800d398df8c3ec65f30707c92 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Li Xinwei <1326710505@qq.com>
|
|
||||||
Date: Tue, 8 Dec 2020 15:36:01 +0800
|
|
||||||
Subject: [PATCH 17/28] Use load(qt_tool) for qmltime
|
|
||||||
|
|
||||||
The qmltime should be a tool, not a normal executable or an app.
|
|
||||||
|
|
||||||
Change-Id: I64c76877907297a6a817ba5903786bcc7fba8fdd
|
|
||||||
Reviewed-by: Alexandru Croitor <alexandru.croitor@qt.io>
|
|
||||||
(cherry picked from commit e6e262da1423bcb7cfe3db9f83fe0df54483c8d4)
|
|
||||||
---
|
|
||||||
tools/qmltime/qmltime.pro | 3 ++-
|
|
||||||
1 file changed, 2 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/tools/qmltime/qmltime.pro b/tools/qmltime/qmltime.pro
|
|
||||||
index c915f6e8c1..366d90f75b 100644
|
|
||||||
--- a/tools/qmltime/qmltime.pro
|
|
||||||
+++ b/tools/qmltime/qmltime.pro
|
|
||||||
@@ -1,4 +1,3 @@
|
|
||||||
-TEMPLATE = app
|
|
||||||
TARGET = qmltime
|
|
||||||
QT += qml quick
|
|
||||||
QT += quick-private
|
|
||||||
@@ -12,3 +11,5 @@ QMAKE_TARGET_DESCRIPTION = QML Time
|
|
||||||
|
|
||||||
SOURCES += qmltime.cpp
|
|
||||||
HEADERS += qmltime.h
|
|
||||||
+
|
|
||||||
+load(qt_tool)
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -0,0 +1,138 @@
|
|||||||
|
From 9bb03b5dcc21275986df3d8b0efb6f28cdc583ec Mon Sep 17 00:00:00 2001
|
||||||
|
From: Wang Chuan <ouchuanm@outlook.com>
|
||||||
|
Date: Mon, 5 Apr 2021 11:41:48 +0800
|
||||||
|
Subject: [PATCH 18/20] QQuickTextInput: update cursor rectangle after padding
|
||||||
|
changed
|
||||||
|
|
||||||
|
The position of cursor delegate needs to be updated when we change
|
||||||
|
padding, otherwise it will be in a wrong position.
|
||||||
|
|
||||||
|
Fixes: QTBUG-91867
|
||||||
|
Pick-to: 5.12 5.15 6.0 6.1
|
||||||
|
Change-Id: I89ca84fe893ebf517ab67890196eede14a4055d7
|
||||||
|
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
||||||
|
(cherry picked from commit d98694c4023881673259ba040c10df7e71ec3d37)
|
||||||
|
---
|
||||||
|
src/quick/items/qquicktextinput.cpp | 5 ++++
|
||||||
|
.../checkCursorDelegateWhenPaddingChanged.qml | 16 ++++++++++
|
||||||
|
.../qquicktextinput/tst_qquicktextinput.cpp | 30 +++++++++++++++++++
|
||||||
|
3 files changed, 51 insertions(+)
|
||||||
|
create mode 100644 tests/auto/quick/qquicktextinput/data/checkCursorDelegateWhenPaddingChanged.qml
|
||||||
|
|
||||||
|
diff --git a/src/quick/items/qquicktextinput.cpp b/src/quick/items/qquicktextinput.cpp
|
||||||
|
index 079bf58abe..7d0d05700a 100644
|
||||||
|
--- a/src/quick/items/qquicktextinput.cpp
|
||||||
|
+++ b/src/quick/items/qquicktextinput.cpp
|
||||||
|
@@ -2952,6 +2952,7 @@ void QQuickTextInputPrivate::setTopPadding(qreal value, bool reset)
|
||||||
|
}
|
||||||
|
if ((!reset && !qFuzzyCompare(oldPadding, value)) || (reset && !qFuzzyCompare(oldPadding, padding()))) {
|
||||||
|
updateLayout();
|
||||||
|
+ q->updateCursorRectangle();
|
||||||
|
emit q->topPaddingChanged();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -2966,6 +2967,7 @@ void QQuickTextInputPrivate::setLeftPadding(qreal value, bool reset)
|
||||||
|
}
|
||||||
|
if ((!reset && !qFuzzyCompare(oldPadding, value)) || (reset && !qFuzzyCompare(oldPadding, padding()))) {
|
||||||
|
updateLayout();
|
||||||
|
+ q->updateCursorRectangle();
|
||||||
|
emit q->leftPaddingChanged();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -2980,6 +2982,7 @@ void QQuickTextInputPrivate::setRightPadding(qreal value, bool reset)
|
||||||
|
}
|
||||||
|
if ((!reset && !qFuzzyCompare(oldPadding, value)) || (reset && !qFuzzyCompare(oldPadding, padding()))) {
|
||||||
|
updateLayout();
|
||||||
|
+ q->updateCursorRectangle();
|
||||||
|
emit q->rightPaddingChanged();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -2994,6 +2997,7 @@ void QQuickTextInputPrivate::setBottomPadding(qreal value, bool reset)
|
||||||
|
}
|
||||||
|
if ((!reset && !qFuzzyCompare(oldPadding, value)) || (reset && !qFuzzyCompare(oldPadding, padding()))) {
|
||||||
|
updateLayout();
|
||||||
|
+ q->updateCursorRectangle();
|
||||||
|
emit q->bottomPaddingChanged();
|
||||||
|
}
|
||||||
|
}
|
||||||
|
@@ -4712,6 +4716,7 @@ void QQuickTextInput::setPadding(qreal padding)
|
||||||
|
|
||||||
|
d->extra.value().padding = padding;
|
||||||
|
d->updateLayout();
|
||||||
|
+ updateCursorRectangle();
|
||||||
|
emit paddingChanged();
|
||||||
|
if (!d->extra.isAllocated() || !d->extra->explicitTopPadding)
|
||||||
|
emit topPaddingChanged();
|
||||||
|
diff --git a/tests/auto/quick/qquicktextinput/data/checkCursorDelegateWhenPaddingChanged.qml b/tests/auto/quick/qquicktextinput/data/checkCursorDelegateWhenPaddingChanged.qml
|
||||||
|
new file mode 100644
|
||||||
|
index 0000000000..e6f07b4687
|
||||||
|
--- /dev/null
|
||||||
|
+++ b/tests/auto/quick/qquicktextinput/data/checkCursorDelegateWhenPaddingChanged.qml
|
||||||
|
@@ -0,0 +1,16 @@
|
||||||
|
+import QtQuick 2.12
|
||||||
|
+
|
||||||
|
+Rectangle {
|
||||||
|
+ width: 200
|
||||||
|
+ height: 200
|
||||||
|
+ TextInput {
|
||||||
|
+ objectName: "textInput"
|
||||||
|
+ leftPadding: 10
|
||||||
|
+ focus: true
|
||||||
|
+ cursorDelegate: Rectangle {
|
||||||
|
+ objectName: "cursorDelegate"
|
||||||
|
+ width: 5
|
||||||
|
+ color: "red"
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
+}
|
||||||
|
diff --git a/tests/auto/quick/qquicktextinput/tst_qquicktextinput.cpp b/tests/auto/quick/qquicktextinput/tst_qquicktextinput.cpp
|
||||||
|
index 2e64c80b85..ac502bcb28 100644
|
||||||
|
--- a/tests/auto/quick/qquicktextinput/tst_qquicktextinput.cpp
|
||||||
|
+++ b/tests/auto/quick/qquicktextinput/tst_qquicktextinput.cpp
|
||||||
|
@@ -236,6 +236,7 @@ private slots:
|
||||||
|
void QTBUG_51115_readOnlyResetsSelection();
|
||||||
|
void QTBUG_77814_InsertRemoveNoSelection();
|
||||||
|
|
||||||
|
+ void checkCursorDelegateWhenPaddingChanged();
|
||||||
|
private:
|
||||||
|
void simulateKey(QWindow *, int key);
|
||||||
|
|
||||||
|
@@ -7054,6 +7055,35 @@ void tst_qquicktextinput::QTBUG_77814_InsertRemoveNoSelection()
|
||||||
|
QCOMPARE(textInput->selectedText(), QString());
|
||||||
|
}
|
||||||
|
|
||||||
|
+void tst_qquicktextinput::checkCursorDelegateWhenPaddingChanged()
|
||||||
|
+{
|
||||||
|
+ QQuickView view;
|
||||||
|
+ view.setSource(testFileUrl("checkCursorDelegateWhenPaddingChanged.qml"));
|
||||||
|
+ view.show();
|
||||||
|
+ QVERIFY(QTest::qWaitForWindowExposed(&view));
|
||||||
|
+
|
||||||
|
+ QQuickTextInput *textInput = view.rootObject()->findChild<QQuickTextInput *>("textInput");
|
||||||
|
+ QVERIFY(textInput);
|
||||||
|
+
|
||||||
|
+ QQuickItem *cursorDelegate = textInput->findChild<QQuickItem *>("cursorDelegate");
|
||||||
|
+ QVERIFY(cursorDelegate);
|
||||||
|
+
|
||||||
|
+ QCOMPARE(cursorDelegate->x(), textInput->leftPadding());
|
||||||
|
+ QCOMPARE(cursorDelegate->y(), textInput->topPadding());
|
||||||
|
+
|
||||||
|
+ textInput->setPadding(5);
|
||||||
|
+ QCOMPARE(cursorDelegate->x(), textInput->leftPadding());
|
||||||
|
+ QCOMPARE(cursorDelegate->y(), textInput->topPadding());
|
||||||
|
+
|
||||||
|
+ textInput->setTopPadding(10);
|
||||||
|
+ QCOMPARE(cursorDelegate->x(), textInput->leftPadding());
|
||||||
|
+ QCOMPARE(cursorDelegate->y(), textInput->topPadding());
|
||||||
|
+
|
||||||
|
+ textInput->setLeftPadding(10);
|
||||||
|
+ QCOMPARE(cursorDelegate->x(), textInput->leftPadding());
|
||||||
|
+ QCOMPARE(cursorDelegate->y(), textInput->topPadding());
|
||||||
|
+}
|
||||||
|
+
|
||||||
|
QTEST_MAIN(tst_qquicktextinput)
|
||||||
|
|
||||||
|
#include "tst_qquicktextinput.moc"
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,71 +0,0 @@
|
|||||||
From 83100a84f2b0068b4cf725896bbb810415908334 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Maximilian Goldstein <max.goldstein@qt.io>
|
|
||||||
Date: Tue, 8 Dec 2020 09:26:36 +0100
|
|
||||||
Subject: [PATCH 18/28] qqmlistmodel: Fix crash when modelCache is null
|
|
||||||
|
|
||||||
Fixes: QTBUG-89173
|
|
||||||
Change-Id: Ife82518808fc5504ec42407e80ed3de89ed4adeb
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
(cherry picked from commit c3860cd04bbc089ef95bc441a1f8f1e46f9606f8)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/qmlmodels/qqmllistmodel.cpp | 2 +-
|
|
||||||
.../qml/qqmllistmodel/tst_qqmllistmodel.cpp | 22 +++++++++++++++++++
|
|
||||||
2 files changed, 23 insertions(+), 1 deletion(-)
|
|
||||||
|
|
||||||
diff --git a/src/qmlmodels/qqmllistmodel.cpp b/src/qmlmodels/qqmllistmodel.cpp
|
|
||||||
index e07951cab3..8830e08097 100644
|
|
||||||
--- a/src/qmlmodels/qqmllistmodel.cpp
|
|
||||||
+++ b/src/qmlmodels/qqmllistmodel.cpp
|
|
||||||
@@ -703,7 +703,7 @@ void ListModel::set(int elementIndex, QV4::Object *object, ListModel::SetElement
|
|
||||||
} else if (propertyValue->isNullOrUndefined()) {
|
|
||||||
if (reason == SetElement::WasJustInserted) {
|
|
||||||
QQmlError err;
|
|
||||||
- auto memberName = propertyName->toString(m_modelCache->engine())->toQString();
|
|
||||||
+ auto memberName = propertyName->toString(v4)->toQString();
|
|
||||||
err.setDescription(QString::fromLatin1("%1 is %2. Adding an object with a %2 member does not create a role for it.").arg(memberName, propertyValue->isNull() ? QLatin1String("null") : QLatin1String("undefined")));
|
|
||||||
qmlWarning(nullptr, err);
|
|
||||||
} else {
|
|
||||||
diff --git a/tests/auto/qml/qqmllistmodel/tst_qqmllistmodel.cpp b/tests/auto/qml/qqmllistmodel/tst_qqmllistmodel.cpp
|
|
||||||
index d54e3467b7..1953798a15 100644
|
|
||||||
--- a/tests/auto/qml/qqmllistmodel/tst_qqmllistmodel.cpp
|
|
||||||
+++ b/tests/auto/qml/qqmllistmodel/tst_qqmllistmodel.cpp
|
|
||||||
@@ -130,6 +130,7 @@ private slots:
|
|
||||||
void dynamic_roles_crash_QTBUG_38907();
|
|
||||||
void nestedListModelIteration();
|
|
||||||
void undefinedAppendShouldCauseError();
|
|
||||||
+ void nullPropertyCrash();
|
|
||||||
};
|
|
||||||
|
|
||||||
bool tst_qqmllistmodel::compareVariantList(const QVariantList &testList, QVariant object)
|
|
||||||
@@ -1723,6 +1724,27 @@ void tst_qqmllistmodel::undefinedAppendShouldCauseError()
|
|
||||||
QScopedPointer<QObject>(component.create());
|
|
||||||
}
|
|
||||||
|
|
||||||
+// QTBUG-89173
|
|
||||||
+void tst_qqmllistmodel::nullPropertyCrash()
|
|
||||||
+{
|
|
||||||
+ QQmlEngine engine;
|
|
||||||
+ QQmlComponent component(&engine);
|
|
||||||
+ component.setData(
|
|
||||||
+ R"(import QtQuick 2.15
|
|
||||||
+ ListView {
|
|
||||||
+ model: ListModel { id: listModel }
|
|
||||||
+
|
|
||||||
+ delegate: Item {}
|
|
||||||
+
|
|
||||||
+ Component.onCompleted: {
|
|
||||||
+ listModel.append({"a": "value1", "b":[{"c":"value2"}]})
|
|
||||||
+ listModel.append({"a": "value2", "b":[{"c":null}]})
|
|
||||||
+ }
|
|
||||||
+ })",
|
|
||||||
+ QUrl());
|
|
||||||
+ QTest::ignoreMessage(QtMsgType::QtWarningMsg, "<Unknown File>: c is null. Adding an object with a null member does not create a role for it.");
|
|
||||||
+ QScopedPointer<QObject>(component.create());
|
|
||||||
+}
|
|
||||||
|
|
||||||
QTEST_MAIN(tst_qqmllistmodel)
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,101 +0,0 @@
|
|||||||
From 1241e4f3c3ec010ae121f5d56c3e9405ec43231f Mon Sep 17 00:00:00 2001
|
|
||||||
From: Andy Shaw <andy.shaw@qt.io>
|
|
||||||
Date: Fri, 6 Nov 2020 13:30:12 +0100
|
|
||||||
Subject: [PATCH 19/28] Show a tableview even if the syncView has an empty
|
|
||||||
model
|
|
||||||
|
|
||||||
By showing the tableview, we can be sure that headerviews will be
|
|
||||||
visible even in the syncView has an empty model.
|
|
||||||
|
|
||||||
Fixes: QTBUG-87526
|
|
||||||
Change-Id: I68c8b119122a2d2f88c2afbeb2d6c71a83a3ce33
|
|
||||||
Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
|
|
||||||
(cherry picked from commit 27c254203b3e7dd6d3a4445feb205fbe98c32d30)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/quick/items/qquicktableview.cpp | 7 +--
|
|
||||||
.../qquicktableview/tst_qquicktableview.cpp | 43 +++++++++++++++++++
|
|
||||||
2 files changed, 45 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquicktableview.cpp b/src/quick/items/qquicktableview.cpp
|
|
||||||
index 7b73fcb393..1349d308d7 100644
|
|
||||||
--- a/src/quick/items/qquicktableview.cpp
|
|
||||||
+++ b/src/quick/items/qquicktableview.cpp
|
|
||||||
@@ -1760,11 +1760,8 @@ void QQuickTableViewPrivate::calculateTopLeft(QPoint &topLeftCell, QPointF &topL
|
|
||||||
const auto syncView_d = syncView->d_func();
|
|
||||||
|
|
||||||
if (syncView_d->loadedItems.isEmpty()) {
|
|
||||||
- // The sync view contains no loaded items. This probably means
|
|
||||||
- // that it has not been rebuilt yet. Which also means that
|
|
||||||
- // we cannot rebuild anything before this happens.
|
|
||||||
- topLeftCell.rx() = kEdgeIndexNotSet;
|
|
||||||
- topLeftCell.ry() = kEdgeIndexNotSet;
|
|
||||||
+ topLeftCell.rx() = 0;
|
|
||||||
+ topLeftCell.ry() = 0;
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
diff --git a/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp b/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp
|
|
||||||
index 54f73c6e0c..d489a873e4 100644
|
|
||||||
--- a/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp
|
|
||||||
+++ b/tests/auto/quick/qquicktableview/tst_qquicktableview.cpp
|
|
||||||
@@ -176,6 +176,7 @@ private slots:
|
|
||||||
void checkSyncView_connect_late_data();
|
|
||||||
void checkSyncView_connect_late();
|
|
||||||
void checkSyncView_pageFlicking();
|
|
||||||
+ void checkSyncView_emptyModel();
|
|
||||||
void delegateWithRequiredProperties();
|
|
||||||
void checkThatFetchMoreIsCalledWhenScrolledToTheEndOfTable();
|
|
||||||
void replaceModel();
|
|
||||||
@@ -2731,6 +2732,48 @@ void tst_QQuickTableView::checkSyncView_pageFlicking()
|
|
||||||
QVERIFY(tableViewPrivate->scheduledRebuildOptions & QQuickTableViewPrivate::RebuildOption::CalculateNewTopLeftRow);
|
|
||||||
}
|
|
||||||
|
|
||||||
+void tst_QQuickTableView::checkSyncView_emptyModel()
|
|
||||||
+{
|
|
||||||
+ // When a tableview has a syncview with an empty model then it should still be
|
|
||||||
+ // showing the tableview without depending on the syncview. This is particularly
|
|
||||||
+ // important for headerviews for example
|
|
||||||
+ LOAD_TABLEVIEW("syncviewsimple.qml");
|
|
||||||
+ GET_QML_TABLEVIEW(tableViewH);
|
|
||||||
+ GET_QML_TABLEVIEW(tableViewV);
|
|
||||||
+ GET_QML_TABLEVIEW(tableViewHV);
|
|
||||||
+ QQuickTableView *views[] = {tableViewH, tableViewV, tableViewHV};
|
|
||||||
+
|
|
||||||
+ auto model = TestModelAsVariant(100, 100);
|
|
||||||
+
|
|
||||||
+ for (auto view : views)
|
|
||||||
+ view->setModel(model);
|
|
||||||
+
|
|
||||||
+ WAIT_UNTIL_POLISHED_ARG(tableViewHV);
|
|
||||||
+
|
|
||||||
+ // Check that geometry properties are mirrored
|
|
||||||
+ QCOMPARE(tableViewH->columnSpacing(), tableView->columnSpacing());
|
|
||||||
+ QCOMPARE(tableViewH->rowSpacing(), 0);
|
|
||||||
+ QCOMPARE(tableViewH->contentWidth(), tableView->contentWidth());
|
|
||||||
+ QVERIFY(tableViewH->contentHeight() > 0);
|
|
||||||
+ QCOMPARE(tableViewV->columnSpacing(), 0);
|
|
||||||
+ QCOMPARE(tableViewV->rowSpacing(), tableView->rowSpacing());
|
|
||||||
+ QCOMPARE(tableViewV->contentHeight(), tableView->contentHeight());
|
|
||||||
+ QVERIFY(tableViewV->contentWidth() > 0);
|
|
||||||
+
|
|
||||||
+ QCOMPARE(tableViewH->contentX(), tableView->contentX());
|
|
||||||
+ QCOMPARE(tableViewH->contentY(), 0);
|
|
||||||
+ QCOMPARE(tableViewV->contentX(), 0);
|
|
||||||
+ QCOMPARE(tableViewV->contentY(), tableView->contentY());
|
|
||||||
+ QCOMPARE(tableViewHV->contentX(), tableView->contentX());
|
|
||||||
+ QCOMPARE(tableViewHV->contentY(), tableView->contentY());
|
|
||||||
+
|
|
||||||
+ QCOMPARE(tableViewHPrivate->loadedTableOuterRect.left(), tableViewPrivate->loadedTableOuterRect.left());
|
|
||||||
+ QCOMPARE(tableViewHPrivate->loadedTableOuterRect.top(), 0);
|
|
||||||
+
|
|
||||||
+ QCOMPARE(tableViewVPrivate->loadedTableOuterRect.top(), tableViewPrivate->loadedTableOuterRect.top());
|
|
||||||
+ QCOMPARE(tableViewVPrivate->loadedTableOuterRect.left(), 0);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
void tst_QQuickTableView::checkThatFetchMoreIsCalledWhenScrolledToTheEndOfTable()
|
|
||||||
{
|
|
||||||
LOAD_TABLEVIEW("plaintableview.qml");
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -0,0 +1,51 @@
|
|||||||
|
From c8a4894655eef230702e6cc889f3d40fbb5a9426 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Ulf Hermann <ulf.hermann@qt.io>
|
||||||
|
Date: Thu, 3 Feb 2022 10:02:06 +0100
|
||||||
|
Subject: [PATCH 19/20] V4: Do not call dtor of an object we continue to use
|
||||||
|
|
||||||
|
After destroyObject(), the QObjectWrapper is still alive. We might use
|
||||||
|
its heap object again. Furthermore, the Heap::QObjectWrapper dtor does
|
||||||
|
not actually do anything defined. What we want to do here is clear the
|
||||||
|
QObject pointer because we've just gotten rid of the QObject. There is a
|
||||||
|
method for that: Heap::QObjectWrapper::destroy().
|
||||||
|
|
||||||
|
Finally, the internalClass must never ever be nullptr. Assert on that
|
||||||
|
rather than checking it.
|
||||||
|
|
||||||
|
Pick-to: 5.15 6.2 6.3
|
||||||
|
Task-number: QTBUG-100431
|
||||||
|
Change-Id: I794a295c182b2ed4ba80673f58d6143c861b7391
|
||||||
|
Reviewed-by: Andrei Golubev <andrei.golubev@qt.io>
|
||||||
|
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
|
||||||
|
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
||||||
|
(cherry picked from commit 6c197319f34b8098d034f1543eb5feb9d7be54c3)
|
||||||
|
---
|
||||||
|
src/qml/jsruntime/qv4qobjectwrapper.cpp | 5 ++---
|
||||||
|
1 file changed, 2 insertions(+), 3 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/qml/jsruntime/qv4qobjectwrapper.cpp b/src/qml/jsruntime/qv4qobjectwrapper.cpp
|
||||||
|
index 9899c9274e..272b85069f 100644
|
||||||
|
--- a/src/qml/jsruntime/qv4qobjectwrapper.cpp
|
||||||
|
+++ b/src/qml/jsruntime/qv4qobjectwrapper.cpp
|
||||||
|
@@ -1160,8 +1160,7 @@ void Heap::QObjectWrapper::markObjects(Heap::Base *that, QV4::MarkStack *markSta
|
||||||
|
void QObjectWrapper::destroyObject(bool lastCall)
|
||||||
|
{
|
||||||
|
Heap::QObjectWrapper *h = d();
|
||||||
|
- if (!h->internalClass)
|
||||||
|
- return; // destroyObject already got called
|
||||||
|
+ Q_ASSERT(h->internalClass);
|
||||||
|
|
||||||
|
if (h->object()) {
|
||||||
|
QQmlData *ddata = QQmlData::get(h->object(), false);
|
||||||
|
@@ -1191,7 +1190,7 @@ void QObjectWrapper::destroyObject(bool lastCall)
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
- h->~Data();
|
||||||
|
+ h->destroy();
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,58 +0,0 @@
|
|||||||
From 5e0ba6b797ca7843609fc19d8c4c96f6f26aacd2 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Miikka Heikkinen <miikka.heikkinen@qt.io>
|
|
||||||
Date: Tue, 15 Dec 2020 12:43:40 +0200
|
|
||||||
Subject: [PATCH 20/28] DesignerSupport: Don't skip already inspected objects
|
|
||||||
|
|
||||||
Already inspected objects should not be skipped when determining their
|
|
||||||
properties, as recursive call will always have different base name for
|
|
||||||
the properties.
|
|
||||||
|
|
||||||
Internally we don't need inspectedObjects list at all anymore, but
|
|
||||||
it's kept to avoid changing API and in case the caller is interested
|
|
||||||
in inspected objects.
|
|
||||||
|
|
||||||
Fixes: QDS-3301
|
|
||||||
Change-Id: I76198b96d420e2a5ae6b13cfee65df4bce22d8f5
|
|
||||||
Pick-to: dev
|
|
||||||
Reviewed-by: Mahmoud Badri <mahmoud.badri@qt.io>
|
|
||||||
Reviewed-by: Thomas Hartmann <thomas.hartmann@qt.io>
|
|
||||||
---
|
|
||||||
.../designer/qquickdesignersupportproperties.cpp | 15 ++++-----------
|
|
||||||
1 file changed, 4 insertions(+), 11 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/quick/designer/qquickdesignersupportproperties.cpp b/src/quick/designer/qquickdesignersupportproperties.cpp
|
|
||||||
index 335795acf1..fb6a5fb324 100644
|
|
||||||
--- a/src/quick/designer/qquickdesignersupportproperties.cpp
|
|
||||||
+++ b/src/quick/designer/qquickdesignersupportproperties.cpp
|
|
||||||
@@ -137,11 +137,8 @@ QQuickDesignerSupport::PropertyNameList QQuickDesignerSupportProperties::propert
|
|
||||||
if (inspectedObjects == nullptr)
|
|
||||||
inspectedObjects = &localObjectList;
|
|
||||||
|
|
||||||
-
|
|
||||||
- if (inspectedObjects->contains(object))
|
|
||||||
- return propertyNameList;
|
|
||||||
-
|
|
||||||
- inspectedObjects->append(object);
|
|
||||||
+ if (!inspectedObjects->contains(object))
|
|
||||||
+ inspectedObjects->append(object);
|
|
||||||
|
|
||||||
const QMetaObject *metaObject = object->metaObject();
|
|
||||||
for (int index = 0; index < metaObject->propertyCount(); ++index) {
|
|
||||||
@@ -194,12 +191,8 @@ QQuickDesignerSupport::PropertyNameList QQuickDesignerSupportProperties::allProp
|
|
||||||
if (inspectedObjects == nullptr)
|
|
||||||
inspectedObjects = &localObjectList;
|
|
||||||
|
|
||||||
-
|
|
||||||
- if (inspectedObjects->contains(object))
|
|
||||||
- return propertyNameList;
|
|
||||||
-
|
|
||||||
- inspectedObjects->append(object);
|
|
||||||
-
|
|
||||||
+ if (!inspectedObjects->contains(object))
|
|
||||||
+ inspectedObjects->append(object);
|
|
||||||
|
|
||||||
const QMetaObject *metaObject = object->metaObject();
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -0,0 +1,84 @@
|
|||||||
|
From 02105099301450c890e1caba977ef44efdc43da7 Mon Sep 17 00:00:00 2001
|
||||||
|
From: Vlad Zahorodnii <vlad.zahorodnii@kde.org>
|
||||||
|
Date: Sat, 29 Jan 2022 21:59:33 +0200
|
||||||
|
Subject: [PATCH 20/20] Make sure QQuickWidget and its offscreen window's
|
||||||
|
screens are always in sync
|
||||||
|
|
||||||
|
By default, the offscreen window is placed on the primary screen.
|
||||||
|
However, if the parent widget argument is passed to the QQuickWidget's
|
||||||
|
constructor, then QQuickWidget's and the offscreen window's screens can
|
||||||
|
be different and that can create rendering issues, e.g. blurry text if
|
||||||
|
the primary screen and QQuickWidget's screen have different scale
|
||||||
|
factors.
|
||||||
|
|
||||||
|
Change-Id: I10c62b5635664f943b11828773f14017f198a770
|
||||||
|
Reviewed-by: David Edmundson <davidedmundson@kde.org>
|
||||||
|
Reviewed-by: Laszlo Agocs <laszlo.agocs@qt.io>
|
||||||
|
(cherry picked from commit a2a2734bffa1459639b31fb3f4f83873ba44ab5c)
|
||||||
|
---
|
||||||
|
src/quickwidgets/qquickwidget.cpp | 26 +++++++++++---------------
|
||||||
|
1 file changed, 11 insertions(+), 15 deletions(-)
|
||||||
|
|
||||||
|
diff --git a/src/quickwidgets/qquickwidget.cpp b/src/quickwidgets/qquickwidget.cpp
|
||||||
|
index 39780f8de3..223d91f579 100644
|
||||||
|
--- a/src/quickwidgets/qquickwidget.cpp
|
||||||
|
+++ b/src/quickwidgets/qquickwidget.cpp
|
||||||
|
@@ -106,6 +106,7 @@ void QQuickWidgetPrivate::init(QQmlEngine* e)
|
||||||
|
|
||||||
|
renderControl = new QQuickWidgetRenderControl(q);
|
||||||
|
offscreenWindow = new QQuickWindow(*new QQuickOffcreenWindowPrivate(),renderControl);
|
||||||
|
+ offscreenWindow->setScreen(q->screen());
|
||||||
|
offscreenWindow->setTitle(QString::fromLatin1("Offscreen"));
|
||||||
|
offscreenWindow->setObjectName(QString::fromLatin1("QQuickOffScreenWindow"));
|
||||||
|
// Do not call create() on offscreenWindow.
|
||||||
|
@@ -901,9 +902,7 @@ void QQuickWidgetPrivate::createContext()
|
||||||
|
|
||||||
|
context = new QOpenGLContext;
|
||||||
|
context->setFormat(offscreenWindow->requestedFormat());
|
||||||
|
- const QWindow *win = q->window()->windowHandle();
|
||||||
|
- if (win && win->screen())
|
||||||
|
- context->setScreen(win->screen());
|
||||||
|
+ context->setScreen(q->screen());
|
||||||
|
QOpenGLContext *shareContext = qt_gl_global_share_context();
|
||||||
|
if (!shareContext)
|
||||||
|
shareContext = QWidgetPrivate::get(q->window())->shareContext();
|
||||||
|
@@ -1520,19 +1519,16 @@ bool QQuickWidget::event(QEvent *e)
|
||||||
|
d->handleWindowChange();
|
||||||
|
break;
|
||||||
|
|
||||||
|
- case QEvent::ScreenChangeInternal:
|
||||||
|
- if (QWindow *window = this->window()->windowHandle()) {
|
||||||
|
- QScreen *newScreen = window->screen();
|
||||||
|
-
|
||||||
|
- if (d->offscreenWindow)
|
||||||
|
- d->offscreenWindow->setScreen(newScreen);
|
||||||
|
- if (d->offscreenSurface)
|
||||||
|
- d->offscreenSurface->setScreen(newScreen);
|
||||||
|
+ case QEvent::ScreenChangeInternal: {
|
||||||
|
+ QScreen *newScreen = screen();
|
||||||
|
+ if (d->offscreenWindow)
|
||||||
|
+ d->offscreenWindow->setScreen(newScreen);
|
||||||
|
+ if (d->offscreenSurface)
|
||||||
|
+ d->offscreenSurface->setScreen(newScreen);
|
||||||
|
#if QT_CONFIG(opengl)
|
||||||
|
- if (d->context)
|
||||||
|
- d->context->setScreen(newScreen);
|
||||||
|
+ if (d->context)
|
||||||
|
+ d->context->setScreen(newScreen);
|
||||||
|
#endif
|
||||||
|
- }
|
||||||
|
|
||||||
|
if (d->useSoftwareRenderer
|
||||||
|
#if QT_CONFIG(opengl)
|
||||||
|
@@ -1545,7 +1541,7 @@ bool QQuickWidget::event(QEvent *e)
|
||||||
|
d->render(true);
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
-
|
||||||
|
+ }
|
||||||
|
case QEvent::Show:
|
||||||
|
case QEvent::Move:
|
||||||
|
d->updatePosition();
|
||||||
|
--
|
||||||
|
2.35.1
|
||||||
|
|
@ -1,114 +0,0 @@
|
|||||||
From 5a7aa7881fa2c7abffb3d34a6b642fe4efcadbf4 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Date: Thu, 17 Dec 2020 11:22:34 +0100
|
|
||||||
Subject: [PATCH 21/28] QML: Fix proxy iteration
|
|
||||||
|
|
||||||
If the target of a proxy was extensible, we did not set the
|
|
||||||
iteratorTarget to its correct value, and thus the ForInIteratorObject
|
|
||||||
would not be usable.
|
|
||||||
|
|
||||||
Fixes: QTBUG-86323
|
|
||||||
Change-Id: Id1924ac4087bab38c006b8eba92b619b79d36b7a
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit dd740d6b3469448dc1fd31c1742781e923e9f274)
|
|
||||||
---
|
|
||||||
src/qml/jsruntime/qv4proxy.cpp | 8 +++--
|
|
||||||
.../qqmlecmascript/data/proxyIteration.qml | 29 +++++++++++++++++++
|
|
||||||
.../qml/qqmlecmascript/tst_qqmlecmascript.cpp | 10 +++++++
|
|
||||||
3 files changed, 45 insertions(+), 2 deletions(-)
|
|
||||||
create mode 100644 tests/auto/qml/qqmlecmascript/data/proxyIteration.qml
|
|
||||||
|
|
||||||
diff --git a/src/qml/jsruntime/qv4proxy.cpp b/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
index 1505eae426..8bfc9fc3ba 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
+++ b/src/qml/jsruntime/qv4proxy.cpp
|
|
||||||
@@ -624,8 +624,10 @@ OwnPropertyKeyIterator *ProxyObject::virtualOwnPropertyKeys(const Object *m, Val
|
|
||||||
else
|
|
||||||
targetNonConfigurableKeys->push_back(keyAsValue);
|
|
||||||
}
|
|
||||||
- if (target->isExtensible() && targetNonConfigurableKeys->getLength() == 0)
|
|
||||||
+ if (target->isExtensible() && targetNonConfigurableKeys->getLength() == 0) {
|
|
||||||
+ *iteratorTarget = *m;
|
|
||||||
return new ProxyObjectOwnPropertyKeyIterator(trapKeys);
|
|
||||||
+ }
|
|
||||||
|
|
||||||
ScopedArrayObject uncheckedResultKeys(scope, scope.engine->newArrayObject());
|
|
||||||
uncheckedResultKeys->copyArrayData(trapKeys);
|
|
||||||
@@ -639,8 +641,10 @@ OwnPropertyKeyIterator *ProxyObject::virtualOwnPropertyKeys(const Object *m, Val
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
- if (target->isExtensible())
|
|
||||||
+ if (target->isExtensible()) {
|
|
||||||
+ *iteratorTarget = *m;
|
|
||||||
return new ProxyObjectOwnPropertyKeyIterator(trapKeys);
|
|
||||||
+ }
|
|
||||||
|
|
||||||
len = targetConfigurableKeys->getLength();
|
|
||||||
for (uint i = 0; i < len; ++i) {
|
|
||||||
diff --git a/tests/auto/qml/qqmlecmascript/data/proxyIteration.qml b/tests/auto/qml/qqmlecmascript/data/proxyIteration.qml
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..affba7d9f1
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/qml/qqmlecmascript/data/proxyIteration.qml
|
|
||||||
@@ -0,0 +1,29 @@
|
|
||||||
+import QtQml 2
|
|
||||||
+
|
|
||||||
+QtObject {
|
|
||||||
+ id: root
|
|
||||||
+ property int sum
|
|
||||||
+ Component.onCompleted: {
|
|
||||||
+ const target = { prop1: 1, prop2: 2, prop3: 3 };
|
|
||||||
+ const handler = {
|
|
||||||
+ get: function(target, key) {
|
|
||||||
+ return target[key]+1;
|
|
||||||
+ },
|
|
||||||
+ ownKeys: function() {
|
|
||||||
+ return ["prop1", "prop3"];
|
|
||||||
+ },
|
|
||||||
+ getOwnPropertyDescriptor: function(target, key) {
|
|
||||||
+ return {
|
|
||||||
+ value: this.get(target, key),
|
|
||||||
+ enumerable: true,
|
|
||||||
+ configurable: true
|
|
||||||
+ };
|
|
||||||
+ }
|
|
||||||
+ };
|
|
||||||
+ const proxy = new Proxy(target, handler);
|
|
||||||
+ for (var prop in proxy) {
|
|
||||||
+ root.sum += proxy[prop] // prop2 gets skipped, the values of 1 and 3 get incremented
|
|
||||||
+ }
|
|
||||||
+ // so root.sum should be 6 now
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
diff --git a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
index 3a9d1bfb4c..9198d3bebf 100644
|
|
||||||
--- a/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
+++ b/tests/auto/qml/qqmlecmascript/tst_qqmlecmascript.cpp
|
|
||||||
@@ -382,6 +382,7 @@ private slots:
|
|
||||||
void semicolonAfterProperty();
|
|
||||||
void hugeStack();
|
|
||||||
void variantConversionMethod();
|
|
||||||
+ void proxyIteration();
|
|
||||||
void proxyHandlerTraps();
|
|
||||||
void gcCrashRegressionTest();
|
|
||||||
|
|
||||||
@@ -9306,6 +9307,15 @@ void tst_qqmlecmascript::variantConversionMethod()
|
|
||||||
QCOMPARE(obj.funcCalled, QLatin1String("QModelIndex"));
|
|
||||||
}
|
|
||||||
|
|
||||||
+void tst_qqmlecmascript::proxyIteration()
|
|
||||||
+{
|
|
||||||
+ QQmlEngine engine;
|
|
||||||
+ QQmlComponent component(&engine, testFileUrl("proxyIteration.qml"));
|
|
||||||
+ QScopedPointer<QObject> root(component.create());
|
|
||||||
+ QVERIFY2(root != nullptr, qPrintable(component.errorString()));
|
|
||||||
+ QCOMPARE(root->property("sum").toInt(), 6);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
void tst_qqmlecmascript::proxyHandlerTraps()
|
|
||||||
{
|
|
||||||
const QString expression = QStringLiteral(R"SNIPPET(
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,146 +0,0 @@
|
|||||||
From bb0ce1ffd48aa69da03dc43bd314351519ebf0d7 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Date: Tue, 8 Dec 2020 14:12:47 +0100
|
|
||||||
Subject: [PATCH 22/28] Fix IC properties in same file
|
|
||||||
|
|
||||||
Also fixes typename and metatype registration for inline components.
|
|
||||||
|
|
||||||
Done-with: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Fixes: QTBUG-89173
|
|
||||||
Change-Id: I97d65d5539b577a8828d5711e5f2e79c8568b441
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit c2ca14ce22551ea72544b6e2b3a19823b6dc3050)
|
|
||||||
---
|
|
||||||
src/qml/qml/qqmlpropertycachecreator.cpp | 9 +++++++
|
|
||||||
src/qml/qml/qqmlpropertycachecreator_p.h | 2 ++
|
|
||||||
src/qml/qml/qqmlpropertyvalidator.cpp | 25 +++++++++++++++++++
|
|
||||||
src/qml/qml/qqmltypedata.cpp | 4 +--
|
|
||||||
.../data/inlineComponentsSameFile.qml | 11 ++++++++
|
|
||||||
.../qml/qqmllanguage/tst_qqmllanguage.cpp | 1 +
|
|
||||||
6 files changed, 49 insertions(+), 3 deletions(-)
|
|
||||||
create mode 100644 tests/auto/qml/qqmllanguage/data/inlineComponentsSameFile.qml
|
|
||||||
|
|
||||||
diff --git a/src/qml/qml/qqmlpropertycachecreator.cpp b/src/qml/qml/qqmlpropertycachecreator.cpp
|
|
||||||
index 36581bda4e..88d80d88ab 100644
|
|
||||||
--- a/src/qml/qml/qqmlpropertycachecreator.cpp
|
|
||||||
+++ b/src/qml/qml/qqmlpropertycachecreator.cpp
|
|
||||||
@@ -90,6 +90,15 @@ QByteArray QQmlPropertyCacheCreatorBase::createClassNameTypeByUrl(const QUrl &ur
|
|
||||||
QByteArray::number(classIndexCounter.fetchAndAddRelaxed(1));
|
|
||||||
}
|
|
||||||
|
|
||||||
+QByteArray QQmlPropertyCacheCreatorBase::createClassNameForInlineComponent(const QUrl &baseUrl, int icId)
|
|
||||||
+{
|
|
||||||
+ QByteArray baseName = createClassNameTypeByUrl(baseUrl);
|
|
||||||
+ if (baseName.isEmpty())
|
|
||||||
+ baseName = QByteArray("ANON_QML_IC_") + QByteArray::number(classIndexCounter.fetchAndAddRelaxed(1));
|
|
||||||
+ baseName += "_" + QByteArray::number(icId);
|
|
||||||
+ return baseName;
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
QQmlBindingInstantiationContext::QQmlBindingInstantiationContext(int referencingObjectIndex, const QV4::CompiledData::Binding *instantiatingBinding,
|
|
||||||
const QString &instantiatingPropertyName, QQmlPropertyCache *referencingObjectPropertyCache)
|
|
||||||
: referencingObjectIndex(referencingObjectIndex)
|
|
||||||
diff --git a/src/qml/qml/qqmlpropertycachecreator_p.h b/src/qml/qml/qqmlpropertycachecreator_p.h
|
|
||||||
index 6b02d6fb98..77e3763a49 100644
|
|
||||||
--- a/src/qml/qml/qqmlpropertycachecreator_p.h
|
|
||||||
+++ b/src/qml/qml/qqmlpropertycachecreator_p.h
|
|
||||||
@@ -104,6 +104,8 @@ public:
|
|
||||||
static int metaTypeForPropertyType(QV4::CompiledData::BuiltinType type);
|
|
||||||
|
|
||||||
static QByteArray createClassNameTypeByUrl(const QUrl &url);
|
|
||||||
+
|
|
||||||
+ static QByteArray createClassNameForInlineComponent(const QUrl &baseUrl, int icId);
|
|
||||||
};
|
|
||||||
|
|
||||||
template <typename ObjectContainer>
|
|
||||||
diff --git a/src/qml/qml/qqmlpropertyvalidator.cpp b/src/qml/qml/qqmlpropertyvalidator.cpp
|
|
||||||
index 3587609301..3a1f33113f 100644
|
|
||||||
--- a/src/qml/qml/qqmlpropertyvalidator.cpp
|
|
||||||
+++ b/src/qml/qml/qqmlpropertyvalidator.cpp
|
|
||||||
@@ -651,6 +651,19 @@ bool QQmlPropertyValidator::canCoerce(int to, QQmlPropertyCache *fromMo) const
|
|
||||||
{
|
|
||||||
QQmlPropertyCache *toMo = enginePrivate->rawPropertyCacheForType(to);
|
|
||||||
|
|
||||||
+ if (toMo == nullptr) {
|
|
||||||
+ // if we have an inline component from the current file,
|
|
||||||
+ // it is not properly registered at this point, as registration
|
|
||||||
+ // only occurs after the whole file has been validated
|
|
||||||
+ // Therefore we need to check the ICs here
|
|
||||||
+ for (const auto& icDatum : compilationUnit->inlineComponentData) {
|
|
||||||
+ if (icDatum.typeIds.id == to) {
|
|
||||||
+ toMo = compilationUnit->propertyCaches.at(icDatum.objectIndex);
|
|
||||||
+ break;
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
while (fromMo) {
|
|
||||||
if (fromMo == toMo)
|
|
||||||
return true;
|
|
||||||
@@ -746,6 +759,18 @@ QQmlError QQmlPropertyValidator::validateObjectBinding(QQmlPropertyData *propert
|
|
||||||
// effect the properties on the type, but don't effect assignability
|
|
||||||
// Using -1 for the minor version ensures that we get the raw metaObject.
|
|
||||||
QQmlPropertyCache *propertyMetaObject = enginePrivate->rawPropertyCacheForType(propType, -1);
|
|
||||||
+ if (!propertyMetaObject) {
|
|
||||||
+ // if we have an inline component from the current file,
|
|
||||||
+ // it is not properly registered at this point, as registration
|
|
||||||
+ // only occurs after the whole file has been validated
|
|
||||||
+ // Therefore we need to check the ICs here
|
|
||||||
+ for (const auto& icDatum: compilationUnit->inlineComponentData) {
|
|
||||||
+ if (icDatum.typeIds.id == property->propType()) {
|
|
||||||
+ propertyMetaObject = compilationUnit->propertyCaches.at(icDatum.objectIndex);
|
|
||||||
+ break;
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
|
|
||||||
if (propertyMetaObject) {
|
|
||||||
// Will be true if the assigned type inherits propertyMetaObject
|
|
||||||
diff --git a/src/qml/qml/qqmltypedata.cpp b/src/qml/qml/qqmltypedata.cpp
|
|
||||||
index fc1d0cfbcf..92a90ea677 100644
|
|
||||||
--- a/src/qml/qml/qqmltypedata.cpp
|
|
||||||
+++ b/src/qml/qml/qqmltypedata.cpp
|
|
||||||
@@ -283,9 +283,7 @@ void setupICs(const ObjectContainer &container, QHash<int, InlineComponentData>
|
|
||||||
for (int i = 0; i != container->objectCount(); ++i) {
|
|
||||||
auto root = container->objectAt(i);
|
|
||||||
for (auto it = root->inlineComponentsBegin(); it != root->inlineComponentsEnd(); ++it) {
|
|
||||||
- auto url = finalUrl;
|
|
||||||
- url.setFragment(QString::number(it->objectIndex));
|
|
||||||
- const QByteArray &className = QQmlPropertyCacheCreatorBase::createClassNameTypeByUrl(url);
|
|
||||||
+ const QByteArray &className = QQmlPropertyCacheCreatorBase::createClassNameForInlineComponent(finalUrl, it->objectIndex);
|
|
||||||
InlineComponentData icDatum(QQmlMetaType::registerInternalCompositeType(className), int(it->objectIndex), int(it->nameIndex), 0, 0, 0);
|
|
||||||
icData->insert(it->objectIndex, icDatum);
|
|
||||||
}
|
|
||||||
diff --git a/tests/auto/qml/qqmllanguage/data/inlineComponentsSameFile.qml b/tests/auto/qml/qqmllanguage/data/inlineComponentsSameFile.qml
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..87cac10200
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/qml/qqmllanguage/data/inlineComponentsSameFile.qml
|
|
||||||
@@ -0,0 +1,11 @@
|
|
||||||
+import QtQml 2.15
|
|
||||||
+
|
|
||||||
+QtObject {
|
|
||||||
+ component IC : QtObject {
|
|
||||||
+ property string name
|
|
||||||
+ property int age
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ property IC other: IC { name: "Toby"; age: 30 }
|
|
||||||
+ property list<IC> listProp: [IC { name: "Alfred Ill"; age: 65 }, IC { name: "Claire Zachanassian"; age: 62}]
|
|
||||||
+}
|
|
||||||
diff --git a/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp b/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
|
|
||||||
index 8adcbc1837..e247a139ec 100644
|
|
||||||
--- a/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
|
|
||||||
+++ b/tests/auto/qml/qqmllanguage/tst_qqmllanguage.cpp
|
|
||||||
@@ -5604,6 +5604,7 @@ void tst_qqmllanguage::inlineComponent_data()
|
|
||||||
QTest::newRow("Alias resolves correctly") << testFileUrl("inlineComponentWithAlias.qml") << QColorConstants::Svg::lime << 42 << true;
|
|
||||||
|
|
||||||
QTest::newRow("Two inline components in same do not crash (QTBUG-86989)") << testFileUrl("twoInlineComponents.qml") << QColor() << 0 << false;
|
|
||||||
+ QTest::newRow("Inline components used in same file (QTBUG-89173)") << testFileUrl("inlineComponentsSameFile.qml") << QColor() << 0 << false;
|
|
||||||
}
|
|
||||||
|
|
||||||
void tst_qqmllanguage::inlineComponentReferenceCycle_data()
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,201 +0,0 @@
|
|||||||
From 35614462443c100b6753b335b58a134fed4b5c35 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
Date: Wed, 16 Dec 2020 16:45:36 +0100
|
|
||||||
Subject: [PATCH 23/28] JIT: When making memory writable, include the exception
|
|
||||||
handler
|
|
||||||
|
|
||||||
makeWritable() rounds the memory down to the next page boundary. Usually
|
|
||||||
we include the exception handler this way, unless the offset from the
|
|
||||||
page boundary is less than the exception handler size. Make it explicit
|
|
||||||
that we do want the exception handler to be writable, too.
|
|
||||||
|
|
||||||
Fixes: QTBUG-89513
|
|
||||||
Change-Id: I2fb8fb0e1dcc3450b036924463dc1b40d2020c46
|
|
||||||
Reviewed-by: Fabian Kosmale <fabian.kosmale@qt.io>
|
|
||||||
(cherry picked from commit 86a595b126bc6794380dc00af80ec4802f7d058c)
|
|
||||||
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
|
|
||||||
---
|
|
||||||
src/3rdparty/masm/assembler/AssemblerBuffer.h | 4 ++--
|
|
||||||
src/3rdparty/masm/assembler/LinkBuffer.h | 9 +++++----
|
|
||||||
.../masm/assembler/MacroAssemblerCodeRef.h | 6 +++---
|
|
||||||
src/3rdparty/masm/stubs/ExecutableAllocator.h | 11 ++++++++---
|
|
||||||
src/qml/jsruntime/qv4executableallocator.cpp | 14 ++++++++++++--
|
|
||||||
src/qml/jsruntime/qv4executableallocator_p.h | 10 ++++++++--
|
|
||||||
src/qml/jsruntime/qv4functiontable_win64.cpp | 4 ++--
|
|
||||||
7 files changed, 40 insertions(+), 18 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/3rdparty/masm/assembler/AssemblerBuffer.h b/src/3rdparty/masm/assembler/AssemblerBuffer.h
|
|
||||||
index 45874235b6..2292a4c244 100644
|
|
||||||
--- a/src/3rdparty/masm/assembler/AssemblerBuffer.h
|
|
||||||
+++ b/src/3rdparty/masm/assembler/AssemblerBuffer.h
|
|
||||||
@@ -140,9 +140,9 @@ namespace JSC {
|
|
||||||
if (!result)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
- ExecutableAllocator::makeWritable(result->start(), result->sizeInBytes());
|
|
||||||
+ ExecutableAllocator::makeWritable(result->memoryStart(), result->memorySize());
|
|
||||||
|
|
||||||
- memcpy(result->start(), m_buffer, m_index);
|
|
||||||
+ memcpy(result->codeStart(), m_buffer, m_index);
|
|
||||||
|
|
||||||
return result.release();
|
|
||||||
}
|
|
||||||
diff --git a/src/3rdparty/masm/assembler/LinkBuffer.h b/src/3rdparty/masm/assembler/LinkBuffer.h
|
|
||||||
index ba57564a1d..fa669deaf9 100644
|
|
||||||
--- a/src/3rdparty/masm/assembler/LinkBuffer.h
|
|
||||||
+++ b/src/3rdparty/masm/assembler/LinkBuffer.h
|
|
||||||
@@ -333,7 +333,7 @@ inline void LinkBufferBase<MacroAssembler, ExecutableOffsetCalculator>::linkCode
|
|
||||||
m_executableMemory = m_assembler->m_assembler.executableCopy(*m_globalData, ownerUID, effort);
|
|
||||||
if (!m_executableMemory)
|
|
||||||
return;
|
|
||||||
- m_code = m_executableMemory->start();
|
|
||||||
+ m_code = m_executableMemory->codeStart();
|
|
||||||
m_size = m_assembler->m_assembler.codeSize();
|
|
||||||
ASSERT(m_code);
|
|
||||||
}
|
|
||||||
@@ -355,7 +355,8 @@ void LinkBufferBase<MacroAssembler, ExecutableOffsetCalculator>::performFinaliza
|
|
||||||
template <typename MacroAssembler, template <typename T> class ExecutableOffsetCalculator>
|
|
||||||
inline void LinkBufferBase<MacroAssembler, ExecutableOffsetCalculator>::makeExecutable()
|
|
||||||
{
|
|
||||||
- ExecutableAllocator::makeExecutable(code(), static_cast<int>(m_size));
|
|
||||||
+ ExecutableAllocator::makeExecutable(m_executableMemory->memoryStart(),
|
|
||||||
+ m_executableMemory->memorySize());
|
|
||||||
}
|
|
||||||
|
|
||||||
template <typename MacroAssembler>
|
|
||||||
@@ -442,9 +443,9 @@ inline void BranchCompactingLinkBuffer<MacroAssembler>::linkCode(void* ownerUID,
|
|
||||||
m_executableMemory = m_globalData->executableAllocator.allocate(*m_globalData, m_initialSize, ownerUID, effort);
|
|
||||||
if (!m_executableMemory)
|
|
||||||
return;
|
|
||||||
- m_code = (uint8_t*)m_executableMemory->start();
|
|
||||||
+ m_code = (uint8_t*)m_executableMemory->codeStart();
|
|
||||||
ASSERT(m_code);
|
|
||||||
- ExecutableAllocator::makeWritable(m_code, m_initialSize);
|
|
||||||
+ ExecutableAllocator::makeWritable(m_executableMemory->memoryStart(), m_executableMemory->memorySize());
|
|
||||||
uint8_t* inData = (uint8_t*)m_assembler->unlinkedCode();
|
|
||||||
uint8_t* outData = reinterpret_cast<uint8_t*>(m_code);
|
|
||||||
int readPtr = 0;
|
|
||||||
diff --git a/src/3rdparty/masm/assembler/MacroAssemblerCodeRef.h b/src/3rdparty/masm/assembler/MacroAssemblerCodeRef.h
|
|
||||||
index a7e78ad78f..cde9751108 100644
|
|
||||||
--- a/src/3rdparty/masm/assembler/MacroAssemblerCodeRef.h
|
|
||||||
+++ b/src/3rdparty/masm/assembler/MacroAssemblerCodeRef.h
|
|
||||||
@@ -357,11 +357,11 @@ public:
|
|
||||||
}
|
|
||||||
|
|
||||||
MacroAssemblerCodeRef(PassRefPtr<ExecutableMemoryHandle> executableMemory)
|
|
||||||
- : m_codePtr(executableMemory->start())
|
|
||||||
+ : m_codePtr(executableMemory->codeStart())
|
|
||||||
, m_executableMemory(executableMemory)
|
|
||||||
{
|
|
||||||
ASSERT(m_executableMemory->isManaged());
|
|
||||||
- ASSERT(m_executableMemory->start());
|
|
||||||
+ ASSERT(m_executableMemory->codeStart());
|
|
||||||
ASSERT(m_codePtr);
|
|
||||||
}
|
|
||||||
|
|
||||||
@@ -395,7 +395,7 @@ public:
|
|
||||||
{
|
|
||||||
if (!m_executableMemory)
|
|
||||||
return 0;
|
|
||||||
- return m_executableMemory->sizeInBytes();
|
|
||||||
+ return m_executableMemory->codeSize();
|
|
||||||
}
|
|
||||||
|
|
||||||
bool tryToDisassemble(const char* prefix) const
|
|
||||||
diff --git a/src/3rdparty/masm/stubs/ExecutableAllocator.h b/src/3rdparty/masm/stubs/ExecutableAllocator.h
|
|
||||||
index a439c53827..f984704023 100644
|
|
||||||
--- a/src/3rdparty/masm/stubs/ExecutableAllocator.h
|
|
||||||
+++ b/src/3rdparty/masm/stubs/ExecutableAllocator.h
|
|
||||||
@@ -82,9 +82,14 @@ struct ExecutableMemoryHandle : public RefCounted<ExecutableMemoryHandle> {
|
|
||||||
|
|
||||||
inline bool isManaged() const { return true; }
|
|
||||||
|
|
||||||
- void *exceptionHandler() { return m_allocation->exceptionHandler(); }
|
|
||||||
- void *start() { return m_allocation->start(); }
|
|
||||||
- size_t sizeInBytes() { return m_size; }
|
|
||||||
+ void *memoryStart() { return m_allocation->memoryStart(); }
|
|
||||||
+ size_t memorySize() { return m_allocation->memorySize(); }
|
|
||||||
+
|
|
||||||
+ void *exceptionHandlerStart() { return m_allocation->exceptionHandlerStart(); }
|
|
||||||
+ size_t exceptionHandlerSize() { return m_allocation->exceptionHandlerSize(); }
|
|
||||||
+
|
|
||||||
+ void *codeStart() { return m_allocation->codeStart(); }
|
|
||||||
+ size_t codeSize() { return m_size; }
|
|
||||||
|
|
||||||
QV4::ExecutableAllocator::ChunkOfPages *chunk() const
|
|
||||||
{ return m_allocator->chunkForAllocation(m_allocation); }
|
|
||||||
diff --git a/src/qml/jsruntime/qv4executableallocator.cpp b/src/qml/jsruntime/qv4executableallocator.cpp
|
|
||||||
index 7ee6f39aa2..c06773d3c5 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4executableallocator.cpp
|
|
||||||
+++ b/src/qml/jsruntime/qv4executableallocator.cpp
|
|
||||||
@@ -45,12 +45,22 @@
|
|
||||||
|
|
||||||
using namespace QV4;
|
|
||||||
|
|
||||||
-void *ExecutableAllocator::Allocation::exceptionHandler() const
|
|
||||||
+void *ExecutableAllocator::Allocation::exceptionHandlerStart() const
|
|
||||||
{
|
|
||||||
return reinterpret_cast<void*>(addr);
|
|
||||||
}
|
|
||||||
|
|
||||||
-void *ExecutableAllocator::Allocation::start() const
|
|
||||||
+size_t ExecutableAllocator::Allocation::exceptionHandlerSize() const
|
|
||||||
+{
|
|
||||||
+ return QV4::exceptionHandlerSize();
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+void *ExecutableAllocator::Allocation::memoryStart() const
|
|
||||||
+{
|
|
||||||
+ return reinterpret_cast<void*>(addr);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+void *ExecutableAllocator::Allocation::codeStart() const
|
|
||||||
{
|
|
||||||
return reinterpret_cast<void*>(addr + exceptionHandlerSize());
|
|
||||||
}
|
|
||||||
diff --git a/src/qml/jsruntime/qv4executableallocator_p.h b/src/qml/jsruntime/qv4executableallocator_p.h
|
|
||||||
index f98f2c7d33..4735fb151f 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4executableallocator_p.h
|
|
||||||
+++ b/src/qml/jsruntime/qv4executableallocator_p.h
|
|
||||||
@@ -86,8 +86,14 @@ public:
|
|
||||||
, free(true)
|
|
||||||
{}
|
|
||||||
|
|
||||||
- void *exceptionHandler() const;
|
|
||||||
- void *start() const;
|
|
||||||
+ void *memoryStart() const;
|
|
||||||
+ size_t memorySize() const { return size; }
|
|
||||||
+
|
|
||||||
+ void *exceptionHandlerStart() const;
|
|
||||||
+ size_t exceptionHandlerSize() const;
|
|
||||||
+
|
|
||||||
+ void *codeStart() const;
|
|
||||||
+
|
|
||||||
void invalidate() { addr = 0; }
|
|
||||||
bool isValid() const { return addr != 0; }
|
|
||||||
void deallocate(ExecutableAllocator *allocator);
|
|
||||||
diff --git a/src/qml/jsruntime/qv4functiontable_win64.cpp b/src/qml/jsruntime/qv4functiontable_win64.cpp
|
|
||||||
index fc13dc2602..0cb98641cd 100644
|
|
||||||
--- a/src/qml/jsruntime/qv4functiontable_win64.cpp
|
|
||||||
+++ b/src/qml/jsruntime/qv4functiontable_win64.cpp
|
|
||||||
@@ -106,7 +106,7 @@ struct ExceptionHandlerRecord
|
|
||||||
void generateFunctionTable(Function *, JSC::MacroAssemblerCodeRef *codeRef)
|
|
||||||
{
|
|
||||||
ExceptionHandlerRecord *record = reinterpret_cast<ExceptionHandlerRecord *>(
|
|
||||||
- codeRef->executableMemory()->exceptionHandler());
|
|
||||||
+ codeRef->executableMemory()->exceptionHandlerStart());
|
|
||||||
|
|
||||||
record->info.Version = 1;
|
|
||||||
record->info.Flags = 0;
|
|
||||||
@@ -136,7 +136,7 @@ void generateFunctionTable(Function *, JSC::MacroAssemblerCodeRef *codeRef)
|
|
||||||
void destroyFunctionTable(Function *, JSC::MacroAssemblerCodeRef *codeRef)
|
|
||||||
{
|
|
||||||
ExceptionHandlerRecord *record = reinterpret_cast<ExceptionHandlerRecord *>(
|
|
||||||
- codeRef->executableMemory()->exceptionHandler());
|
|
||||||
+ codeRef->executableMemory()->exceptionHandlerStart());
|
|
||||||
if (!RtlDeleteFunctionTable(&record->handler)) {
|
|
||||||
const unsigned int errorCode = GetLastError();
|
|
||||||
qWarning() << "Failed to remove win64 unwind hook. Error code:" << errorCode;
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,103 +0,0 @@
|
|||||||
From e203a185cfab199a89a33b903096d6d0023a8a88 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Shawn Rutledge <shawn.rutledge@qt.io>
|
|
||||||
Date: Wed, 30 Sep 2020 13:51:59 +0200
|
|
||||||
Subject: [PATCH 24/28] doc: explain QQItem event delivery, handlers,
|
|
||||||
setAcceptTouchEvents()
|
|
||||||
|
|
||||||
We quietly recommended calling setAcceptTouchEvents() in the Qt 5.10
|
|
||||||
release notes in any Item subclass that wants to receive touch events,
|
|
||||||
and in the docs for setAcceptTouchEvents() itself; but the message about
|
|
||||||
the impending behavior change might not have been obvious enough.
|
|
||||||
In Qt 6 it becomes mandatory, so clearer docs will hopefully help to
|
|
||||||
stave off bogus bug reports.
|
|
||||||
|
|
||||||
We also never had a great overview of event handling from an Item's
|
|
||||||
perspective; now it's a little better.
|
|
||||||
|
|
||||||
Followup to ab91e7fa02a562d80fd0747f28a60e00c3b45a01 and
|
|
||||||
a97759a336c597327cb82eebc9f45c793aec32c9
|
|
||||||
|
|
||||||
[ChangeLog][QtQuick][QQuickItem] When subclassing QQuickItem, you
|
|
||||||
should call setAcceptTouchEvents(true) if you need the item to receive
|
|
||||||
touch events. It will be required in Qt 6.
|
|
||||||
|
|
||||||
Task-number: QTBUG-87018
|
|
||||||
Task-number: QTBUG-87082
|
|
||||||
Change-Id: I1c7a43979e3665778d61949c9d37c1d085ed594b
|
|
||||||
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
|
|
||||||
(cherry picked from commit 7c648280bb53c4276ba4ae2abf26d070fedde71a)
|
|
||||||
Reviewed-by: Shawn Rutledge <shawn.rutledge@qt.io>
|
|
||||||
---
|
|
||||||
src/quick/items/qquickitem.cpp | 34 +++++++++++++++++++++++++++++-----
|
|
||||||
1 file changed, 29 insertions(+), 5 deletions(-)
|
|
||||||
|
|
||||||
diff --git a/src/quick/items/qquickitem.cpp b/src/quick/items/qquickitem.cpp
|
|
||||||
index ddd67522b9..e02df00595 100644
|
|
||||||
--- a/src/quick/items/qquickitem.cpp
|
|
||||||
+++ b/src/quick/items/qquickitem.cpp
|
|
||||||
@@ -1,9 +1,9 @@
|
|
||||||
/****************************************************************************
|
|
||||||
**
|
|
||||||
-** Copyright (C) 2016 The Qt Company Ltd.
|
|
||||||
+** Copyright (C) 2021 The Qt Company Ltd.
|
|
||||||
** Contact: https://www.qt.io/licensing/
|
|
||||||
**
|
|
||||||
-** This file is part of the QtQuick module of the Qt Toolkit.
|
|
||||||
+** This file is part of the QtQuick module of the Qt Toolkit.fset
|
|
||||||
**
|
|
||||||
** $QT_BEGIN_LICENSE:LGPL$
|
|
||||||
** Commercial License Usage
|
|
||||||
@@ -1883,7 +1883,23 @@ void QQuickItemPrivate::updateSubFocusItem(QQuickItem *scope, bool focus)
|
|
||||||
\endqml
|
|
||||||
|
|
||||||
|
|
||||||
- \section2 Key Handling
|
|
||||||
+ \section2 Event Handling
|
|
||||||
+
|
|
||||||
+ All Item-based visual types can use \l {Qt Quick Input Handlers}{Input Handlers}
|
|
||||||
+ to handle incoming input events (subclasses of QInputEvent), such as mouse,
|
|
||||||
+ touch and key events. This is the preferred declarative way to handle events.
|
|
||||||
+
|
|
||||||
+ An alternative way to handle touch events is to subclass QQuickItem, call
|
|
||||||
+ setAcceptTouchEvents() in the constructor, and override touchEvent().
|
|
||||||
+ \l {QEvent::setAccepted()}{Accept} the entire event to stop delivery to
|
|
||||||
+ items underneath, and to exclusively grab all the event's touch points.
|
|
||||||
+
|
|
||||||
+ Likewise, a QQuickItem subclass can call setAcceptedMouseButtons()
|
|
||||||
+ to register to receive mouse button events, setAcceptHoverEvents()
|
|
||||||
+ to receive hover events (mouse movements while no button is pressed),
|
|
||||||
+ and override the virtual functions mousePressEvent(), mouseMoveEvent(), and
|
|
||||||
+ mouseReleaseEvent(). Those can also accept the event to prevent further
|
|
||||||
+ delivery and get an implicit grab at the same time.
|
|
||||||
|
|
||||||
Key handling is available to all Item-based visual types via the \l Keys
|
|
||||||
attached property. The \e Keys attached property provides basic signals
|
|
||||||
@@ -7301,7 +7317,9 @@ bool QQuickItem::isAncestorOf(const QQuickItem *child) const
|
|
||||||
If an item does not accept the mouse button for a particular mouse event,
|
|
||||||
the mouse event will not be delivered to the item and will be delivered
|
|
||||||
to the next item in the item hierarchy instead.
|
|
||||||
- */
|
|
||||||
+
|
|
||||||
+ \sa acceptTouchEvents()
|
|
||||||
+*/
|
|
||||||
Qt::MouseButtons QQuickItem::acceptedMouseButtons() const
|
|
||||||
{
|
|
||||||
Q_D(const QQuickItem);
|
|
||||||
@@ -7310,7 +7328,13 @@ Qt::MouseButtons QQuickItem::acceptedMouseButtons() const
|
|
||||||
|
|
||||||
/*!
|
|
||||||
Sets the mouse buttons accepted by this item to \a buttons.
|
|
||||||
- */
|
|
||||||
+
|
|
||||||
+ \note In Qt 5, calling setAcceptedMouseButtons() implicitly caused
|
|
||||||
+ an item to receive touch events as well as mouse events; but it was
|
|
||||||
+ recommended to call setAcceptTouchEvents() to subscribe for them.
|
|
||||||
+ In Qt 6, it is necessary to call setAcceptTouchEvents() to continue
|
|
||||||
+ to receive them.
|
|
||||||
+*/
|
|
||||||
void QQuickItem::setAcceptedMouseButtons(Qt::MouseButtons buttons)
|
|
||||||
{
|
|
||||||
Q_D(QQuickItem);
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -1,192 +0,0 @@
|
|||||||
From 55324650f9e759a43dce927f823c9858574106c3 Mon Sep 17 00:00:00 2001
|
|
||||||
From: Alexey Edelev <alexey.edelev@qt.io>
|
|
||||||
Date: Tue, 12 Jan 2021 16:37:09 +0100
|
|
||||||
Subject: [PATCH 36/36] Do not revert properties of deleted objects
|
|
||||||
|
|
||||||
If state contains revert action of properties of deleted objects,
|
|
||||||
we should avoid adding them to apply list
|
|
||||||
|
|
||||||
Fixes: QTBUG-85106
|
|
||||||
Pick-to: 5.15
|
|
||||||
Change-Id: Iff57eb9958a054476096f6d951ab7390277a2b39
|
|
||||||
Reviewed-by: Ulf Hermann <ulf.hermann@qt.io>
|
|
||||||
(cherry picked from commit 96763dbb105fde20431a264789ac27abfdab841c)
|
|
||||||
---
|
|
||||||
src/quick/util/qquickstate.cpp | 5 ++
|
|
||||||
.../data/revertNullObjectBinding.qml | 48 +++++++++++++
|
|
||||||
.../quick/qquickstates/tst_qquickstates.cpp | 68 +++++++++++++++++++
|
|
||||||
3 files changed, 121 insertions(+)
|
|
||||||
create mode 100644 tests/auto/quick/qquickstates/data/revertNullObjectBinding.qml
|
|
||||||
|
|
||||||
diff --git a/src/quick/util/qquickstate.cpp b/src/quick/util/qquickstate.cpp
|
|
||||||
index 71ab1f4d62..6a72754bde 100644
|
|
||||||
--- a/src/quick/util/qquickstate.cpp
|
|
||||||
+++ b/src/quick/util/qquickstate.cpp
|
|
||||||
@@ -635,6 +635,11 @@ void QQuickState::apply(QQuickTransition *trans, QQuickState *revert)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
if (!found) {
|
|
||||||
+ // If revert list contains bindings assigned to deleted objects, we need to
|
|
||||||
+ // prevent reverting properties of those objects.
|
|
||||||
+ if (d->revertList.at(ii).binding() && !d->revertList.at(ii).property().object()) {
|
|
||||||
+ continue;
|
|
||||||
+ }
|
|
||||||
QVariant cur = d->revertList.at(ii).property().read();
|
|
||||||
QQmlPropertyPrivate::removeBinding(d->revertList.at(ii).property());
|
|
||||||
|
|
||||||
diff --git a/tests/auto/quick/qquickstates/data/revertNullObjectBinding.qml b/tests/auto/quick/qquickstates/data/revertNullObjectBinding.qml
|
|
||||||
new file mode 100644
|
|
||||||
index 0000000000..dee82f52ed
|
|
||||||
--- /dev/null
|
|
||||||
+++ b/tests/auto/quick/qquickstates/data/revertNullObjectBinding.qml
|
|
||||||
@@ -0,0 +1,48 @@
|
|
||||||
+import QtQuick 2.12
|
|
||||||
+import Qt.test 1.0
|
|
||||||
+
|
|
||||||
+Item {
|
|
||||||
+ id: root
|
|
||||||
+ readonly property int someProp: 1234
|
|
||||||
+
|
|
||||||
+ property bool state1Active: false
|
|
||||||
+ property bool state2Active: false
|
|
||||||
+ StateGroup {
|
|
||||||
+ states: [
|
|
||||||
+ State {
|
|
||||||
+ id: state1
|
|
||||||
+ name: "state1"
|
|
||||||
+ when: state1Active
|
|
||||||
+ changes: [
|
|
||||||
+ PropertyChanges {
|
|
||||||
+ objectName: "propertyChanges1"
|
|
||||||
+ target: ContainingObj.obj
|
|
||||||
+ prop: root.someProp
|
|
||||||
+ }
|
|
||||||
+ ]
|
|
||||||
+ }
|
|
||||||
+ ]}
|
|
||||||
+ StateGroup {
|
|
||||||
+ states: [
|
|
||||||
+ State {
|
|
||||||
+ id: state2
|
|
||||||
+ name: "state2"
|
|
||||||
+ when: state2Active
|
|
||||||
+ changes: [
|
|
||||||
+ PropertyChanges {
|
|
||||||
+ objectName: "propertyChanges2"
|
|
||||||
+ target: ContainingObj.obj
|
|
||||||
+ prop: 11111
|
|
||||||
+ }
|
|
||||||
+ ]
|
|
||||||
+ }
|
|
||||||
+ ]
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ Component.onCompleted: {
|
|
||||||
+ state1Active = true;
|
|
||||||
+ state2Active = true;
|
|
||||||
+
|
|
||||||
+ ContainingObj.reset()
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
diff --git a/tests/auto/quick/qquickstates/tst_qquickstates.cpp b/tests/auto/quick/qquickstates/tst_qquickstates.cpp
|
|
||||||
index d5fea3cb28..849522454f 100644
|
|
||||||
--- a/tests/auto/quick/qquickstates/tst_qquickstates.cpp
|
|
||||||
+++ b/tests/auto/quick/qquickstates/tst_qquickstates.cpp
|
|
||||||
@@ -79,6 +79,55 @@ private:
|
|
||||||
QML_DECLARE_TYPE(MyRect)
|
|
||||||
QML_DECLARE_TYPEINFO(MyRect, QML_HAS_ATTACHED_PROPERTIES)
|
|
||||||
|
|
||||||
+class RemovableObj : public QObject
|
|
||||||
+{
|
|
||||||
+ Q_OBJECT
|
|
||||||
+ Q_PROPERTY(int prop READ prop WRITE setProp NOTIFY propChanged)
|
|
||||||
+
|
|
||||||
+public:
|
|
||||||
+ RemovableObj(QObject *parent) : QObject(parent), m_prop(4321) { }
|
|
||||||
+ int prop() const { return m_prop; }
|
|
||||||
+
|
|
||||||
+public slots:
|
|
||||||
+ void setProp(int prop)
|
|
||||||
+ {
|
|
||||||
+ if (m_prop == prop)
|
|
||||||
+ return;
|
|
||||||
+
|
|
||||||
+ m_prop = prop;
|
|
||||||
+ emit propChanged(m_prop);
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+signals:
|
|
||||||
+ void propChanged(int prop);
|
|
||||||
+
|
|
||||||
+private:
|
|
||||||
+ int m_prop;
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
+class ContainingObj : public QObject
|
|
||||||
+{
|
|
||||||
+ Q_OBJECT
|
|
||||||
+ Q_PROPERTY(RemovableObj *obj READ obj NOTIFY objChanged)
|
|
||||||
+ RemovableObj *m_obj;
|
|
||||||
+
|
|
||||||
+public:
|
|
||||||
+ ContainingObj() : m_obj(new RemovableObj(this)) { }
|
|
||||||
+ RemovableObj *obj() const { return m_obj; }
|
|
||||||
+
|
|
||||||
+ Q_INVOKABLE void reset()
|
|
||||||
+ {
|
|
||||||
+ if (m_obj) {
|
|
||||||
+ m_obj->deleteLater();
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ m_obj = new RemovableObj(this);
|
|
||||||
+ emit objChanged();
|
|
||||||
+ }
|
|
||||||
+signals:
|
|
||||||
+ void objChanged();
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
class tst_qquickstates : public QQmlDataTest
|
|
||||||
{
|
|
||||||
Q_OBJECT
|
|
||||||
@@ -140,12 +189,20 @@ private slots:
|
|
||||||
void duplicateStateName();
|
|
||||||
void trivialWhen();
|
|
||||||
void parentChangeCorrectReversal();
|
|
||||||
+ void revertNullObjectBinding();
|
|
||||||
};
|
|
||||||
|
|
||||||
void tst_qquickstates::initTestCase()
|
|
||||||
{
|
|
||||||
QQmlDataTest::initTestCase();
|
|
||||||
qmlRegisterType<MyRect>("Qt.test", 1, 0, "MyRectangle");
|
|
||||||
+ qmlRegisterSingletonType<ContainingObj>(
|
|
||||||
+ "Qt.test", 1, 0, "ContainingObj", [](QQmlEngine *engine, QJSEngine *) {
|
|
||||||
+ static ContainingObj instance;
|
|
||||||
+ engine->setObjectOwnership(&instance, QQmlEngine::CppOwnership);
|
|
||||||
+ return &instance;
|
|
||||||
+ });
|
|
||||||
+ qmlRegisterUncreatableType<RemovableObj>("Qt.test", 1, 0, "RemovableObj", "Uncreatable");
|
|
||||||
}
|
|
||||||
|
|
||||||
QByteArray tst_qquickstates::fullDataPath(const QString &path) const
|
|
||||||
@@ -1692,6 +1749,17 @@ void tst_qquickstates::parentChangeCorrectReversal()
|
|
||||||
QCOMPARE(oldX, stayingRectX.read().toDouble());
|
|
||||||
}
|
|
||||||
|
|
||||||
+void tst_qquickstates::revertNullObjectBinding()
|
|
||||||
+{
|
|
||||||
+ QQmlEngine engine;
|
|
||||||
+
|
|
||||||
+ QQmlComponent c(&engine, testFileUrl("revertNullObjectBinding.qml"));
|
|
||||||
+ QScopedPointer<QObject> root { c.create() };
|
|
||||||
+ QVERIFY(root);
|
|
||||||
+ QTest::qWait(10);
|
|
||||||
+ QQmlProperty state2Active(root.get(), "state2Active");
|
|
||||||
+ state2Active.write(false);
|
|
||||||
+}
|
|
||||||
|
|
||||||
QTEST_MAIN(tst_qquickstates)
|
|
||||||
|
|
||||||
--
|
|
||||||
2.31.1
|
|
||||||
|
|
@ -21,11 +21,11 @@ tests.
|
|||||||
Fixes: QTBUG-83890
|
Fixes: QTBUG-83890
|
||||||
Pick-to: 5.15
|
Pick-to: 5.15
|
||||||
Change-Id: I7f4060c2f46ae07611bedceca0d322c5f7f6affb
|
Change-Id: I7f4060c2f46ae07611bedceca0d322c5f7f6affb
|
||||||
========================================================================================================================
|
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/src/quick/items/qquickitemview.cpp
|
diff --git a/src/quick/items/qquickitemview.cpp b/src/quick/items/qquickitemview.cpp
|
||||||
===================================================================
|
index 2b4ca9e2..f2feba2a 100644
|
||||||
--- qtdeclarative-everywhere-src-5.15.2.orig/src/quick/items/qquickitemview.cpp
|
--- a/src/quick/items/qquickitemview.cpp
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/src/quick/items/qquickitemview.cpp
|
+++ b/src/quick/items/qquickitemview.cpp
|
||||||
@@ -1393,7 +1393,7 @@ qreal QQuickItemView::maxYExtent() const
|
@@ -1393,7 +1393,7 @@ qreal QQuickItemView::maxYExtent() const
|
||||||
{
|
{
|
||||||
Q_D(const QQuickItemView);
|
Q_D(const QQuickItemView);
|
||||||
@ -44,10 +44,10 @@ Index: qtdeclarative-everywhere-src-5.15.2/src/quick/items/qquickitemview.cpp
|
|||||||
|
|
||||||
if (d->hData.maxExtentDirty) {
|
if (d->hData.maxExtentDirty) {
|
||||||
d->maxExtent = d->maxExtentForAxis(d->hData, true);
|
d->maxExtent = d->maxExtentForAxis(d->hData, true);
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
diff --git a/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp b/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
||||||
===================================================================
|
index a7aefbe4..afe5c5ac 100644
|
||||||
--- qtdeclarative-everywhere-src-5.15.2.orig/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
--- a/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
+++ b/tests/auto/quick/qquicklistview/tst_qquicklistview.cpp
|
||||||
@@ -73,6 +73,8 @@ public:
|
@@ -73,6 +73,8 @@ public:
|
||||||
tst_QQuickListView();
|
tst_QQuickListView();
|
||||||
|
|
||||||
@ -57,17 +57,17 @@ Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview/tst_q
|
|||||||
void init();
|
void init();
|
||||||
void cleanupTestCase();
|
void cleanupTestCase();
|
||||||
// Test QAbstractItemModel model types
|
// Test QAbstractItemModel model types
|
||||||
@@ -299,6 +301,8 @@ private slots:
|
@@ -300,6 +302,8 @@ private slots:
|
||||||
void requiredObjectListModel();
|
|
||||||
void clickHeaderAndFooterWhenClip();
|
void clickHeaderAndFooterWhenClip();
|
||||||
|
void animatedDelegate();
|
||||||
|
|
||||||
+ // WARNING: please add new tests to tst_qquicklistview2; this file is too slow to work with.
|
+ // WARNING: please add new tests to tst_qquicklistview2; this file is too slow to work with.
|
||||||
+
|
+
|
||||||
private:
|
private:
|
||||||
template <class T> void items(const QUrl &source);
|
template <class T> void items(const QUrl &source);
|
||||||
template <class T> void changed(const QUrl &source);
|
template <class T> void changed(const QUrl &source);
|
||||||
@@ -10094,6 +10098,8 @@ void tst_QQuickListView::clickHeaderAndF
|
@@ -10109,6 +10113,8 @@ void tst_QQuickListView::animatedDelegate()
|
||||||
QVERIFY(root->property("footerPressed").toBool());
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
+// WARNING: please add new tests to tst_qquicklistview2; this file is too slow to work with.
|
+// WARNING: please add new tests to tst_qquicklistview2; this file is too slow to work with.
|
||||||
@ -75,265 +75,10 @@ Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview/tst_q
|
|||||||
QTEST_MAIN(tst_QQuickListView)
|
QTEST_MAIN(tst_QQuickListView)
|
||||||
|
|
||||||
#include "tst_qquicklistview.moc"
|
#include "tst_qquicklistview.moc"
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/data/maxXExtent.qml
|
diff --git a/tests/auto/quick/quick.pro b/tests/auto/quick/quick.pro
|
||||||
===================================================================
|
index 45bcf8a9..00f7d64d 100644
|
||||||
--- /dev/null
|
--- a/tests/auto/quick/quick.pro
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/data/maxXExtent.qml
|
+++ b/tests/auto/quick/quick.pro
|
||||||
@@ -0,0 +1,54 @@
|
|
||||||
+/****************************************************************************
|
|
||||||
+**
|
|
||||||
+** Copyright (C) 2020 The Qt Company Ltd.
|
|
||||||
+** Contact: https://www.qt.io/licensing/
|
|
||||||
+**
|
|
||||||
+** This file is part of the test suite of the Qt Toolkit.
|
|
||||||
+**
|
|
||||||
+** $QT_BEGIN_LICENSE:GPL-EXCEPT$
|
|
||||||
+** Commercial License Usage
|
|
||||||
+** Licensees holding valid commercial Qt licenses may use this file in
|
|
||||||
+** accordance with the commercial license agreement provided with the
|
|
||||||
+** Software or, alternatively, in accordance with the terms contained in
|
|
||||||
+** a written agreement between you and The Qt Company. For licensing terms
|
|
||||||
+** and conditions see https://www.qt.io/terms-conditions. For further
|
|
||||||
+** information use the contact form at https://www.qt.io/contact-us.
|
|
||||||
+**
|
|
||||||
+** GNU General Public License Usage
|
|
||||||
+** Alternatively, this file may be used under the terms of the GNU
|
|
||||||
+** General Public License version 3 as published by the Free Software
|
|
||||||
+** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
|
|
||||||
+** included in the packaging of this file. Please review the following
|
|
||||||
+** information to ensure the GNU General Public License requirements will
|
|
||||||
+** be met: https://www.gnu.org/licenses/gpl-3.0.html.
|
|
||||||
+**
|
|
||||||
+** $QT_END_LICENSE$
|
|
||||||
+**
|
|
||||||
+****************************************************************************/
|
|
||||||
+
|
|
||||||
+import QtQuick 2.15
|
|
||||||
+
|
|
||||||
+Item {
|
|
||||||
+ property alias view: view
|
|
||||||
+
|
|
||||||
+ ListView {
|
|
||||||
+ id: view
|
|
||||||
+ model: 10
|
|
||||||
+ width: 200
|
|
||||||
+ height: 200
|
|
||||||
+
|
|
||||||
+ Rectangle {
|
|
||||||
+ anchors.fill: parent
|
|
||||||
+ color: "transparent"
|
|
||||||
+ border.color: "darkorange"
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ delegate: Rectangle {
|
|
||||||
+ width: 100
|
|
||||||
+ height: 100
|
|
||||||
+ Text {
|
|
||||||
+ text: modelData
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/data/maxYExtent.qml
|
|
||||||
===================================================================
|
|
||||||
--- /dev/null
|
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/data/maxYExtent.qml
|
|
||||||
@@ -0,0 +1,55 @@
|
|
||||||
+/****************************************************************************
|
|
||||||
+**
|
|
||||||
+** Copyright (C) 2020 The Qt Company Ltd.
|
|
||||||
+** Contact: https://www.qt.io/licensing/
|
|
||||||
+**
|
|
||||||
+** This file is part of the test suite of the Qt Toolkit.
|
|
||||||
+**
|
|
||||||
+** $QT_BEGIN_LICENSE:GPL-EXCEPT$
|
|
||||||
+** Commercial License Usage
|
|
||||||
+** Licensees holding valid commercial Qt licenses may use this file in
|
|
||||||
+** accordance with the commercial license agreement provided with the
|
|
||||||
+** Software or, alternatively, in accordance with the terms contained in
|
|
||||||
+** a written agreement between you and The Qt Company. For licensing terms
|
|
||||||
+** and conditions see https://www.qt.io/terms-conditions. For further
|
|
||||||
+** information use the contact form at https://www.qt.io/contact-us.
|
|
||||||
+**
|
|
||||||
+** GNU General Public License Usage
|
|
||||||
+** Alternatively, this file may be used under the terms of the GNU
|
|
||||||
+** General Public License version 3 as published by the Free Software
|
|
||||||
+** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
|
|
||||||
+** included in the packaging of this file. Please review the following
|
|
||||||
+** information to ensure the GNU General Public License requirements will
|
|
||||||
+** be met: https://www.gnu.org/licenses/gpl-3.0.html.
|
|
||||||
+**
|
|
||||||
+** $QT_END_LICENSE$
|
|
||||||
+**
|
|
||||||
+****************************************************************************/
|
|
||||||
+
|
|
||||||
+import QtQuick 2.15
|
|
||||||
+
|
|
||||||
+Item {
|
|
||||||
+ property alias view: view
|
|
||||||
+
|
|
||||||
+ ListView {
|
|
||||||
+ id: view
|
|
||||||
+ model: 10
|
|
||||||
+ width: 200
|
|
||||||
+ height: 200
|
|
||||||
+ orientation: ListView.Horizontal
|
|
||||||
+
|
|
||||||
+ Rectangle {
|
|
||||||
+ anchors.fill: parent
|
|
||||||
+ color: "transparent"
|
|
||||||
+ border.color: "darkorange"
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ delegate: Rectangle {
|
|
||||||
+ width: 100
|
|
||||||
+ height: 100
|
|
||||||
+ Text {
|
|
||||||
+ text: modelData
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+ }
|
|
||||||
+}
|
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/qquicklistview2.pro
|
|
||||||
===================================================================
|
|
||||||
--- /dev/null
|
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/qquicklistview2.pro
|
|
||||||
@@ -0,0 +1,12 @@
|
|
||||||
+CONFIG += testcase
|
|
||||||
+TARGET = tst_qquicklistview2
|
|
||||||
+macos:CONFIG -= app_bundle
|
|
||||||
+
|
|
||||||
+SOURCES += tst_qquicklistview2.cpp
|
|
||||||
+
|
|
||||||
+include (../../shared/util.pri)
|
|
||||||
+include (../shared/util.pri)
|
|
||||||
+
|
|
||||||
+TESTDATA = data/*
|
|
||||||
+
|
|
||||||
+QT += core-private gui-private qml-private quick-private testlib qmltest
|
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/tst_qquicklistview2.cpp
|
|
||||||
===================================================================
|
|
||||||
--- /dev/null
|
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/qquicklistview2/tst_qquicklistview2.cpp
|
|
||||||
@@ -0,0 +1,114 @@
|
|
||||||
+/****************************************************************************
|
|
||||||
+**
|
|
||||||
+** Copyright (C) 2020 The Qt Company Ltd.
|
|
||||||
+** Contact: https://www.qt.io/licensing/
|
|
||||||
+**
|
|
||||||
+** This file is part of the test suite of the Qt Toolkit.
|
|
||||||
+**
|
|
||||||
+** $QT_BEGIN_LICENSE:GPL-EXCEPT$
|
|
||||||
+** Commercial License Usage
|
|
||||||
+** Licensees holding valid commercial Qt licenses may use this file in
|
|
||||||
+** accordance with the commercial license agreement provided with the
|
|
||||||
+** Software or, alternatively, in accordance with the terms contained in
|
|
||||||
+** a written agreement between you and The Qt Company. For licensing terms
|
|
||||||
+** and conditions see https://www.qt.io/terms-conditions. For further
|
|
||||||
+** information use the contact form at https://www.qt.io/contact-us.
|
|
||||||
+**
|
|
||||||
+** GNU General Public License Usage
|
|
||||||
+** Alternatively, this file may be used under the terms of the GNU
|
|
||||||
+** General Public License version 3 as published by the Free Software
|
|
||||||
+** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
|
|
||||||
+** included in the packaging of this file. Please review the following
|
|
||||||
+** information to ensure the GNU General Public License requirements will
|
|
||||||
+** be met: https://www.gnu.org/licenses/gpl-3.0.html.
|
|
||||||
+**
|
|
||||||
+** $QT_END_LICENSE$
|
|
||||||
+**
|
|
||||||
+****************************************************************************/
|
|
||||||
+
|
|
||||||
+#include <QtTest/QtTest>
|
|
||||||
+#include <QtQuickTest/QtQuickTest>
|
|
||||||
+#include <QtQml/qqmlapplicationengine.h>
|
|
||||||
+#include <QtQuick/qquickview.h>
|
|
||||||
+#include <QtQuick/private/qquicklistview_p.h>
|
|
||||||
+
|
|
||||||
+#include "../../shared/util.h"
|
|
||||||
+#include "../shared/viewtestutil.h"
|
|
||||||
+
|
|
||||||
+using namespace QQuickViewTestUtil;
|
|
||||||
+
|
|
||||||
+class tst_QQuickListView2 : public QQmlDataTest
|
|
||||||
+{
|
|
||||||
+ Q_OBJECT
|
|
||||||
+
|
|
||||||
+public:
|
|
||||||
+ tst_QQuickListView2();
|
|
||||||
+
|
|
||||||
+private slots:
|
|
||||||
+ void maxExtent_data();
|
|
||||||
+ void maxExtent();
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
+tst_QQuickListView2::tst_QQuickListView2()
|
|
||||||
+{
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+class FriendlyItemView : public QQuickItemView
|
|
||||||
+{
|
|
||||||
+ friend class ItemViewAccessor;
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
+class ItemViewAccessor
|
|
||||||
+{
|
|
||||||
+public:
|
|
||||||
+ ItemViewAccessor(QQuickItemView *itemView) :
|
|
||||||
+ mItemView(reinterpret_cast<FriendlyItemView*>(itemView))
|
|
||||||
+ {
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ qreal maxXExtent() const
|
|
||||||
+ {
|
|
||||||
+ return mItemView->maxXExtent();
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+ qreal maxYExtent() const
|
|
||||||
+ {
|
|
||||||
+ return mItemView->maxYExtent();
|
|
||||||
+ }
|
|
||||||
+
|
|
||||||
+private:
|
|
||||||
+ FriendlyItemView *mItemView = nullptr;
|
|
||||||
+};
|
|
||||||
+
|
|
||||||
+void tst_QQuickListView2::maxExtent_data()
|
|
||||||
+{
|
|
||||||
+ QTest::addColumn<QString>("qmlFilePath");
|
|
||||||
+
|
|
||||||
+ QTest::addRow("maxXExtent") << "maxXExtent.qml";
|
|
||||||
+ QTest::addRow("maxYExtent") << "maxYExtent.qml";
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+void tst_QQuickListView2::maxExtent()
|
|
||||||
+{
|
|
||||||
+ QFETCH(QString, qmlFilePath);
|
|
||||||
+
|
|
||||||
+ QScopedPointer<QQuickView> window(createView());
|
|
||||||
+ QVERIFY(window);
|
|
||||||
+ window->setSource(testFileUrl(qmlFilePath));
|
|
||||||
+ QVERIFY2(window->status() == QQuickView::Ready, qPrintable(QDebug::toString(window->errors())));
|
|
||||||
+ window->resize(640, 480);
|
|
||||||
+ window->show();
|
|
||||||
+ QVERIFY(QTest::qWaitForWindowExposed(window.data()));
|
|
||||||
+
|
|
||||||
+ QQuickListView *view = window->rootObject()->property("view").value<QQuickListView*>();
|
|
||||||
+ QVERIFY(view);
|
|
||||||
+ ItemViewAccessor viewAccessor(view);
|
|
||||||
+ if (view->orientation() == QQuickListView::Vertical)
|
|
||||||
+ QCOMPARE(viewAccessor.maxXExtent(), 0);
|
|
||||||
+ else if (view->orientation() == QQuickListView::Horizontal)
|
|
||||||
+ QCOMPARE(viewAccessor.maxYExtent(), 0);
|
|
||||||
+}
|
|
||||||
+
|
|
||||||
+QTEST_MAIN(tst_QQuickListView2)
|
|
||||||
+
|
|
||||||
+#include "tst_qquicklistview2.moc"
|
|
||||||
Index: qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/quick.pro
|
|
||||||
===================================================================
|
|
||||||
--- qtdeclarative-everywhere-src-5.15.2.orig/tests/auto/quick/quick.pro
|
|
||||||
+++ qtdeclarative-everywhere-src-5.15.2/tests/auto/quick/quick.pro
|
|
||||||
@@ -67,6 +67,7 @@ QUICKTESTS += \
|
@@ -67,6 +67,7 @@ QUICKTESTS += \
|
||||||
qquickitem2 \
|
qquickitem2 \
|
||||||
qquickitemlayer \
|
qquickitemlayer \
|
||||||
|
@ -1,3 +1,6 @@
|
|||||||
|
# Bug: https://bugzilla.redhat.com/show_bug.cgi?id=2061194
|
||||||
|
%define _lto_cflags %{nil}
|
||||||
|
|
||||||
%global qt_module qtdeclarative
|
%global qt_module qtdeclarative
|
||||||
|
|
||||||
# definition borrowed from qtbase
|
# definition borrowed from qtbase
|
||||||
@ -9,14 +12,14 @@
|
|||||||
|
|
||||||
Summary: Qt5 - QtDeclarative component
|
Summary: Qt5 - QtDeclarative component
|
||||||
Name: qt5-%{qt_module}
|
Name: qt5-%{qt_module}
|
||||||
Version: 5.15.2
|
Version: 5.15.3
|
||||||
Release: 10%{?dist}
|
Release: 1%{?dist}
|
||||||
|
|
||||||
# See LICENSE.GPL LICENSE.LGPL LGPL_EXCEPTION.txt, for details
|
# See LICENSE.GPL LICENSE.LGPL LGPL_EXCEPTION.txt, for details
|
||||||
License: LGPLv2 with exceptions or GPLv3 with exceptions
|
License: LGPLv2 with exceptions or GPLv3 with exceptions
|
||||||
Url: http://www.qt.io
|
Url: http://www.qt.io
|
||||||
%global majmin %(echo %{version} | cut -d. -f1-2)
|
%global majmin %(echo %{version} | cut -d. -f1-2)
|
||||||
Source0: https://download.qt.io/official_releases/qt/%{majmin}/%{version}/submodules/%{qt_module}-everywhere-src-%{version}.tar.xz
|
Source0: https://download.qt.io/official_releases/qt/%{majmin}/%{version}/submodules/%{qt_module}-everywhere-opensource-src-%{version}.tar.xz
|
||||||
|
|
||||||
# header file to workaround multilib issue
|
# header file to workaround multilib issue
|
||||||
# https://bugzilla.redhat.com/show_bug.cgi?id=1441343
|
# https://bugzilla.redhat.com/show_bug.cgi?id=1441343
|
||||||
@ -25,48 +28,27 @@ Source5: qv4global_p-multilib.h
|
|||||||
## upstream patches
|
## upstream patches
|
||||||
## repo: https://invent.kde.org/qt/qt/qtdeclarative
|
## repo: https://invent.kde.org/qt/qt/qtdeclarative
|
||||||
## branch: kde/5.15
|
## branch: kde/5.15
|
||||||
## git format-patch v5.15.2
|
## git format-patch v5.15.3-lts-lgpl
|
||||||
## Some of these are already included in stock 5.15.2 tarball, referenced here for completeness
|
Patch1: 0001-Give-a-warning-when-StyledText-encounters-a-non-supp.patch
|
||||||
#Patch1: 0001-Bump-version.patch
|
Patch2: 0002-Add-missing-limits-include-to-fix-build-with-GCC-11.patch
|
||||||
#Patch2: 0002-Inline-components-Fix-custom-parser-support.patch
|
Patch3: 0003-Document-that-StyledText-also-supports-nbsp-and-quot.patch
|
||||||
#Patch3: 0003-Revert-qquickloader-Free-memory-of-loaded-components.patch
|
Patch4: 0004-Support-apos-in-styled-text.patch
|
||||||
#Patch4: 0004-Add-changes-file-for-Qt-5.15.2.patch
|
Patch5: 0005-Remove-unused-QPointer-QQuickPointerMask.patch
|
||||||
Patch5: 0005-QQuickView-docs-show-correct-usage-of-setInitialProp.patch
|
Patch6: 0006-Include-limits-in-Yarr.h-to-fix-build-with-GCC-11.patch
|
||||||
Patch6: 0006-QQuickWindow-Check-if-QQuickItem-was-not-deleted.patch
|
Patch7: 0007-QQuickLoader-Do-not-incubate-if-the-source-arrives-a.patch
|
||||||
Patch7: 0007-Avoid-GHS-linker-to-optimize-away-QML-type-registrat.patch
|
Patch8: 0008-QQmlDelegateModel-Refresh-the-view-when-a-column-is-.patch
|
||||||
Patch8: 0008-QML-Text-doesn-t-reset-lineCount-when-text-is-empty.patch
|
Patch9: 0009-Fix-sweep-step-for-tainted-QObject-JavaScript-wrappe.patch
|
||||||
Patch9: 0009-Doc-mention-that-INCLUDEPATH-must-be-set-in-some-cas.patch
|
Patch10: 0010-Fix-distorted-text-with-subpixel-matrix-translation.patch
|
||||||
Patch10: 0010-qmlfunctions.qdoc-Add-clarification-to-QML_FOREIGN.patch
|
Patch11: 0011-Revert-Fix-for-possible-crash-in-QSGDefaultLayer-gra.patch
|
||||||
Patch11: 0011-Fix-QML-property-cache-leaks-of-delegate-items.patch
|
Patch12: 0012-QQuickItemAnimation-close-potential-memory-leak.patch
|
||||||
Patch12: 0012-QQuickTextInput-Store-mask-data-in-std-unique_ptr.patch
|
Patch13: 0013-qqmldelegatemodel-Fix-out-of-bounds-cache-removal.patch
|
||||||
Patch13: 0013-Fix-crash-when-calling-hasOwnProperty-on-proxy-objec.patch
|
Patch14: 0014-QQuickWindow-don-t-leak-old-screenChanged-connection.patch
|
||||||
Patch14: 0014-Accessibility-event-is-sent-on-item-s-geometry-chang.patch
|
Patch15: 0015-Fix-TapHandler-so-that-it-actually-registers-a-tap.patch
|
||||||
Patch15: 0015-qmltypes.prf-Take-abi-into-account-for-_metatypes.js.patch
|
Patch16: 0016-Revert-Fix-TapHandler-so-that-it-actually-registers-.patch
|
||||||
Patch16: 0016-qv4qmlcontext-Fix-bounded-signal-expressions-when-de.patch
|
Patch17: 0017-QQmlJs-FixedPoolArray-fix-UB-precondition-violation-.patch
|
||||||
Patch17: 0017-Use-load-qt_tool-for-qmltime.patch
|
Patch18: 0018-QQuickTextInput-update-cursor-rectangle-after-paddin.patch
|
||||||
Patch18: 0018-qqmlistmodel-Fix-crash-when-modelCache-is-null.patch
|
Patch19: 0019-V4-Do-not-call-dtor-of-an-object-we-continue-to-use.patch
|
||||||
Patch19: 0019-Show-a-tableview-even-if-the-syncView-has-an-empty-m.patch
|
Patch20: 0020-Make-sure-QQuickWidget-and-its-offscreen-window-s-sc.patch
|
||||||
Patch20: 0020-DesignerSupport-Don-t-skip-already-inspected-objects.patch
|
|
||||||
Patch21: 0021-QML-Fix-proxy-iteration.patch
|
|
||||||
Patch22: 0022-Fix-IC-properties-in-same-file.patch
|
|
||||||
Patch23: 0023-JIT-When-making-memory-writable-include-the-exceptio.patch
|
|
||||||
Patch24: 0024-doc-explain-QQItem-event-delivery-handlers-setAccept.patch
|
|
||||||
Patch25: 0025-Give-a-warning-when-StyledText-encounters-a-non-supp.patch
|
|
||||||
Patch26: 0026-Add-missing-limits-include-to-fix-build-with-GCC-11.patch
|
|
||||||
Patch27: 0027-Document-that-StyledText-also-supports-nbsp-and-quot.patch
|
|
||||||
Patch28: 0028-Support-apos-in-styled-text.patch
|
|
||||||
Patch29: 0029-Remove-unused-QPointer-QQuickPointerMask.patch
|
|
||||||
Patch30: 0030-Include-limits-in-Yarr.h-to-fix-build-with-GCC-11.patch
|
|
||||||
Patch31: 0031-QQuickLoader-Do-not-incubate-if-the-source-arrives-a.patch
|
|
||||||
Patch32: 0032-QQmlDelegateModel-Refresh-the-view-when-a-column-is-.patch
|
|
||||||
Patch33: 0033-Fix-sweep-step-for-tainted-QObject-JavaScript-wrappe.patch
|
|
||||||
Patch34: 0034-Fix-distorted-text-with-subpixel-matrix-translation.patch
|
|
||||||
Patch35: 0035-Revert-Fix-for-possible-crash-in-QSGDefaultLayer-gra.patch
|
|
||||||
Patch36: 0036-Do-not-revert-properties-of-deleted-objects.patch
|
|
||||||
Patch37: 0037-QQuickItemAnimation-close-potential-memory-leak.patch
|
|
||||||
Patch38: 0038-qqmldelegatemodel-Fix-out-of-bounds-cache-removal.patch
|
|
||||||
Patch39: 0039-QQuickWindow-don-t-leak-old-screenChanged-connection.patch
|
|
||||||
|
|
||||||
|
|
||||||
## upstreamable patches
|
## upstreamable patches
|
||||||
Patch100: %{name}-gcc11.patch
|
Patch100: %{name}-gcc11.patch
|
||||||
@ -269,6 +251,10 @@ make check -k -C tests ||:
|
|||||||
%endif
|
%endif
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Fri Mar 25 2022 Jan Grulich <jgrulich@redhat.com> - 5.15.3-1
|
||||||
|
- 5.15.3 + sync with Fedora
|
||||||
|
Resolves: bz#2061355
|
||||||
|
|
||||||
* Thu Dec 09 2021 Jan Grulich <jgrulich@redhat.com> - 5.15.2-10
|
* Thu Dec 09 2021 Jan Grulich <jgrulich@redhat.com> - 5.15.2-10
|
||||||
- Sync with Fedora
|
- Sync with Fedora
|
||||||
Resolves: bz#2025420
|
Resolves: bz#2025420
|
||||||
|
Loading…
Reference in New Issue
Block a user