Re: [cdt-patch] Mainline Patch for PR 61965/63990 (scalability)

[John Camelon <jcamelon@xxxxxxxxxx>]

+1 wrt our Groups (Rename & SelectionSearch).
 
Thanks again Thomas!

JohnC
www.eclipse.org/cdt

cdt-patch-admin@xxxxxxxxxxx wrote on 05/25/2004 05:47:17
PM:

> 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)
> 
> [attachment "CEditorScalabilityPatch.txt" deleted by John
Camelon/Ottawa/IBM]