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

[Douglas Schaefer <dschaefe@xxxxxxxxxx>]

Wicked. So I assume we want to get this patch into M9 (and probably 
Chris's patch)?

Doug Schaefer, IBM's Eclipse CDT Architect
Ottawa (Palladium), Ontario, Canada



Thomas Fletcher <ThomasF@xxxxxxx> 
Sent by: cdt-patch-admin@xxxxxxxxxxx
05/26/2004 08:13 AM
Please respond to
cdt-patch


To
"'cdt-patch@xxxxxxxxxxx'" <cdt-patch@xxxxxxxxxxx>
cc

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






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)
>
>
_______________________________________________
cdt-patch mailing list
cdt-patch@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/cdt-patch