Summary: | PDE source lookup director creates source container for each request | ||||||
---|---|---|---|---|---|---|---|
Product: | [Eclipse Project] PDE | Reporter: | Darin Wright <darin.eclipse> | ||||
Component: | UI | Assignee: | Darin Wright <darin.eclipse> | ||||
Status: | RESOLVED FIXED | QA Contact: | |||||
Severity: | normal | ||||||
Priority: | P3 | CC: | agarcher, baumanbr, caniszczyk, Curtis_Windatt, Michael_Rennie, wassim.melhem | ||||
Version: | 3.3 | Keywords: | bugday, performance | ||||
Target Milestone: | 3.4 M2 | Flags: | baumanbr:
review+
|
||||
Hardware: | PC | ||||||
OS: | Windows XP | ||||||
Whiteboard: | |||||||
Attachments: |
|
Description
Darin Wright
2007-04-12 17:54:31 EDT
This could be a good bug to get some new feet wet with :) ok, sweet. first one to get the patch wins the race. Your picture will go on our famous PDE committer/contributor webpage. http://www.eclipse.org/pde/pde-ui/committers/committers.php gogogogogo Created attachment 77924 [details]
patch
Adds a cache of source containers to the PDE source lookup director. The source lookup query retrieves source containers from the director rather than building them on each request. Containers are disposed when the director is disposed (when the launch is removed). Containers are added to the cache lazily - the first time a container is required for a location/bundle.
Makes moving between stack frames (source lookup) in different bundles more snappy.
just had to beat me to it :) not axesome That was quick! Debug doesn't mess around. That's what I'm talking about, bug reporters who actually solve the bug they report ;) Darin, looks very good, not to mention the quick turn around. I like the custom key object :) Patch applied as-is to HEAD. Will be available in 3.4M2. Admitedly - I had an unfair advantage here since it was an implementation of an interface that debug defines. However, it's good practice to eat your own dog food. The PDE source lookup director is interesting in that it performs a dynamic source lookup based on bundles, rather than searching along a path for source (i.e. like a typical classpath). Now we need a picture of Darin featuring a "leafy background" to put him on PDE's hall of fame webpage. (In reply to comment #9) > Now we need a picture of Darin featuring a "leafy background" to put him on > PDE's hall of fame webpage. Actually, if you look at his sametime picture, you will notice that it promotes the PDE trademarked "leafy background". He was clearly born ready. I am not sure this bug challenged Darin to step out of his comfort zone. It's an optimization of a source lookup director that is based on the debug framework. Can we have him write a form-based editor or something? :) |