Skip to main content

[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.



Back to the top