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

[John Camelon <jcamelon@xxxxxxxxxx>]

I need Thomas' patch ASAP as I am in
profile hell right now and can use all the help I can get. 

JohnC
www.eclipse.org/cdt

cdt-patch-admin@xxxxxxxxxxx wrote on 05/26/2004 09:34:12
AM:

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