<table><tr><td style="">sredman created this revision.<br />sredman added a reviewer: KDE Connect.<br />Herald added a project: KDE Connect.<br />Herald added a subscriber: kdeconnect.<br />sredman 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/D16553">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>After using the ConversationsDbusInterface for a little while, there can be significant (MBs) memory usage of cached messages. The QDBusAbstractAdaptor does not like to be manually deleted, but it looks like it is safe to do so after constructing a new one</p>

<p>This contradicts the comment in the BatteryPlugin and the BatteryDbusInterface, which says deletelater() is not safe. Has Qt been updated since then?</p></div></div><br /><div><strong>TEST PLAN</strong><div><ul class="remarkup-list">
<li class="remarkup-list-item">Run daemon</li>
<li class="remarkup-list-item">Hopefully experience no crashes after many phone reconnects</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R224 KDE Connect</div></div></div><br /><div><strong>BRANCH</strong><div><div>ConversationsDbusMemoryLeak</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D16553">https://phabricator.kde.org/D16553</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/sms/smsplugin.cpp<br />
plugins/sms/smsplugin.h</div></div></div><br /><div><strong>To: </strong>sredman, KDE Connect<br /><strong>Cc: </strong>kdeconnect, skymoore, wistak, dvalencia, rmenezes, julioc, Leptopoda, timothyc, jdvr, yannux, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, menasshock, tctara, apol<br /></div>