Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [platform-debug-dev] Debug Commands to move to core plug-in in 3.3 M5


Debuggers,

I've been working some more on this problem, so those that are interested should keep up with https://bugs.eclipse.org/bugs/show_bug.cgi?id=168434. The bug has two implementation approaches attached.

Rather than using the "result collector" pattern, it seems that a "request" and "request processor" pattern is more applicable, which could also be extended to the viewer API.

Thanks,

Darin Wright



Darin Wright/Ottawa/IBM@IBMCA
Sent by: platform-debug-dev-bounces@xxxxxxxxxxx

12/18/2006 12:30 PM

Please respond to
"Eclipse Platform Debug component developers list."        <platform-debug-dev@xxxxxxxxxxx>

To
platform-debug-dev@xxxxxxxxxxx, dsdp-dd-dev@xxxxxxxxxxx
cc
Subject
[platform-debug-dev] Debug Commands to move to core plug-in in 3.3        M5






Debug developers,


In 3.3 M3, the org.eclipse.debug.ui.commands API package was added to the debug ui plug-in to provide retargettable debug commands. These commands will be moved to the debug core plug-in during 3.3 M5 as they do not (need to) reference any UI interfaces. As well, since these command interfaces are replacements for the standard "command interfaces" (ITerminate, etc.), which live in the core, it makes sense for these commands to live in the core as well.


The original command API was based on the use of an IStatusMonitor to accept progress information and collect enabled state as a result. Other platform teams have suggested that we should separate progress reporting and result collection to be more consistent with the "result collector" pattern used in the platform and to avoid overriding the function of a progress monitor.


We will likely follow the same pattern in the asynchronous viewer implementation - that is, separate the progress monitor and result collector.


I've entered the following bug: https://bugs.eclipse.org/bugs/show_bug.cgi?id=168434


Sorry for the temporary API disruption.


Darin Wright
_______________________________________________
platform-debug-dev mailing list
platform-debug-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/platform-debug-dev


Back to the top