<table><tr><td style="">mtijink created this revision.<br />mtijink added a reviewer: KDE Connect.<br />mtijink requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11438">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Previously commit <a href="https://phabricator.kde.org/R224:895a9206ffafd4b9725f567fd069fd655f200139" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">895a9206ffafd4b9725</a> tried to fix the issues of<br />
already-freed notifications by using a QPointer. This only sets the<br />
pointer to null when deleted. Thus, commit <a href="https://phabricator.kde.org/R224:6495e8dc538610fb679df0f1522243103a3a1789" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">6495e8dc538610fb67</a> was needed<br />
too.</p>

<p>I tried to fix this here with a shared pointer, hopefully fixing bug 389518.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>Receiving notifications still works fine for me.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>notifications-shared-ptrs</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11438">https://phabricator.kde.org/D11438</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/notifications/notification.h<br />
plugins/notifications/notificationsdbusinterface.cpp<br />
plugins/notifications/notificationsdbusinterface.h</div></div></div><br /><div><strong>To: </strong>mtijink, KDE Connect<br /><strong>Cc: </strong>KDE Connect<br /></div>