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

[Thomas Fletcher <ThomasF@xxxxxxx>]

Proof is in the pudding:  Overnight run on mainline head with
this patch cycled 200 000+ times stable with no memory growth.

>-----Original Message-----
>From: Thomas Fletcher [mailto:ThomasF@xxxxxxx]
>Sent: May 25, 2004 5:47 PM
>To: 'cdt-patch@xxxxxxxxxxx'
>Subject: [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)
>
>