Merge lp:~aacid/kubuntu-packaging/qtdeclarative-opensource-src into lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src

Proposed by Albert Astals Cid
Status: Merged
Merged at revision: 159
Proposed branch: lp:~aacid/kubuntu-packaging/qtdeclarative-opensource-src
Merge into: lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src
Diff against target: 51 lines (+31/-0)
3 files modified
debian/changelog (+6/-0)
debian/patches/parenttosubcreator_qqmlobjectcreator.patch (+24/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~aacid/kubuntu-packaging/qtdeclarative-opensource-src
Reviewer Review Type Date Requested Status
Michał Sawicz (community) Approve
PS Jenkins bot continuous-integration Approve
Kubuntu Packagers Pending
Review via email: mp+225119@code.launchpad.net

Commit message

QQMLObjectCreator: Pass down parent to subCreator

Temporary fix until a better one comes up as suggested in https://bugreports.qt-project.org/browse/QTBUG-39966

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

Yup.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2014-06-19 19:14:47 +0000
3+++ debian/changelog 2014-07-01 08:30:34 +0000
4@@ -1,3 +1,9 @@
5+qtdeclarative-opensource-src (5.3.0-3ubuntu5) utopic; urgency=medium
6+
7+ * QQMLObjectCreator: Pass down parent to subCreator
8+
9+ -- Albert Astals Cid <albert.astals@canonical.com> Tue, 01 Jul 2014 10:17:13 +0200
10+
11 qtdeclarative-opensource-src (5.3.0-3ubuntu4) utopic; urgency=medium
12
13 [ Ricardo Salveti de Araujo ]
14
15=== added file 'debian/patches/parenttosubcreator_qqmlobjectcreator.patch'
16--- debian/patches/parenttosubcreator_qqmlobjectcreator.patch 1970-01-01 00:00:00 +0000
17+++ debian/patches/parenttosubcreator_qqmlobjectcreator.patch 2014-07-01 08:30:34 +0000
18@@ -0,0 +1,24 @@
19+commit 249b1a73fdb01fc27e036fc4334925674c281793
20+Author: Albert Astals Cid <albert.astals@canonical.com>
21+Date: Mon Jun 30 17:22:55 2014 +0200
22+
23+ Pass down parent to subCreator
24+
25+ Otherwise the object might get garbage collected while it's being created
26+
27+ Change-Id: I553c3432d5242bcd89723b72d8f8ae82577abaf9
28+ Task-number: QTBUG-39966
29+
30+diff --git a/src/qml/qml/qqmlobjectcreator.cpp b/src/qml/qml/qqmlobjectcreator.cpp
31+index 36c7dfb..cceb9b2 100644
32+--- a/src/qml/qml/qqmlobjectcreator.cpp
33++++ b/src/qml/qml/qqmlobjectcreator.cpp
34+@@ -1076,7 +1076,7 @@ QObject *QQmlObjectCreator::createInstance(int index, QObject *parent, bool isCo
35+ }
36+
37+ QQmlObjectCreator subCreator(context, typeRef->component, sharedState.data());
38+- instance = subCreator.create();
39++ instance = subCreator.create(-1, parent);
40+ if (!instance) {
41+ errors += subCreator.errors;
42+ return 0;
43
44=== modified file 'debian/patches/series'
45--- debian/patches/series 2014-06-18 08:15:27 +0000
46+++ debian/patches/series 2014-07-01 08:30:34 +0000
47@@ -5,3 +5,4 @@
48 Make-ItemViews-displayMargin-work-correctly-when-set.patch
49 v4_yarr_jit_push_pop_addressTempRegister.patch
50 fix_qqmlobjectcreator.patch
51+parenttosubcreator_qqmlobjectcreator.patch

Subscribers

People subscribed via source and target branches