[PATCH 3 of 3] (issue56) Do not delete the certificate immediately

Wald Commits scm-commit at wald.intevation.org
Mon Jul 7 12:54:08 CEST 2014


# HG changeset patch
# User Andre Heinecke <andre.heinecke at intevation.de>
# Date 1404730442 -7200
# Node ID 9bfaced5cf59531820b383fd23dce50b42cb51e8
# Parent  e93bbcf70a12444140459bdd1c8c4d41051fbfbf
(issue56) Do not delete the certificate immediately

    When the selection changed the remove cert is triggered by
    a slection change event in the certificateitem. Deleting
    it immediately would delete the trigger of the call.

    Instead we deleteLater so that the widget get's cleaned up
    in the next mainloop iteration when it is no longer needed.

diff -r e93bbcf70a12 -r 9bfaced5cf59 ui/certificatelistwidget.cpp
--- a/ui/certificatelistwidget.cpp	Mon Jul 07 12:45:06 2014 +0200
+++ b/ui/certificatelistwidget.cpp	Mon Jul 07 12:54:02 2014 +0200
@@ -43,7 +43,7 @@
     for (int i = 0; i < mCertificateWidgets.size(); i++) {
         if (mCertificateWidgets[i]->certificate() == cert) {
             mLayout.removeWidget(mCertificateWidgets[i]);
-            delete(mCertificateWidgets[i]);
+            mCertificateWidgets[i]->deleteLater();
             mCertificateWidgets.removeAt(i);
             break;
         }


More information about the Trustbridge-commits mailing list