Loading...

XML

Word

Printable

Type: Bug
Resolution: Unresolved
Priority: P1: Critical
Fix Version/s: 6.8, 6.11
Affects Version/s: 6.8.4, 6.9.1, 6.10.0 Beta2
Component/s: QML: Declarative and Javascript Engine
Labels:
None
Environment:
Windows 10 22H2, MSVC 2022 x64

Commits:
88adccfae (tqtc/lts-6.8), 44a9caffe (tqtc/lts-6.8), 4775329a2 (tqtc/lts-6.8), 984a4669a (tqtc/lts-6.8), 4e3f3ace8 (tqtc/lts-6.8), e0d4577a1 (tqtc/lts-6.8), 7215bf804 (tqtc/lts-6.8), 3ad626b87 (tqtc/lts-6.8), 2bebb36d2 (tqtc/lts-6.8)

Based on the example from ~~QTBUG-138919~~/QTBUG-138944. CppObj is a QML_ELEMENT while CppBridge is a QML_SINGLETON.

Code

import QtQuick
import JSOwnershipStudy

Window {
    width: 400
    height: 300
    visible: true

    property list<CppObj> listProp: CppBridge.multipleObjsFrom(  0) // IDs   0 --  99
    property var           varProp: CppBridge.multipleObjsFrom(100) // IDs 100 -- 199
}

Steps to reproduce
Run and quit the example

Outcomes when quitting the example

(All versions) The objects stored in listProp are destroyed upon quitting (Expected)
(Qt 6.5) The objects stored in varProp are destroyed upon quitting (Expected)
(Qt 6.8+) The objects stored in varProp are never destroyed (Not Expected)

The workaround for ~~QTBUG-138919~~ suggested by Luca Di Sera (setting QML_DISABLE_DISK_CACHE=1) does not work here.

relates to

QTBUG-138919 [Reg 6.5.9 -> 6.8.4] Unreferenced objects with JavaScriptOwnership are no longer destroyed

Closed

QTBUG-139059 Generated code does not track objects on JavaScript stack

Closed

resulted in

QTBUG-139053 Re-design transfer of QObject ownership to JavaScript

Open

split from

QTBUG-138944 List of objects with JavaScriptOwnership are not ref-counted correctly when assigned to ListView.model

In Progress

- Issue Only
- Show All Reviews
- Show Open Reviews

For Gerrit Dashboard: QTBUG-139025
#	Subject	Branch	Project	Status	CR	V
673206,3	QtQml: Store detached Sequence objects on the JS heap	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: NEW	0	+1
667498,7	WIP: Implement markObjects for QV4::Sequence	dev	qt/qtdeclarative	Status: ABANDONED	-2	0
668052,18	QtQml: Store detached VariantAssociation objects on the JS heap	dev	qt/qtdeclarative	Status: MERGED	+2	0
668640,4	QtQml: Add method to generate a QVariantHash from a JS object	dev	qt/qtdeclarative	Status: MERGED	+2	0
668744,24	QtQml: Store detached Sequence objects on the JS heap	dev	qt/qtdeclarative	Status: MERGED	+2	0
668839,2	QtQml: Add method to generate a QVariantHash from a JS object	6.10	qt/qtdeclarative	Status: MERGED	+2	0
668879,2	QtQml: Add method to generate a QVariantHash from a JS object	6.9	qt/qtdeclarative	Status: MERGED	+2	0
669180,2	QtQml: Add method to generate a QVariantHash from a JS object	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
670149,3	QtQml: Store detached VariantAssociation objects on the JS heap	6.10	qt/qtdeclarative	Status: MERGED	+2	0
670468,3	QtQml: Store detached VariantAssociation objects on the JS heap	6.9	qt/qtdeclarative	Status: MERGED	+2	0
670978,5	WIP: QtQml: Slim down external interface of QV4::Sequence	dev	qt/qtdeclarative	Status: ABANDONED	-2	0
671285,5	QtQml: Refactor QV4::Sequence's raw container operations	dev	qt/qtdeclarative	Status: MERGED	+2	0
671286,5	QtQml: Factor out Sequence's variant data creation	dev	qt/qtdeclarative	Status: MERGED	+2	0
671287,5	QtQml: Inline containerOwnPropertyKeys into its only user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671288,5	QtQml: Move Sequence's length accessors into SequencePrototype	dev	qt/qtdeclarative	Status: MERGED	+2	0
671320,6	QtQml: Move Sequence's {load\|store}Reference into the heap object	dev	qt/qtdeclarative	Status: MERGED	+2	0
671321,6	QtQml: Remove some methods from Sequence's public API	dev	qt/qtdeclarative	Status: MERGED	+2	0
671322,6	QtQml: Inline Sequence::containerGetIndexed() into its only user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671323,6	QtQml: Inline Sequence::containerPutIndex into its only user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671324,6	QtQml: Inline containerDeleteIndexedProperty into its only user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671325,6	QtQml: Inline Sequence::containerIsEqualTo into its only user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671326,6	QtQml: Eliminate Heap::Sequence::hasData()	dev	qt/qtdeclarative	Status: MERGED	+2	0
671327,6	QtQml: Accept Heap::Sequence in doGetIndexed()	dev	qt/qtdeclarative	Status: MERGED	+2	0
671347,5	QtQml: Inline Sequence's shiftInline into its ownly user	dev	qt/qtdeclarative	Status: MERGED	+2	0
671348,5	QtQml: Heap::Sequence::isReference() where appropriate	dev	qt/qtdeclarative	Status: MERGED	+2	0
671349,5	QtQml: Rephrase Sequence's appendInline()	dev	qt/qtdeclarative	Status: MERGED	+2	0
671350,5	QtQml: Avoid repeated retrieval of Heap::Sequence::storagePointer()	dev	qt/qtdeclarative	Status: MERGED	+2	0
671351,7	QtQml: Extract method to create the inline storage for Sequence	dev	qt/qtdeclarative	Status: MERGED	+2	0
672533,2	QtQml: Refactor QV4::Sequence's raw container operations	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672534,2	QtQml: Factor out Sequence's variant data creation	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672535,2	QtQml: Inline containerOwnPropertyKeys into its only user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672536,2	QtQml: Move Sequence's length accessors into SequencePrototype	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672537,2	QtQml: Move Sequence's {load\|store}Reference into the heap object	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672538,2	QtQml: Remove some methods from Sequence's public API	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672539,2	QtQml: Inline Sequence::containerGetIndexed() into its only user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672540,2	QtQml: Inline Sequence::containerPutIndex into its only user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672541,2	QtQml: Inline containerDeleteIndexedProperty into its only user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672542,2	QtQml: Inline Sequence::containerIsEqualTo into its only user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672543,2	QtQml: Eliminate Heap::Sequence::hasData()	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672544,2	QtQml: Accept Heap::Sequence in doGetIndexed()	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672545,2	QtQml: Inline Sequence's shiftInline into its ownly user	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672546,2	QtQml: Heap::Sequence::isReference() where appropriate	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672547,2	QtQml: Rephrase Sequence's appendInline()	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672548,2	QtQml: Avoid repeated retrieval of Heap::Sequence::storagePointer()	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672549,2	QtQml: Extract method to create the inline storage for Sequence	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672605,2	QtQml: Refactor QV4::Sequence's raw container operations	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672766,3	QtQml: Store detached Sequence objects on the JS heap	6.10	qt/qtdeclarative	Status: MERGED	+2	0
672784,2	QtQml: Factor out Sequence's variant data creation	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672785,2	QtQml: Inline containerOwnPropertyKeys into its only user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672786,2	QtQml: Move Sequence's length accessors into SequencePrototype	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672787,2	QtQml: Move Sequence's {load\|store}Reference into the heap object	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672788,2	QtQml: Remove some methods from Sequence's public API	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672789,2	QtQml: Inline Sequence::containerGetIndexed() into its only user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672790,2	QtQml: Inline Sequence::containerPutIndex into its only user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672791,2	QtQml: Inline containerDeleteIndexedProperty into its only user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672792,2	QtQml: Inline Sequence::containerIsEqualTo into its only user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672793,2	QtQml: Eliminate Heap::Sequence::hasData()	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672794,2	QtQml: Accept Heap::Sequence in doGetIndexed()	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672795,2	QtQml: Inline Sequence's shiftInline into its ownly user	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672796,2	QtQml: Heap::Sequence::isReference() where appropriate	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672797,2	QtQml: Rephrase Sequence's appendInline()	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672798,2	QtQml: Avoid repeated retrieval of Heap::Sequence::storagePointer()	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672799,2	QtQml: Extract method to create the inline storage for Sequence	6.9	qt/qtdeclarative	Status: MERGED	+2	0
672803,4	QtQml: Refactor QV4::Sequence's raw container operations	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672877,4	QtQml: Factor out Sequence's variant data creation	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672878,2	QtQml: Inline containerOwnPropertyKeys into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672879,2	QtQml: Move Sequence's length accessors into SequencePrototype	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672880,4	QtQml: Move Sequence's {load\|store}Reference into the heap object	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672881,4	QtQml: Remove some methods from Sequence's public API	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672882,4	QtQml: Inline Sequence::containerGetIndexed() into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672883,4	QtQml: Inline Sequence::containerPutIndex into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672884,4	QtQml: Inline containerDeleteIndexedProperty into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672885,4	QtQml: Inline Sequence::containerIsEqualTo into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672886,6	QtQml: Eliminate Heap::Sequence::hasData()	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672887,6	QtQml: Accept Heap::Sequence in doGetIndexed()	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672888,6	QtQml: Inline Sequence's shiftInline into its ownly user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672889,6	QtQml: Heap::Sequence::isReference() where appropriate	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672890,6	QtQml: Rephrase Sequence's appendInline()	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672891,2	QtQml: Avoid repeated retrieval of Heap::Sequence::storagePointer()	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
672892,6	QtQml: Extract method to create the inline storage for Sequence	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
673053,5	QtQml: Inline Sequence::sort() into its only user	tqtc/lts-6.8	qt/tqtc-qtdeclarative	Status: MERGED	+2	0
673154,2	QtQml: Store detached Sequence objects on the JS heap	6.9	qt/qtdeclarative	Status: MERGED	+2	0