From c58a08aaad2ad5eb69da91f1d6e14e26e75d513c Mon Sep 17 00:00:00 2001 From: Ken Ryall Date: Tue, 25 May 2010 19:44:06 +0000 Subject: [PATCH] Bug 309128. --- .../eclipse/cdt/launch/AbstractCLaunchDelegate.java | 8 ++++++++ .../eclipse/cdt/launch/AbstractCLaunchDelegate2.java | 10 +++++++++- 2 files changed, 17 insertions(+), 1 deletion(-) diff --git a/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate.java b/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate.java index 1a46a467cf5..5a14503e439 100644 --- a/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate.java +++ b/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate.java @@ -74,6 +74,14 @@ import org.eclipse.debug.ui.RefreshTab; import com.ibm.icu.text.DateFormat; import com.ibm.icu.text.MessageFormat; +/** + * AbstractCLaunchDelegate is the launch delegate used by most CDI based debuggers. + * It has been superseded by AbstractCLaunchDelegate2 which is used by most DSF based + * debuggers. AbstractCLaunchDelegate has been left unmodified because it is commonly + * used by CDT clients and contains lots of obscure code created long ago to handle + * issues whose relevance is unclear today. + * + */ abstract public class AbstractCLaunchDelegate extends LaunchConfigurationDelegate { /** diff --git a/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate2.java b/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate2.java index 4e96654a429..ced7b2e3cfc 100644 --- a/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate2.java +++ b/launch/org.eclipse.cdt.launch/src/org/eclipse/cdt/launch/AbstractCLaunchDelegate2.java @@ -37,7 +37,15 @@ import org.eclipse.debug.core.ILaunchConfiguration; import org.eclipse.debug.core.model.LaunchConfigurationDelegate; /** - * @author Ken Ryall + * AbstractCLaunchDelegate2 is used by most DSF based debuggers. It replaces AbstractCLaunchDelegate + * which is the launch delegate used by most CDI based debuggers. + * + * While it is technically possible to merge the two, AbstractCLaunchDelegate has been left + * unmodified because it is commonly used by CDT clients and contains lots of obscure code + * created long ago to handle issues whose relevance is unclear today. + * + */ +/** * @since 6.1 * */