[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
Re: [cdt-patch] Mainline Patch for PR 61965/63990 (scalability)
|
>
> Folks,
>
> This patch addresses the scalability issues discussed (with the test
> driver) in PR 61965 and also happens to fix the NPE in PR 63990. This
> patch hits a fair number of files in order to properly dispose of
> listeners and resources which were limiting the C/C++ Editor to only
> opening and closing a file repeatedly ~500 times before running out
> of memory. JUnit tests run clean, but component owners should check
> the update since the following classes were touched:
>
> CPluginImages,MouseClickListener,CEditorErrorTickUpdater,
> CEditorActionContribution, CContentOutlinePage, CEditor,
> SelectionSearchGroup, MemberFilterActionGroup, RefactoringActionGroup
>
> An incidental upside is that we also use significantly less memory
> (~5-10M) on a single editor isntance opening now then we did before.
>
> For the ChangeLog:
> - Properly detach listeners and dispose of components to address
> resource and memory leaks from CEditor stress test (PR 61965)
>
Done. Many thanks for the nice work.