You could apply it and comment it or something.<br><br><div class="gmail_quote">On Fri, Jun 12, 2009 at 11:00 AM, David Nolden <span dir="ltr"><<a href="mailto:david.nolden.kdevelop@art-master.de">david.nolden.kdevelop@art-master.de</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Am Freitag 12 Juni 2009 18:19:24 schrieb Peter Oberndorfer:<br>
<div class="im">> Running valgrind on some duchain tests it seems Buckets are leaked in<br>
> ItemRepository::close()<br>
><br>
> Attached patch should fix this.<br>
><br>
> A question if i understood it correctly:<br>
> m_fastBuckets is just for being able to access buckets in m_buckets fast<br>
> without any function call.<br>
</div>Yes exactly.<br>
<br>
Actually I think this patch should better not be applied, even if it is<br>
correct. The buckets contain the actual data that is used by many different<br>
items, for example macros or identifiers. ItemRepository::close() is only<br>
called during application shutdow, and the buckets do not contain any<br>
meaningful destructors, thus it has no negative side-effects to let them leak.<br>
<br>
Data that is stored within such buckets is for example permanently held by<br>
KDevelop::Identifier and KDevelop::QualifiedIdentifier. Deleting the buckets<br>
would mean that from that point on, any use of an Identifier or<br>
QualifiedIdentifier could lead to a crash during shutdown, depending on the<br>
shutdown sequence.<br>
<br>
Since the shutdown sequence seems to be nearly random, and we've had quite a<br>
few such shutdown sequence problems already, it's probably better to just let<br>
this leak intentionally, and just add a comment describing that problem<br>
instead of the qDeleteAll.<br>
<br>
Greetings, David<br>
<br>
<br>
_______________________________________________<br>
KDevelop-devel mailing list<br>
<a href="mailto:KDevelop-devel@kdevelop.org">KDevelop-devel@kdevelop.org</a><br>
<a href="https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel" target="_blank">https://barney.cs.uni-potsdam.de/mailman/listinfo/kdevelop-devel</a><br>
</blockquote></div><br>