| Summary: | Remove All Terminated causes dancing scrollbar | ||
|---|---|---|---|
| Product: | [Eclipse Project] Platform | Reporter: | Peter Burka <peter_burka> |
| Component: | Debug | Assignee: | Darin Wright <darin.eclipse> |
| Status: | RESOLVED DUPLICATE | QA Contact: | |
| Severity: | enhancement | ||
| Priority: | P3 | CC: | jared_burns, jed.anderson |
| Version: | 2.0 | ||
| Target Milestone: | --- | ||
| Hardware: | PC | ||
| OS: | Windows XP | ||
| Whiteboard: | |||
|
Description
Peter Burka
Cool. And you'd like us to fix it? The problem is that we remove the launches from the viewer one at a time. This is because the LaunchManager's API only allows us to remove one launch at a time. We could add a method to remove multiple launches at once, because ILaunchManager is "not intended to be implemented by clients" but this wouldn't help us because the ILaunchListener API, which *is* intended to be implemented by clients only takes a single launch. I think the only way to work around this bug would be to add a new listener, IMultipleLaunchListener, which extends ILaunchListener and adds methods for multiple-launch notifications. We'd have to still support the old ILaunchListener, but we could deprecate it at migrate our code to the new listener. This would allow our UI to remove all of the launches in one operation. To avoid new API, we should investigate the action unhooking the view as a launch listener, removing the lanuches, and then re-hooking and refreshing the view. *** Bug 22907 has been marked as a duplicate of this bug. *** I believe the correct long term solution is introduce the new interface. This also means adding new methods to the launch manager. I am worried that there are other interfaces in the debug API that should take arrays of elements rather than single elements. The breakpoint, launch, and expression listener interfaces are all singular. |