Merge lp:~timo-jyrinki/kubuntu-packaging/qtdeclarative_clean_chain_of_incubated_objects into lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src

Proposed by Timo Jyrinki
Status: Merged
Merged at revision: 136
Proposed branch: lp:~timo-jyrinki/kubuntu-packaging/qtdeclarative_clean_chain_of_incubated_objects
Merge into: lp:~kubuntu-packagers/kubuntu-packaging/qtdeclarative-opensource-src
Diff against target: 64 lines (+44/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/clean_chain_of_incubated_objects.patch (+36/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~timo-jyrinki/kubuntu-packaging/qtdeclarative_clean_chain_of_incubated_objects
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Kubuntu Packagers Pending
Review via email: mp+210978@code.launchpad.net

Commit message

* debian/patches/clean_chain_of_incubated_objects.patch:
  - Backport from upstream https://codereview.qt-project.org/#change,80893

To post a comment you must log in.
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

No CI here either it seems. Built fine.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

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-03-10 16:36:07 +0000
3+++ debian/changelog 2014-03-14 08:04:38 +0000
4@@ -1,3 +1,10 @@
5+qtdeclarative-opensource-src (5.2.1-3ubuntu9) trusty; urgency=medium
6+
7+ * debian/patches/clean_chain_of_incubated_objects.patch:
8+ - Backport from upstream https://codereview.qt-project.org/#change,80893
9+
10+ -- Timo Jyrinki <timo-jyrinki@ubuntu.com> Fri, 14 Mar 2014 07:56:15 +0000
11+
12 qtdeclarative-opensource-src (5.2.1-3ubuntu8) trusty; urgency=medium
13
14 * debian/libqt5qml5.symbols:
15
16=== added file 'debian/patches/clean_chain_of_incubated_objects.patch'
17--- debian/patches/clean_chain_of_incubated_objects.patch 1970-01-01 00:00:00 +0000
18+++ debian/patches/clean_chain_of_incubated_objects.patch 2014-03-14 08:04:38 +0000
19@@ -0,0 +1,36 @@
20+--- qtdeclarative-opensource-src-5.2.1.orig/src/qml/qml/qqmlincubator.cpp
21++++ qtdeclarative-opensource-src-5.2.1/src/qml/qml/qqmlincubator.cpp
22+@@ -168,6 +168,16 @@ void QQmlIncubatorPrivate::clear()
23+ nextWaitingFor.remove();
24+ waitingOnMe = 0;
25+ }
26++
27++ // if we're waiting on any incubators then they should be cleared too.
28++ while (waitingFor.first()) {
29++ QQmlIncubator * i = static_cast<QQmlIncubatorPrivate*>(waitingFor.first())->q;
30++ if (i)
31++ i->clear();
32++ }
33++
34++ vme.reset();
35++ vmeGuard.clear();
36+ }
37+
38+ /*!
39+@@ -557,16 +567,6 @@ void QQmlIncubator::clear()
40+
41+ d->clear();
42+
43+- // if we're waiting on any incubators then they should be cleared too.
44+- while (d->waitingFor.first()) {
45+- QQmlIncubator * i = static_cast<QQmlIncubatorPrivate*>(d->waitingFor.first())->q;
46+- if (i)
47+- i->clear();
48+- }
49+-
50+- d->vme.reset();
51+- d->vmeGuard.clear();
52+-
53+ Q_ASSERT(d->compiledData == 0);
54+ Q_ASSERT(d->waitingOnMe.data() == 0);
55+ Q_ASSERT(d->waitingFor.isEmpty());
56
57=== modified file 'debian/patches/series'
58--- debian/patches/series 2014-03-07 18:06:52 +0000
59+++ debian/patches/series 2014-03-14 08:04:38 +0000
60@@ -11,3 +11,4 @@
61 Fix-crash-when-accessing-var-properties-in-objects-w.patch
62 aarch64.patch
63 ppc64el.patch
64+clean_chain_of_incubated_objects.patch

Subscribers

People subscribed via source and target branches