Bug 102619 - Performance problem refreshing tree views in CElementContentProvider (patch)
Summary: Performance problem refreshing tree views in CElementContentProvider (patch)
Status: RESOLVED FIXED
Alias: None
Product: CDT
Classification: Tools
Component: cdt-core (show other bugs)
Version: 3.0   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.0 RC2   Edit
Assignee: Alain Magloire CLA
QA Contact:
URL:
Whiteboard:
Keywords: contributed
Depends on:
Blocks: 108205
  Show dependency tree
 
Reported: 2005-07-03 22:04 EDT by Robert O'Callahan CLA
Modified: 2008-06-19 13:05 EDT (History)
0 users

See Also:

Attachments
fix (6.45 KB, patch)
2005-07-03 22:06 EDT, Robert O'Callahan CLA 		bjorn.freeman-benson: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Robert O'Callahan CLA 2005-07-03 22:04:51 EDT 
Sometimes while working with my huge project, Eclipse hogs the CPU for a while
(seconds to minutes), while remaining responsive. The progress window doesn't
show anything happening. I stopped it with the debugger and found that it was
processing thousands of async UI updates dispatched by CElementContentProvider
to refresh the tree view. Many of these messages (50%?) were targeted at the
entire project.

I've fixed this bug by having CElementContentProvider remember the set of
refresh targets it currently has async UI updates pending for. If there's
already an update scheduled against an element, we don't post it again. It seems
to work well.
Comment 1 Robert O'Callahan CLA 2005-07-03 22:06:05 EDT 
Created attachment 24313 [details]
fix

Fix as described.
Comment 2 Alain Magloire CLA 2005-07-06 10:43:04 EDT 
Sounds like a nice performace improvement.
Comment 3 Alain Magloire CLA 2005-07-06 10:43:37 EDT 
Patch apply to the head,
Please verify.

Thanks.