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
=== modified file 'debian/changelog'
--- debian/changelog 2014-06-19 19:14:47 +0000
+++ debian/changelog 2014-07-01 08:30:34 +0000
@@ -1,3 +1,9 @@
1qtdeclarative-opensource-src (5.3.0-3ubuntu5) utopic; urgency=medium
2
3 * QQMLObjectCreator: Pass down parent to subCreator
4
5 -- Albert Astals Cid <albert.astals@canonical.com> Tue, 01 Jul 2014 10:17:13 +0200
6
1qtdeclarative-opensource-src (5.3.0-3ubuntu4) utopic; urgency=medium7qtdeclarative-opensource-src (5.3.0-3ubuntu4) utopic; urgency=medium
28
3 [ Ricardo Salveti de Araujo ]9 [ Ricardo Salveti de Araujo ]
410
=== added file 'debian/patches/parenttosubcreator_qqmlobjectcreator.patch'
--- debian/patches/parenttosubcreator_qqmlobjectcreator.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/parenttosubcreator_qqmlobjectcreator.patch 2014-07-01 08:30:34 +0000
@@ -0,0 +1,24 @@
1commit 249b1a73fdb01fc27e036fc4334925674c281793
2Author: Albert Astals Cid <albert.astals@canonical.com>
3Date: Mon Jun 30 17:22:55 2014 +0200
4
5 Pass down parent to subCreator
6
7 Otherwise the object might get garbage collected while it's being created
8
9 Change-Id: I553c3432d5242bcd89723b72d8f8ae82577abaf9
10 Task-number: QTBUG-39966
11
12diff --git a/src/qml/qml/qqmlobjectcreator.cpp b/src/qml/qml/qqmlobjectcreator.cpp
13index 36c7dfb..cceb9b2 100644
14--- a/src/qml/qml/qqmlobjectcreator.cpp
15+++ b/src/qml/qml/qqmlobjectcreator.cpp
16@@ -1076,7 +1076,7 @@ QObject *QQmlObjectCreator::createInstance(int index, QObject *parent, bool isCo
17 }
18
19 QQmlObjectCreator subCreator(context, typeRef->component, sharedState.data());
20- instance = subCreator.create();
21+ instance = subCreator.create(-1, parent);
22 if (!instance) {
23 errors += subCreator.errors;
24 return 0;
025
=== modified file 'debian/patches/series'
--- debian/patches/series 2014-06-18 08:15:27 +0000
+++ debian/patches/series 2014-07-01 08:30:34 +0000
@@ -5,3 +5,4 @@
5Make-ItemViews-displayMargin-work-correctly-when-set.patch5Make-ItemViews-displayMargin-work-correctly-when-set.patch
6v4_yarr_jit_push_pop_addressTempRegister.patch6v4_yarr_jit_push_pop_addressTempRegister.patch
7fix_qqmlobjectcreator.patch7fix_qqmlobjectcreator.patch
8parenttosubcreator_qqmlobjectcreator.patch

Subscribers

People subscribed via source and target branches