diff options
author | Roland Grunberg | 2012-11-08 15:18:41 +0000 |
---|---|---|
committer | Camilo Bernal | 2012-11-08 18:38:40 +0000 |
commit | 9eef02e6fee02cd2f28aaae14d9eaff38fc524a1 (patch) | |
tree | e9edb6c841bb948b6ede746d92eebb6bd5f18b16 | |
parent | 60be5ab5b5c7a27b8a8a4d96f86ef3e84fdac16b (diff) | |
download | org.eclipse.linuxtools-9eef02e6fee02cd2f28aaae14d9eaff38fc524a1.tar.gz org.eclipse.linuxtools-9eef02e6fee02cd2f28aaae14d9eaff38fc524a1.tar.xz org.eclipse.linuxtools-9eef02e6fee02cd2f28aaae14d9eaff38fc524a1.zip |
Bug 393886: ProviderLaunchConfigurationDelegate should throw CoreException.
The CoreException that a launch configuration delegate may throw should
be handled by the underlying framework.
Change-Id: Ia2eb8e8cb4a54b665ce1532b93a17b5299e5267b
Reviewed-on: https://git.eclipse.org/r/8606
Tested-by: Hudson CI
Reviewed-by: Camilo Bernal <cabernal@redhat.com>
IP-Clean: Camilo Bernal <cabernal@redhat.com>
Tested-by: Camilo Bernal <cabernal@redhat.com>
-rw-r--r-- | profiling/org.eclipse.linuxtools.profiling.launch/src/org/eclipse/linuxtools/internal/profiling/launch/provider/launch/ProviderLaunchConfigurationDelegate.java | 26 |
1 files changed, 11 insertions, 15 deletions
diff --git a/profiling/org.eclipse.linuxtools.profiling.launch/src/org/eclipse/linuxtools/internal/profiling/launch/provider/launch/ProviderLaunchConfigurationDelegate.java b/profiling/org.eclipse.linuxtools.profiling.launch/src/org/eclipse/linuxtools/internal/profiling/launch/provider/launch/ProviderLaunchConfigurationDelegate.java index cc69811a7e..5de6677137 100644 --- a/profiling/org.eclipse.linuxtools.profiling.launch/src/org/eclipse/linuxtools/internal/profiling/launch/provider/launch/ProviderLaunchConfigurationDelegate.java +++ b/profiling/org.eclipse.linuxtools.profiling.launch/src/org/eclipse/linuxtools/internal/profiling/launch/provider/launch/ProviderLaunchConfigurationDelegate.java @@ -28,26 +28,22 @@ public class ProviderLaunchConfigurationDelegate extends @Override public void launch(ILaunchConfiguration config, String mode, - ILaunch launch, IProgressMonitor monitor) { - try { + ILaunch launch, IProgressMonitor monitor) throws CoreException { - if (config != null) { - // get provider id from configuration. - String providerId = config.getAttribute( - ProviderProfileConstants.PROVIDER_CONFIG_ATT, ""); + if (config != null) { + // get provider id from configuration. + String providerId = config.getAttribute( + ProviderProfileConstants.PROVIDER_CONFIG_ATT, ""); - // get delegate associated with provider id. - ProfileLaunchConfigurationDelegate delegate = ProviderFramework.getConfigurationDelegateFromId(providerId); + // get delegate associated with provider id. + ProfileLaunchConfigurationDelegate delegate = ProviderFramework + .getConfigurationDelegateFromId(providerId); - // launch delegate - if (delegate != null) { - delegate.launch(config, mode, launch, monitor); - } + // launch delegate + if (delegate != null) { + delegate.launch(config, mode, launch, monitor); } - } catch (CoreException e) { - e.printStackTrace(); } - return; } /** |