address-book-service

Merge lp:~renatofilho/address-book-service/fix-1518486 into lp:address-book-service

  1. fix-1518486
  2. Merge into trunk
Proposed by Renato Araujo Oliveira Filho
Status: Merged
Approved by: Arthur Mello
Approved revision: 220
Merged at revision: 223
Proposed branch: lp:~renatofilho/address-book-service/fix-1518486
Merge into: lp:address-book-service
Diff against target: 51 lines (+5/-4)
2 files modified
lib/dirtycontact-notify.cpp (+3/-3)
lib/dirtycontact-notify.h (+2/-1)
To merge this branch: bzr merge lp:~renatofilho/address-book-service/fix-1518486
Reviewer Review Type Date Requested Status
Arthur Mello (community) Approve
system-apps-ci-bot continuous-integration Approve
Review via email: mp+306327@code.launchpad.net

Commit message

Check if adaptor was destroyed before use it.

To post a comment you must log in.
Revision history for this message
system-apps-ci-bot (system-apps-ci-bot) wrote :

PASSED: Continuous integration, rev:220
https://jenkins.canonical.com/system-apps/job/lp-address-book-service-ci/14/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build/1589
    SUCCESS: https://jenkins.canonical.com/system-apps/job/test-0-autopkgtest/label=phone-armhf,release=vivid+overlay,testname=default/377
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-0-fetch/1589
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=vivid+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=xenial+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=amd64,release=yakkety/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=vivid+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=xenial+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=armhf,release=yakkety/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=vivid+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=xenial+overlay/1437/artifact/output/*zip*/output.zip
    SUCCESS: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1437
        deb: https://jenkins.canonical.com/system-apps/job/build-2-binpkg/arch=i386,release=yakkety/1437/artifact/output/*zip*/output.zip

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-apps/job/lp-address-book-service-ci/14/rebuild

review: Approve (continuous-integration)
Revision history for this message
Arthur Mello (artmello) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lib/dirtycontact-notify.cpp'
2--- lib/dirtycontact-notify.cpp 2015-12-08 15:46:17 +0000
3+++ lib/dirtycontact-notify.cpp 2016-09-21 12:19:17 +0000
4@@ -36,7 +36,7 @@
5
6 void DirtyContactsNotify::insertAddedContacts(QSet<QString> ids)
7 {
8- if (!m_adaptor->isReady()) {
9+ if (!m_adaptor || !m_adaptor->isReady()) {
10 return;
11 }
12
13@@ -71,7 +71,7 @@
14
15 void DirtyContactsNotify::insertRemovedContacts(QSet<QString> ids)
16 {
17- if (!m_adaptor->isReady()) {
18+ if (!m_adaptor || !m_adaptor->isReady()) {
19 return;
20 }
21
22@@ -90,7 +90,7 @@
23
24 void DirtyContactsNotify::insertChangedContacts(QSet<QString> ids)
25 {
26- if (!m_adaptor->isReady()) {
27+ if (!m_adaptor || !m_adaptor->isReady()) {
28 return;
29 }
30
31
32=== modified file 'lib/dirtycontact-notify.h'
33--- lib/dirtycontact-notify.h 2015-09-17 13:39:44 +0000
34+++ lib/dirtycontact-notify.h 2016-09-21 12:19:17 +0000
35@@ -23,6 +23,7 @@
36 #include <QtCore/QTimer>
37 #include <QtCore/QSet>
38 #include <QtCore/QString>
39+#include <QtCore/QPointer>
40
41 namespace galera {
42
43@@ -48,7 +49,7 @@
44 void emitSignals();
45
46 private:
47- AddressBookAdaptor *m_adaptor;
48+ QPointer<AddressBookAdaptor> m_adaptor;
49 QTimer m_timer;
50 QSet<QString> m_contactsChanged;
51 QSet<QString> m_contactsAdded;

Subscribers

People subscribed via source and target branches