Attachment #167248 for bug #310304

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.




package org.eclipse.cdt.debug.core;

import java.io.ByteArrayOutputStream;
import java.io.FileNotFoundException;
import java.io.IOException;
import java.nio.charset.Charset;
import java.nio.charset.CharsetDecoder;
import com.ibm.icu.text.MessageFormat;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Iterator;

import org.eclipse.cdt.debug.core.model.ICWatchpoint;
import org.eclipse.cdt.debug.core.model.ICWatchpoint2;
import org.eclipse.cdt.debug.internal.core.model.CFloatingPointValue;
import org.eclipse.core.resources.IFile;
import org.eclipse.core.resources.IMarker;
import org.eclipse.core.resources.IProject;
import org.eclipse.core.resources.ResourcesPlugin;

import org.eclipse.core.runtime.IPath;
import org.eclipse.core.runtime.IStatus;
import org.eclipse.core.runtime.Path;
import org.eclipse.core.runtime.Status;
import org.eclipse.debug.core.DebugPlugin;
import org.eclipse.debug.core.ILaunchConfiguration;
import org.eclipse.debug.core.IStatusHandler;
import org.eclipse.debug.core.model.IBreakpoint;
import org.w3c.dom.Document;

import com.ibm.icu.text.MessageFormat;

/**
 * Utility methods.
 */

        }
        return new Path(path);
    }

	/**
	 * Returns the ICProject associated with the project setting in the Main tab
	 * of a CDT launch configuration, or throws a CoreException providing a
	 * reason (e.g., the setting is empty, the project no longer exists, the
	 * isn't a CDT one, etc).
	 * 
	 * @param config
	 *            the launch configuration
	 * @return an ICProject; never null.
	 * @throws CoreException
	 * @since 7.0
	 */
	public static ICProject verifyCProject(ILaunchConfiguration config) throws CoreException {
		String name = CDebugUtils.getProjectName(config);
		if (name == null) {
			throwCoreException(DebugCoreMessages.getString("CDebugUtils.C_Project_not_specified"),  //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROJECT);
		}
		ICProject cproject = CDebugUtils.getCProject(config);
		if (cproject == null) {
			IProject proj = ResourcesPlugin.getWorkspace().getRoot().getProject(name);
			if (!proj.exists()) {
				throwCoreException(DebugCoreMessages.getFormattedString("CDebugUtils.Project_NAME_does_not_exist", name),  //$NON-NLS-1$
						ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
			} else if (!proj.isOpen()) {
				throwCoreException(DebugCoreMessages.getFormattedString("CDebugUtils.Project_NAME_is_closed", name), //$NON-NLS-1$
						ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
			}
			throwCoreException(DebugCoreMessages.getString("CDebugUtils.Not_a_C_CPP_project"),  //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
		}
		return cproject;
	}

	/**
	 * Returns an IPath for the file referenced in the <i>C/C++ Application</i>
	 * setting of a CDT launch configuration. Typically, the file is obtained by
	 * combining the <i>C/C++ Application</i> setting with the <i>Project</i>
	 * setting. If unable to combine and resolve these settings to a valid file,
	 * a CoreException is thrown that provides the reason why. There are many
	 * such possible reasons (a problem with the <i>Project</i> setting, an
	 * empty <i>C/C++ Application</i> setting, the combined settings doesn't
	 * resolve to an existing file, etc).
	 * 
	 * @param config
	 *            the launch configuration
	 * @param ignoreProjectSetting
	 *            if true, resolve the file using only the <i>C/C++
	 *            Application</i> setting. Do not take the <i>Project</i>
	 *            setting into account.
	 * @return an IPath; never null
	 * @throws CoreException
	 * @since 7.0
	 */
	public static IPath verifyProgramPath(ILaunchConfiguration config, boolean ignoreProjectSetting) throws CoreException {
		ICProject cproject = null;
		if (!ignoreProjectSetting) {
			cproject = verifyCProject(config);	// will throw exception if project setting not valid
		}
		IPath programPath = CDebugUtils.getProgramPath(config);
		if (programPath == null || programPath.isEmpty()) {
			throwCoreException(DebugCoreMessages.getString("CDebugUtils.Program_file_not_specified"), //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROGRAM);
		}
		
		if (programPath != null) {	// this check is here only to avoid warning; compiler can't tell we'll throw an exception above
			if (!programPath.isAbsolute() && (cproject != null)) {
				// See if we can brute-force append the program path to the
				// project location. This allows us to support the program file
				// being outside the project, even outside the workspace, without
				// requiring a linked resource (e.g., the setting could be 
				// "..\..\some\dir\myprogram.exe")
				IPath location = cproject.getProject().getLocation();
				if (location != null) {
					programPath = location.append(programPath);
					if (!programPath.toFile().exists()) {
						// Try looking in the project for the file. This
						// supports linked resources.
						IFile projFile = null;
						try {
							projFile = cproject.getProject().getFile(CDebugUtils.getProgramPath(config));
						}
						catch (IllegalArgumentException exc) {
							// thrown if relative path that resolves to a root file (e.g., "..\somefile")							
						}	
						if (projFile != null && projFile.exists()) {
							programPath = projFile.getLocation();
						}
					}
				}
			}
			if (!programPath.toFile().exists()) {
				throwCoreException(
						DebugCoreMessages.getString("CDebugUtils.Program_file_does_not_exist"), //$NON-NLS-1$
						new FileNotFoundException(
								DebugCoreMessages.getFormattedString("CDebugUtils.PROGRAM_PATH_not_found", programPath.toOSString())), //$NON-NLS-1$
						ICDTLaunchConfigurationConstants.ERR_PROGRAM_NOT_EXIST);
			}
		}
		return programPath;
	}

	/**
	 * Variant that expects (requires) the launch configuration to have a valid
	 * <i>Project</i> setting. See
	 * {@link #verifyProgramPath(ILaunchConfiguration, boolean)}
	 * 
	 * @since 7.0
	 */
	public static IPath verifyProgramPath(ILaunchConfiguration config) throws CoreException {
		return verifyProgramPath(config, false); 
	}
	
	/** Throws a CoreException. Clutter-reducing utility method. */
	private static void throwCoreException(String msg, int code) throws CoreException {
		throwCoreException(msg, null, code);
	}

	/** Throws a CoreException. Clutter-reducing utility method. */
	private static void throwCoreException(String msg, Exception innerException, int code) throws CoreException {
		throw new CoreException(new Status(IStatus.ERROR, CDebugCorePlugin.getUniqueIdentifier(), code, msg, innerException));
	}
}

Lines 13-18 Link Here

(-)src/org/eclipse/cdt/debug/core/DebugCoreMessages.java (+10 lines)
13	import java.util.MissingResourceException;	13	import java.util.MissingResourceException;
14	import java.util.ResourceBundle;	14	import java.util.ResourceBundle;
15		15
		16	import com.ibm.icu.text.MessageFormat;
		17
16	public class DebugCoreMessages {	18	public class DebugCoreMessages {
17		19
18	private static final String BUNDLE_NAME = "org.eclipse.cdt.debug.core.DebugCoreMessages";//$NON-NLS-1$	20	private static final String BUNDLE_NAME = "org.eclipse.cdt.debug.core.DebugCoreMessages";//$NON-NLS-1$
Lines 22-27 Link Here
22	private DebugCoreMessages() {	24	private DebugCoreMessages() {
23	}	25	}
24		26
		27	static String getFormattedString(String key, String arg) {
		28	return MessageFormat.format(getString(key), new String[]{arg});
		29	}
		30
		31	static String getFormattedString(String key, String[] args) {
		32	return MessageFormat.format(getString(key), args);
		33	}
		34
25	public static String getString( String key ) {	35	public static String getString( String key ) {
26	try {	36	try {
27	return RESOURCE_BUNDLE.getString( key );	37	return RESOURCE_BUNDLE.getString( key );

Lines 23-26 Link Here

(-)src/org/eclipse/cdt/debug/core/DebugCoreMessages.properties (+8 lines)
23	CDebugUtils.Hardware=Hardware	23	CDebugUtils.Hardware=Hardware
24	CDebugUtils.Temporary=Temporary	24	CDebugUtils.Temporary=Temporary
25	CDebugUtils.Software=Software	25	CDebugUtils.Software=Software
		26	CDebugUtils.C_Project_not_specified=A project was not specified in the launch configuration.
		27	CDebugUtils.Not_a_C_CPP_project=The project specified in the launch configuration is not a C/C++ one.
		28	CDebugUtils.PROGRAM_PATH_not_found={0} not found
		29	CDebugUtils.Program_file_does_not_exist=The program file specified in the launch configuration does not exist
		30	CDebugUtils.Program_file_not_specified=A program file was not specified in the launch configuration.
		31	CDebugUtils.Project_NAME_does_not_exist=Project {0} does not exist. Please check that your launch configuration specifies a valid project in your workspace.
		32	CDebugUtils.Project_NAME_is_closed=Project {0} is closed
		33
26	CDIDebugModel.0=Unable to parser binary information from file	34	CDIDebugModel.0=Unable to parser binary information from file




import java.util.List;
import java.util.Properties;

import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContribution;
import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContributionFactory;
import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.IGDBJtagDevice;

            // Below steps are specific to JTag hardware debugging
            
        /*
         * Retrieve the IGDBJtagDevice instance
         */
        new Step() {
			@Override

				rm.done();
            }},
        /*
         * Execute symbol loading
         */
        new Step() {
			@Override
			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS)) {
						String symbolsFileName = null;

						// New setting in Helios. Default is true. Check for existence
						// in order to support older launch configs
						if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS) &&
								config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS)) {
							IPath programFile = CDebugUtils.verifyProgramPath(config);
							if (programFile != null) {
								symbolsFileName = programFile.toOSString();
							}
						}
						else {
							symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, IGDBJtagConstants.DEFAULT_SYMBOLS_FILE_NAME);
							if (symbolsFileName.length() > 0) {
								symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName);
							} else {
								symbolsFileName = null;
							}
						}
						
						if (symbolsFileName == null) {
		        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, Messages.getString("GDBJtagDebugger.err_no_img_file"), null)); //$NON-NLS-1$
		        			rm.done();
		        			return;
						}

						// Escape windows path separator characters TWICE, once for Java and once for GDB.						
						symbolsFileName = symbolsFileName.replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$

						String symbolsOffset = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, IGDBJtagConstants.DEFAULT_SYMBOLS_OFFSET);
						if (symbolsOffset.length() > 0) {
							symbolsOffset = "0x" + symbolsOffset;					
						}
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
						queueCommands(commands, rm);									
						
					} else {
						rm.done();
					}
				} catch (CoreException e) {
        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot load symbol", e)); //$NON-NLS-1$
        			rm.done();
				}
			}},

        /*
         * Hook up to remote target
         */
        new Step() {

			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_DO_RESET, IGDBJtagConstants.DEFAULT_DO_RESET)) {
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doReset(commands);
						queueCommands(commands, rm);

			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT)) {
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doHalt(commands);
						queueCommands(commands, rm);								

			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, IGDBJtagConstants.DEFAULT_INIT_COMMANDS);
					userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);
					if (userCmd.length() > 0) {
						String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$
						
						CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);
						crm.setDoneCount(commands.length);
						for (int i = 0; i < commands.length; ++i) {
							fCommandControl.queueCommand(
							    new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),
							    new DataRequestMonitor<MIInfo>(getExecutor(), crm));
						}
					}
					else {
						rm.done();
					}
				} catch (CoreException e) {
        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined init commands", e)); //$NON-NLS-1$

			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					String imageFileName = null;
					if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE)) {
						// New setting in Helios. Default is true. Check for existence
						// in order to support older launch configs
						if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE) &&
								config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_IMAGE)) {
							IPath programFile = CDebugUtils.verifyProgramPath(config);
							if (programFile != null) {
								imageFileName = programFile.toOSString();
							}
						} else {
							rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Image name cannot be empty", null)); //$NON-NLS-1$
							rm.done();
						}
						else {
							imageFileName = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, IGDBJtagConstants.DEFAULT_IMAGE_FILE_NAME); //$NON-NLS-1$
							if (imageFileName.length() > 0) {
								imageFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(imageFileName);
							} else {
								imageFileName = null;
							}
						}
						
						if (imageFileName == null) {
		        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, Messages.getString("GDBJtagDebugger.err_no_img_file"), null)); //$NON-NLS-1$
		        			rm.done();
		        			return;
						}

						// Escape windows path separator characters TWICE, once for Java and once for GDB.						
						imageFileName = imageFileName.replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$

						String imageOffset = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET);
						if (imageOffset.length() > 0) {
							imageOffset = (imageFileName.endsWith(".elf")) ? "" : "0x" + config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET); //$NON-NLS-2$ //$NON-NLS-4$					
						}
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doLoadImage(imageFileName, imageOffset, commands);
						queueCommands(commands, rm);									
					} 
					else {
						rm.done();
					}
				} catch (CoreException e) {

        			rm.done();
				}
			}},
        /*
         * Execute symbol loading
         */
        new Step() {
			@Override
			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS)) {
						// Escape windows path separator characters TWICE, once for Java and once for GDB.
						String symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, ""); //$NON-NLS-1$
						if (symbolsFileName.length() > 0) {
							symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName).replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$
							String symbolsOffset = "0x" + config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, ""); //$NON-NLS-1$ //$NON-NLS-2$
							List<String> commands = new ArrayList<String>();
							fGdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
							queueCommands(commands, rm);									
						} else {
							rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Symbol name cannot be empty", null)); //$NON-NLS-1$
							rm.done();
						}
					} else {
						rm.done();
					}
				} catch (CoreException e) {
        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot load symbol", e)); //$NON-NLS-1$
        			rm.done();
				}
			}
        },
        /* 
         * Start tracking the breakpoints once we know we are connected to the target (necessary for remote debugging) 
         */

				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER)) {
						String pcRegister = config.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_PC_REGISTER)); //$NON-NLS-1$
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doSetPC(pcRegister, commands);
						queueCommands(commands, rm);								

				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					if (config.getAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT)) {
						String stopAt = config.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, IGDBJtagConstants.DEFAULT_STOP_AT); //$NON-NLS-1$
						List<String> commands = new ArrayList<String>();
						fGdbJtagDevice.doStopAt(stopAt, commands);
						queueCommands(commands, rm);								

			public void execute(RequestMonitor rm) {
				ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
				try {
					String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, IGDBJtagConstants.DEFAULT_RUN_COMMANDS); //$NON-NLS-1$
					if (userCmd.length() > 0) {
						userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);
						String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$
						
						CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);
						crm.setDoneCount(commands.length);
						for (int i = 0; i < commands.length; ++i) {
							fCommandControl.queueCommand(
							    new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),
							    new DataRequestMonitor<MIInfo>(getExecutor(), crm));
						}
					}
					else {
						rm.done();
					}
				} catch (CoreException e) {
        			rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined run commands", e)); //$NON-NLS-1$

	private IGDBJtagDevice getGDBJtagDevice (ILaunchConfiguration config) 
	throws CoreException, NullPointerException {
		IGDBJtagDevice gdbJtagDevice = null;
		String jtagDeviceName = config.getAttribute(IGDBJtagConstants.ATTR_JTAG_DEVICE, IGDBJtagConstants.DEFAULT_JTAG_DEVICE); //$NON-NLS-1$
		GDBJtagDeviceContribution[] availableDevices = GDBJtagDeviceContributionFactory.getInstance().getGDBJtagDeviceContribution();
		for (GDBJtagDeviceContribution availableDevice : availableDevices) {
			if (jtagDeviceName.equals(availableDevice.getDeviceName())) {
				gdbJtagDevice = availableDevice.getDevice();
				gdbJtagDevice = availableDevices[i].getDevice();
				break;
			}
		}




/*******************************************************************************
 * Copyright (c) 2007 - 2010 QNX Software Systems and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at

import java.io.File;
import java.util.ArrayList;
import java.util.List;

import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;
import org.eclipse.cdt.debug.core.CDebugCorePlugin;
import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;
import org.eclipse.cdt.debug.core.cdi.ICDISession;
import org.eclipse.cdt.debug.core.cdi.model.ICDITarget;

import org.eclipse.cdt.debug.mi.core.command.MIGDBSetNewConsole;
import org.eclipse.cdt.debug.mi.core.output.MIInfo;
import org.eclipse.core.runtime.CoreException;
import org.eclipse.core.runtime.IPath;
import org.eclipse.core.runtime.IProgressMonitor;
import org.eclipse.core.runtime.IStatus;
import org.eclipse.core.runtime.MultiStatus;


			List<String> commands = new ArrayList<String>();

			// hook up to remote target
			if (submonitor.isCanceled()) {
				throw new OperationCanceledException();
			}
			
			// execute symbol load
			boolean doLoadSymbols = config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS);
			if (doLoadSymbols) {
				String symbolsFileName = null;
				// New setting in Helios. Default is true. Check for existence
				// in order to support older launch configs
				if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS) &&
						config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS)) {
					IPath programFile = CDebugUtils.verifyProgramPath(config);
					if (programFile != null) {
						symbolsFileName = programFile.toOSString();
					}
				}
				else {
					symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, IGDBJtagConstants.DEFAULT_SYMBOLS_FILE_NAME);
					if (symbolsFileName.length() > 0) {
						symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName);
					}
				}
				if (symbolsFileName == null) {
					// The launch config GUI should prevent this from happening, but just in case					
					throw new CoreException(new Status( IStatus.ERROR,
							Activator.getUniqueIdentifier(),
							-1, Messages.getString("GDBJtagDebugger.err_no_sym_file"), null));
				}

				// Escape windows path separator characters TWICE, once for Java and once for GDB.
				symbolsFileName = symbolsFileName.replace("\\", "\\\\");

				String symbolsOffset = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, IGDBJtagConstants.DEFAULT_SYMBOLS_OFFSET);
				if (symbolsOffset.length() > 0) {
					symbolsOffset = "0x" + symbolsOffset;					
				}
				commands.clear();
				gdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
				monitor.beginTask(Messages.getString("GDBJtagDebugger.loading_symbols"), 1); //$NON-NLS-1$				
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(15));
			}

			if (submonitor.isCanceled()) {
				throw new OperationCanceledException();
			}

			// hook up to remote target
			boolean useRemote = config.getAttribute(IGDBJtagConstants.ATTR_USE_REMOTE_TARGET, IGDBJtagConstants.DEFAULT_USE_REMOTE_TARGET);
			if (useRemote) {
				submonitor.subTask(Messages.getString("GDBJtagDebugger.2")); //$NON-NLS-1$
				String ipAddress = config.getAttribute(IGDBJtagConstants.ATTR_IP_ADDRESS, IGDBJtagConstants.DEFAULT_IP_ADDRESS);
				int portNumber = config.getAttribute(IGDBJtagConstants.ATTR_PORT_NUMBER, IGDBJtagConstants.DEFAULT_PORT_NUMBER);
				commands.clear();
				gdbJtagDevice.doRemote(ipAddress, portNumber, commands);
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(10));
				if (submonitor.isCanceled()) {

			submonitor.setWorkRemaining(80); // compensate for optional work above

			// Run device-specific code to reset the board
			if (config.getAttribute(IGDBJtagConstants.ATTR_DO_RESET, IGDBJtagConstants.DEFAULT_DO_RESET)) {
				commands.clear();
				gdbJtagDevice.doReset(commands);
				int defaultDelay = gdbJtagDevice.getDefaultDelay();

			submonitor.setWorkRemaining(65); // compensate for optional work above

			// Run device-specific code to halt the board
			if (config.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT)) {
				commands.clear();
				gdbJtagDevice.doHalt(commands);
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(15));

			// execute load
			boolean doLoad = config.getAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE);
			if (doLoad) {
				String imageFileName = null;

				// New setting in Helios. Default is true. Check for existence
				// in order to support older launch configs
				if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE) &&
						config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_IMAGE)) {
					IPath programFile = CDebugUtils.verifyProgramPath(config);
					if (programFile != null) {
						imageFileName = programFile.toOSString();
					}
				}
				else {
					imageFileName = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, IGDBJtagConstants.DEFAULT_IMAGE_FILE_NAME);
					if (imageFileName.length() > 0) {
						imageFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(imageFileName);
					}
				}
				if (imageFileName == null) {
					// The launch config GUI should prevent this from happening, but just in case
					throw new CoreException(new Status( IStatus.ERROR,
							Activator.getUniqueIdentifier(),
							-1, Messages.getString("GDBJtagDebugger.err_no_img_file"), null));
				}

				// Escape windows path separator characters TWICE, once for Java and once for GDB.
				imageFileName = imageFileName.replace("\\", "\\\\");

				String imageOffset = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET);
				if (imageOffset.length() > 0) {
					imageOffset = (imageFileName.endsWith(".elf")) ? "" : "0x" + config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET);					
					commands.clear();
					gdbJtagDevice.doLoadImage(imageFileName, imageOffset, commands);
					executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(20));
				}
				
				commands.clear();
				gdbJtagDevice.doLoadImage(imageFileName, imageOffset, commands);
				monitor.beginTask(Messages.getString("GDBJtagDebugger.loading_image"), 1); //$NON-NLS-1$
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(20));
			}
			submonitor.setWorkRemaining(15); // compensate for optional work above

			// execute symbol load
			boolean doLoadSymbols = config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS);
			if (doLoadSymbols) {
				String symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, ""); //$NON-NLS-1$
				if (symbolsFileName.length() > 0) {
					symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName).replace("\\", "\\\\");
					String symbolsOffset = "0x" + config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, ""); //$NON-NLS-2$
					commands.clear();
					gdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
					executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(15));
				}
			}
		} catch (OperationCanceledException e) {
			if (launch != null && launch.canTerminate()) {
				launch.terminate();

			// Set program counter
			boolean setPc = config.getAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER);
			if (setPc) {
				String pcRegister = config.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_PC_REGISTER)); //$NON-NLS-1$
				gdbJtagDevice.doSetPC(pcRegister, commands);
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(20));
			}

			monitor.beginTask(Messages.getString("GDBJtagDebugger.18"), 1); //$NON-NLS-1$
			boolean setStopAt = config.getAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT);
			if (setStopAt) {
				String stopAt = config.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, IGDBJtagConstants.DEFAULT_STOP_AT); //$NON-NLS-1$
				commands.clear();
				gdbJtagDevice.doStopAt(stopAt, commands);
				executeGDBScript(getGDBScript(commands), miSession, submonitor.newChild(20));

	private void executeGDBScript(String script, MISession miSession, 
			IProgressMonitor monitor) throws CoreException {
		// Try to execute any extra command
		if (script == null || script.length() == 0)
			return;
		script = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(script);
		String[] commands = script.split("\\r?\\n");

	private IGDBJtagDevice getGDBJtagDevice (ILaunchConfiguration config) 
		throws CoreException, NullPointerException {
		IGDBJtagDevice gdbJtagDevice = null;
		String jtagDeviceName = config.getAttribute(IGDBJtagConstants.ATTR_JTAG_DEVICE, IGDBJtagConstants.DEFAULT_JTAG_DEVICE); //$NON-NLS-1$
		GDBJtagDeviceContribution[] availableDevices = GDBJtagDeviceContributionFactory.
			getInstance().getGDBJtagDeviceContribution();
		for (int i = 0; i < availableDevices.length; i++) {

		}
		return sb.toString();
	}
	
}




/*******************************************************************************
 * Copyright (c) 2007 - 2010 QNX Software Systems and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at

 *******************************************************************************/
package org.eclipse.cdt.debug.gdbjtag.core;

import java.io.File;

import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;
import org.eclipse.cdt.core.model.ICProject;
import org.eclipse.cdt.debug.core.CDIDebugModel;
import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;
import org.eclipse.cdt.debug.core.cdi.CDIException;
import org.eclipse.cdt.debug.core.cdi.ICDISession;


		if (mode.equals(ILaunchManager.DEBUG_MODE)) {
			GDBJtagDebugger debugger = new GDBJtagDebugger();
			ICProject project = CDebugUtils.verifyCProject(configuration);
			IPath exePath = CDebugUtils.verifyProgramPath(configuration);
			ICDISession session = debugger.createSession(launch, null, submonitor.newChild(1));
			ICDISession session = debugger.createSession(launch, exeFile, submonitor.newChild(1));
			IBinaryObject exeBinary = null;
			if ( exePath != null ) {
				exeBinary = verifyBinary(project, exePath);

Lines 1-5 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/core/IGDBJtagConstants.java (-5 / +23 lines)
1	/*******************************************************************************	1	/*******************************************************************************
2	* Copyright (c) 2007 - 2008 QNX Software Systems and others.	2	* Copyright (c) 2007 - 2010 QNX Software Systems and others.
3	* All rights reserved. This program and the accompanying materials	3	* All rights reserved. This program and the accompanying materials
4	* are made available under the terms of the Eclipse Public License v1.0	4	* are made available under the terms of the Eclipse Public License v1.0
5	* which accompanies this distribution, and is available at	5	* which accompanies this distribution, and is available at
Lines 47-61 Link Here
47	public static final String ATTR_STOP_AT = Activator.PLUGIN_ID + ".stopAt"; //$NON-NLS-1$	47	public static final String ATTR_STOP_AT = Activator.PLUGIN_ID + ".stopAt"; //$NON-NLS-1$
48	public static final String ATTR_SET_RESUME = Activator.PLUGIN_ID + ".setResume"; //$NON-NLS-1$	48	public static final String ATTR_SET_RESUME = Activator.PLUGIN_ID + ".setResume"; //$NON-NLS-1$
49	public static final String ATTR_RUN_COMMANDS = Activator.PLUGIN_ID + ".runCommands"; //$NON-NLS-1$	49	public static final String ATTR_RUN_COMMANDS = Activator.PLUGIN_ID + ".runCommands"; //$NON-NLS-1$
50		50	/** @since 7.0 */ public static final String ATTR_USE_PROJ_BINARY_FOR_IMAGE = Activator.PLUGIN_ID + ".useProjBinaryForImage"; //$NON-NLS-1$
		51	/** @since 7.0 */ public static final String ATTR_USE_FILE_FOR_IMAGE = Activator.PLUGIN_ID + ".useFileForImage"; //$NON-NLS-1$
		52	/** @since 7.0 */ public static final String ATTR_USE_PROJ_BINARY_FOR_SYMBOLS = Activator.PLUGIN_ID + ".useProjBinaryForSymbols"; //$NON-NLS-1$
		53	/** @since 7.0 */ public static final String ATTR_USE_FILE_FOR_SYMBOLS = Activator.PLUGIN_ID + ".useFileForSymbols"; //$NON-NLS-1$
		54
51	public static final boolean DEFAULT_DO_RESET = true;	55	public static final boolean DEFAULT_DO_RESET = true;
52	public static final boolean DEFAULT_DO_HALT = true;	56	public static final boolean DEFAULT_DO_HALT = true;
53	public static final int DEFAULT_DELAY = 3;	57	public static final int DEFAULT_DELAY = 3;
54	public static final boolean DEFAULT_LOAD_IMAGE = false;	58	public static final boolean DEFAULT_LOAD_IMAGE = true;
55	public static final boolean DEFAULT_LOAD_SYMBOLS = false;	59	public static final boolean DEFAULT_LOAD_SYMBOLS = true;
56	public static final boolean DEFAULT_SET_PC_REGISTER = false;	60	public static final boolean DEFAULT_SET_PC_REGISTER = false;
57	public static final boolean DEFAULT_SET_STOP_AT = false;	61	public static final boolean DEFAULT_SET_STOP_AT = false;
58	public static final boolean DEFAULT_SET_RESUME = false;	62	public static final boolean DEFAULT_SET_RESUME = false;
59	public static final boolean DEFAULT_USE_DEFAULT_RUN = true;	63	public static final boolean DEFAULT_USE_DEFAULT_RUN = true;
60		64
		65	/** @since 7.0 */ public static final String DEFAULT_INIT_COMMANDS = ""; //$NON-NLS-1$
		66	/** @since 7.0 */ public static final String DEFAULT_IMAGE_FILE_NAME = ""; //$NON-NLS-1$
		67	/** @since 7.0 */ public static final String DEFAULT_SYMBOLS_FILE_NAME = ""; //$NON-NLS-1$
		68	/** @since 7.0 */ public static final String DEFAULT_RUN_COMMANDS = ""; //$NON-NLS-1$
		69	/** @since 7.0 */ public static final boolean DEFAULT_USE_PROJ_BINARY_FOR_IMAGE = true;
		70	/** @since 7.0 */ public static final boolean DEFAULT_USE_FILE_FOR_IMAGE = false;
		71	/** @since 7.0 */ public static final boolean DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS = true;
		72	/** @since 7.0 */ public static final boolean DEFAULT_USE_FILE_FOR_SYMBOLS = false;
		73	/** @since 7.0 */ public static final String DEFAULT_IMAGE_OFFSET = ""; //$NON-NLS-1$
		74	/** @since 7.0 */ public static final String DEFAULT_SYMBOLS_OFFSET = ""; //$NON-NLS-1$
		75	/** @since 7.0 */ public static final String DEFAULT_PC_REGISTER = ""; //$NON-NLS-1$
		76	/** @since 7.0 */ public static final String DEFAULT_STOP_AT = ""; //$NON-NLS-1$
		77	/** @since 7.0 */ public static final String DEFAULT_JTAG_DEVICE = ""; //$NON-NLS-1$
		78
61	}	79	}

Lines 1-5 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/core/Message.properties (-2 / +6 lines)
1	###############################################################################	1	###############################################################################
2	# Copyright (c) 2008 QNX Software Systems and others.	2	# Copyright (c) 2008 - 2010 QNX Software Systems and others.
3	# All rights reserved. This program and the accompanying materials	3	# All rights reserved. This program and the accompanying materials
4	# are made available under the terms of the Eclipse Public License v1.0	4	# are made available under the terms of the Eclipse Public License v1.0
5	# which accompanies this distribution, and is available at	5	# which accompanies this distribution, and is available at
Lines 15-19 Link Here
15	GDBJtagDebugger.21=GDB command:	15	GDBJtagDebugger.21=GDB command:
16	GDBJtagDebugger.22=Failed command	16	GDBJtagDebugger.22=Failed command
17	GDBJtagDebugger.3=Executing init commands	17	GDBJtagDebugger.3=Executing init commands
18	GDBJtagDebugger.5=Loading image	18	GDBJtagDebugger.loading_image=Loading image
		19	GDBJtagDebugger.loading_symbols=Loading symbolics
19	src.common.No_answer=No answer	20	src.common.No_answer=No answer
		21
		22	GDBJtagDebugger.err_no_sym_file=Symbolics loading was requested but file was not specified or not found.
		23	GDBJtagDebugger.err_no_img_file=Image loading was requested but file was not specified or not found.




	 */
	public void doLoadImage(String imageFileName, String imageOffset, Collection commands) {
		String file = escapeScpaces(imageFileName);
		
		if (imageOffset.length() > 0) {
			// 'restore' simply puts the program into memory. 
			addCmd(commands, "restore " + file + " " + imageOffset);
		}
		else {
			// 'load' puts the program into memory and sets the PC. To see why
			// we do this when no offset is specified, see
			// https://bugs.eclipse.org/bugs/show_bug.cgi?id=310304#c20
			addCmd(commands, "load " + file);			
		}
		
		// 'exec-file' specifies the program as the context for getting memory.
		// Basically, it tells gdb "this is the program we'll be debugging"
		addCmd(commands, "exec-file " + file);
	}

	/* (non-Javadoc)

	 */
	public void doLoadSymbol(String symbolFileName, String symbolOffset, Collection commands) {
		String file = escapeScpaces(symbolFileName);
		if (symbolOffset == null || (symbolOffset.length() == 0)) {
			addCmd(commands, "symbol-file " + file);
		}
		else {
			addCmd(commands, "add-sym " + file + " " + symbolOffset);			
		}
	}
	
	protected String escapeScpaces(String file) {




		gdbCommand.setLayoutData(gd);
		gdbCommand.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});


		jtagDevice.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				updateDeviceIpPort(jtagDevice.getText());
				scheduleUpdateJob();
			}
		});


		ipAddress.setLayoutData(gd);
		ipAddress.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});


		});
		portNumber.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
	}




		gdbCommand.setLayoutData(gd);
		gdbCommand.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});


		commandFactory.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				commandFactoryChanged();
				scheduleUpdateJob();
			}
		});
	}

		miProtocol = new Combo(comp, SWT.READ_ONLY | SWT.DROP_DOWN);
		miProtocol.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
	}

		jtagDevice.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				updateDeviceIpPort(jtagDevice.getText());
				scheduleUpdateJob();
			}
		});
		

		ipAddress.setLayoutData(gd);
		ipAddress.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		

		});
		portNumber.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
	}





import java.io.File;

import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.gdbjtag.core.IGDBJtagConstants;
import org.eclipse.core.resources.IResource;
import org.eclipse.core.resources.ResourcesPlugin;

import org.eclipse.swt.events.ModifyListener;
import org.eclipse.swt.events.SelectionAdapter;
import org.eclipse.swt.events.SelectionEvent;
import org.eclipse.swt.events.SelectionListener;
import org.eclipse.swt.events.VerifyEvent;
import org.eclipse.swt.events.VerifyListener;
import org.eclipse.swt.graphics.Image;

	boolean resume = false;
	
	Text runCommands;
	
	// New GUI added to address bug 310304
	private Button useProjectBinaryForImage;
	private Button useFileForImage;
	private Button useProjectBinaryForSymbols;
	private Button useFileForSymbols;
	private Label imageOffsetLabel;
	private Label symbolsOffsetLabel;
	private Label projBinaryLabel1;
	private Label projBinaryLabel2;

	public String getName() {
		return TAB_NAME;

		});
		delay.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		

		initCommands.setLayoutData(gd);
		initCommands.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent evt) {
				scheduleUpdateJob();
			}
		});
		

		comp.setLayoutData(new GridData(GridData.FILL_HORIZONTAL));
		comp.setLayout(layout);
		
		SelectionListener radioButtonListener = new SelectionListener() {
			public void widgetSelected(SelectionEvent e) {
				updateLaunchConfigurationDialog();
				updateUseFileEnablement();
			}
			public void widgetDefaultSelected(SelectionEvent e) {
			}
		};
		
		useProjectBinaryForImage = new Button(comp, SWT.RADIO);
		useProjectBinaryForImage.setText(Messages.getString("GDBJtagStartupTab.useProjectBinary_Label"));
		useProjectBinaryForImage.setToolTipText(Messages.getString("GDBJtagStartupTab.useProjectBinary_ToolTip"));
		gd = new GridData();
		gd.horizontalSpan = 1;
		useProjectBinaryForImage.setLayoutData(gd);
		useProjectBinaryForImage.addSelectionListener(radioButtonListener);
		
		projBinaryLabel1 = new Label(comp, SWT.NONE);
		gd = new GridData(GridData.FILL_HORIZONTAL);
		gd.horizontalSpan = 3;
		projBinaryLabel1.setLayoutData(gd);
		
		useFileForImage = new Button(comp, SWT.RADIO);
		useFileForImage.setText(Messages.getString("GDBJtagStartupTab.useFile_Label"));
		gd = new GridData();
		gd.horizontalSpan = 1;
		useFileForImage.setLayoutData(gd);
		useFileForImage.addSelectionListener(radioButtonListener);		

		imageFileName = new Text(comp, SWT.BORDER);
		gd = new GridData(GridData.FILL_HORIZONTAL);
		gd.horizontalSpan = 1;
		imageFileName.setLayoutData(gd);
		imageFileName.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		

			}
		});	
		
		imageOffsetLabel = new Label(comp, SWT.NONE);
		imageOffsetLabel.setText(Messages.getString("GDBJtagStartupTab.imageOffsetLabel_Text"));
		imageOffset = new Text(comp, SWT.BORDER);
		gd = new GridData();

		});
		imageOffset.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		
		
		loadSymbols = new Button(group, SWT.CHECK);
		loadSymbols.setText(Messages.getString("GDBJtagStartupTab.loadSymbols_Text"));
		gd = new GridData();

		layout.numColumns = 4;
		comp.setLayoutData(new GridData(GridData.FILL_HORIZONTAL));
		comp.setLayout(layout);

		useProjectBinaryForSymbols = new Button(comp, SWT.RADIO);
		useProjectBinaryForSymbols.setText(Messages.getString("GDBJtagStartupTab.useProjectBinary_Label"));
		useProjectBinaryForSymbols.setToolTipText(Messages.getString("GDBJtagStartupTab.useProjectBinary_ToolTip"));
		gd = new GridData();
		gd.horizontalSpan = 1;
		useProjectBinaryForSymbols.setLayoutData(gd);
		useProjectBinaryForSymbols.addSelectionListener(radioButtonListener);
		
		projBinaryLabel2 = new Label(comp, SWT.NONE);
		gd = new GridData(GridData.FILL_HORIZONTAL);
		gd.horizontalSpan = 3;
		projBinaryLabel2.setLayoutData(gd);
		
		useFileForSymbols = new Button(comp, SWT.RADIO);
		useFileForSymbols.setText(Messages.getString("GDBJtagStartupTab.useFile_Label"));
		gd = new GridData();
		gd.horizontalSpan = 1;
		useFileForSymbols.setLayoutData(gd);
		useFileForSymbols.addSelectionListener(radioButtonListener);
		
		Label symbolLabel = new Label(comp, SWT.NONE);
		symbolLabel.setText(Messages.getString("GDBJtagStartupTab.symbolsLabel_Text"));	
		symbolsFileName = new Text(comp, SWT.BORDER);
		gd = new GridData(GridData.FILL_HORIZONTAL);
		gd.horizontalSpan = 1;
		symbolsFileName.setLayoutData(gd);
		symbolsFileName.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});	
		

			}
		});
		
		symbolsOffsetLabel = new Label(comp, SWT.NONE);
		symbolsOffsetLabel.setText(Messages.getString("GDBJtagStartupTab.symbolsOffsetLabel_Text"));
		symbolsOffset = new Text(comp, SWT.BORDER);
		gd = new GridData();

		});
		symbolsOffset.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		
	}
	
	private void updateUseFileEnablement() {
		boolean enabled = loadImage.getSelection() && useFileForImage.getSelection();
		imageFileName.setEnabled(enabled);
		imageFileBrowseWs.setEnabled(enabled);
		imageFileBrowse.setEnabled(enabled);
		
		enabled = loadSymbols.getSelection() && useFileForSymbols.getSelection();
		symbolsFileName.setEnabled(enabled);
		symbolsFileBrowseWs.setEnabled(enabled);
		symbolsFileBrowse.setEnabled(enabled);
	}

	public void createRunOptionGroup(Composite parent) {
		Group group = new Group(parent, SWT.NONE);
		GridLayout layout = new GridLayout();

		});
		pcRegister.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});
		

		stopAt.setLayoutData(gd);
		stopAt.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent e) {
				scheduleUpdateJob();
			}
		});



	private void loadImageChanged() {
		boolean enabled = loadImage.getSelection();
		useProjectBinaryForImage.setEnabled(enabled);
		useFileForImage.setEnabled(enabled);
		imageFileBrowse.setEnabled(enabled);
		imageOffset.setEnabled(enabled);
		imageOffsetLabel.setEnabled(enabled);
		updateUseFileEnablement();
	}
	
	private void loadSymbolsChanged() {
		boolean enabled = loadSymbols.getSelection();
		useProjectBinaryForSymbols.setEnabled(enabled);
		useFileForSymbols.setEnabled(enabled);
		symbolsFileBrowse.setEnabled(enabled);
		symbolsOffset.setEnabled(enabled);
		symbolsOffsetLabel.setEnabled(enabled);
		updateUseFileEnablement();		
	}
	
	private void pcRegisterChanged() {

		runCommands.setLayoutData(gd);
		runCommands.addModifyListener(new ModifyListener() {
			public void modifyText(ModifyEvent evt) {
				scheduleUpdateJob();
			}
		});
	}

		setMessage(null);

		if (loadImage.getSelection()) {
			if (!useProjectBinaryForImage.getSelection()) {
				if (imageFileName.getText().trim().length() == 0) {
					setErrorMessage(Messages.getString("GDBJtagStartupTab.imageFileName_not_specified"));
					return false;
				}
	
				try {
					String path = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(imageFileName.getText().trim());
					IPath filePath = new Path(path);
					if (!filePath.toFile().exists()) {
						setErrorMessage(Messages.getString("GDBJtagStartupTab.imageFileName_does_not_exist"));
						return false;
					}
				} catch (CoreException e) { // string substitution throws this if expression doesn't resolve 
					setErrorMessage(Messages.getString("GDBJtagStartupTab.imageFileName_does_not_exist"));
					return false;
				}
			} catch (CoreException e) {
				Activator.getDefault().getLog().log(e.getStatus());
			}
		} else {
			setErrorMessage(null);
		}
		if (loadSymbols.getSelection()) {
			if (!useProjectBinaryForSymbols.getSelection()) {
				if (symbolsFileName.getText().trim().length() == 0) {
					setErrorMessage(Messages.getString("GDBJtagStartupTab.symbolsFileName_not_specified"));
					return false;
				}
			
				try {
					String path = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName.getText().trim());
					IPath filePath = new Path(path);
					if (!filePath.toFile().exists()) {
						setErrorMessage(Messages.getString("GDBJtagStartupTab.symbolsFileName_does_not_exist"));
						return false;
					}
				} catch (CoreException e) { // string substitution throws this if expression doesn't resolve 
					setErrorMessage(Messages.getString("GDBJtagStartupTab.symbolsFileName_does_not_exist"));
					return false;
				}
			} catch (CoreException e) {
				Activator.getDefault().getLog().log(e.getStatus());
			}
		} else {
			setErrorMessage(null);

	}
	
	/* (non-Javadoc)
	 * @see org.eclipse.debug.ui.ILaunchConfigurationTab#initializeFrom(org.eclipse.debug.core.ILaunchConfiguration)
	 */
//	protected void updateLaunchConfigurationDialog() {
//		super.updateLaunchConfigurationDialog();
//		isValid(getLaunchConfigurationDialog());
//	}
	
	public void initializeFrom(ILaunchConfiguration configuration) {
		try {
			// Initialization Commands
			doReset.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_DO_RESET, IGDBJtagConstants.DEFAULT_DO_RESET));
			doResetChanged();
			doHalt.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT));
			delay.setText(String.valueOf(configuration.getAttribute(IGDBJtagConstants.ATTR_DELAY, IGDBJtagConstants.DEFAULT_DELAY)));
			doHalt.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT));
			initCommands.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, IGDBJtagConstants.DEFAULT_INIT_COMMANDS));
			
			// Load Image...
			loadImage.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE));
			useProjectBinaryForImage.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_IMAGE));
			useFileForImage.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_FILE_FOR_IMAGE));
			imageFileName.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, IGDBJtagConstants.DEFAULT_IMAGE_FILE_NAME));
			imageOffset.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET));
			
			//.. and Symbols
			imageOffset.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, "")); //$NON-NLS-1$
			loadSymbols.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS));
			useProjectBinaryForSymbols.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS));
			useFileForSymbols.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_FILE_FOR_SYMBOLS));
			symbolsFileName.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, IGDBJtagConstants.DEFAULT_SYMBOLS_FILE_NAME));
			symbolsOffset.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, IGDBJtagConstants.DEFAULT_SYMBOLS_OFFSET));
			
			// Runtime Options
			setPcRegister.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER));
			pcRegister.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, IGDBJtagConstants.DEFAULT_PC_REGISTER));
			pcRegister.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, "")); //$NON-NLS-1$
			setStopAt.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT));
			stopAt.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, IGDBJtagConstants.DEFAULT_STOP_AT));
			stopAt.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, "")); //$NON-NLS-1$
			setResume.setSelection(configuration.getAttribute(IGDBJtagConstants.ATTR_SET_RESUME, IGDBJtagConstants.DEFAULT_SET_RESUME));
			
			// Run Commands
			runCommands.setText(configuration.getAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, IGDBJtagConstants.DEFAULT_RUN_COMMANDS));

			String programName = CDebugUtils.getProgramName(configuration);
			if (programName != null) {
				int lastSlash = programName.indexOf('\\');
				if (lastSlash >= 0) {
					programName = programName.substring(lastSlash + 1);
				}
				lastSlash = programName.indexOf('/');
				if (lastSlash >= 0) {
					programName = programName.substring(lastSlash + 1);
				}
				projBinaryLabel1.setText(programName);
				projBinaryLabel2.setText(programName);
			}
		
			doResetChanged();
			loadImageChanged();
			loadSymbolsChanged();
			pcRegisterChanged();
			stopAtChanged();
			resumeChanged();
			updateUseFileEnablement();
			
		} catch (CoreException e) {
			Activator.getDefault().getLog().log(e.getStatus());
		}
	}

	/* (non-Javadoc)
	 * @see org.eclipse.debug.ui.ILaunchConfigurationTab#performApply(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy)
	 */
	public void performApply(ILaunchConfigurationWorkingCopy configuration) {
		
		// Initialization Commands
		configuration.setAttribute(IGDBJtagConstants.ATTR_DO_RESET, doReset.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_DELAY, Integer.parseInt(delay.getText()));
		configuration.setAttribute(IGDBJtagConstants.ATTR_DO_HALT, doHalt.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, initCommands.getText());
		
		// Load Image...
		configuration.setAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, loadImage.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, useProjectBinaryForImage.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_IMAGE, useFileForImage.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, imageFileName.getText().trim());
		configuration.setAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, imageOffset.getText());
		
		//.. and Symbols
		configuration.setAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, loadSymbols.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, useProjectBinaryForSymbols.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_SYMBOLS, useFileForSymbols.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, symbolsFileName.getText().trim());
		configuration.setAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, symbolsOffset.getText());
		
		// Runtime Options
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, setPcRegister.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, pcRegister.getText());
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, setStopAt.getSelection());
		configuration.setAttribute(IGDBJtagConstants.ATTR_STOP_AT, stopAt.getText());
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_RESUME, setResume.getSelection());
		
		// Run Commands
		configuration.setAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, runCommands.getText());
	}

	/* (non-Javadoc)
	 * @see org.eclipse.debug.ui.ILaunchConfigurationTab#setDefaults(org.eclipse.debug.core.ILaunchConfigurationWorkingCopy)
	 */
	public void setDefaults(ILaunchConfigurationWorkingCopy configuration) {
		// Initialization Commands
		configuration.setAttribute(IGDBJtagConstants.ATTR_DO_RESET, IGDBJtagConstants.DEFAULT_DO_RESET);
		configuration.setAttribute(IGDBJtagConstants.ATTR_DELAY, IGDBJtagConstants.DEFAULT_DELAY);
		configuration.setAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT);
		configuration.setAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, IGDBJtagConstants.DEFAULT_INIT_COMMANDS);

		// Load Image...
		configuration.setAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE);
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_IMAGE);
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_FILE_FOR_IMAGE);
		configuration.setAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, IGDBJtagConstants.DEFAULT_IMAGE_FILE_NAME);
		configuration.setAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET);

		//.. and Symbols
		configuration.setAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS);
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS);
		configuration.setAttribute(IGDBJtagConstants.ATTR_USE_FILE_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_FILE_FOR_SYMBOLS);
		configuration.setAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, IGDBJtagConstants.DEFAULT_SYMBOLS_FILE_NAME);
		configuration.setAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, IGDBJtagConstants.DEFAULT_SYMBOLS_OFFSET);
		
		// Runtime Options
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER);
		configuration.setAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, IGDBJtagConstants.DEFAULT_PC_REGISTER);
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT);
		configuration.setAttribute(IGDBJtagConstants.ATTR_STOP_AT, IGDBJtagConstants.DEFAULT_STOP_AT);
		configuration.setAttribute(IGDBJtagConstants.ATTR_SET_RESUME, IGDBJtagConstants.DEFAULT_SET_RESUME);

		// Run Commands
		configuration.setAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, IGDBJtagConstants.DEFAULT_RUN_COMMANDS); 
	}
}

Lines 28-33 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/ui/JtagUi.properties (+3 lines)
28	GDBJtagStartupTab.symbolsFileBrowseWs_Title=Select symbols file	28	GDBJtagStartupTab.symbolsFileBrowseWs_Title=Select symbols file
29	GDBJtagStartupTab.symbolsFileBrowse_Title=Select symbols file	29	GDBJtagStartupTab.symbolsFileBrowse_Title=Select symbols file
30	GDBJtagStartupTab.symbolsOffsetLabel_Text=Symbols offset (hex):	30	GDBJtagStartupTab.symbolsOffsetLabel_Text=Symbols offset (hex):
		31	GDBJtagStartupTab.useProjectBinary_Label=Use project binary:
		32	GDBJtagStartupTab.useFile_Label=Use file:
		33	GDBJtagStartupTab.useProjectBinary_ToolTip=Use C/C++ application specified in the Main tab.
31		34
32	GDBJtagStartupTab.runOptionGroup_Text=Runtime Options	35	GDBJtagStartupTab.runOptionGroup_Text=Runtime Options
33	GDBJtagStartupTab.setPcRegister_Text=Set program counter at (hex):	36	GDBJtagStartupTab.setPcRegister_Text=Set program counter at (hex):




	 */
	@Deprecated
	protected IFile getProgramFile(ILaunchConfiguration config) throws CoreException {
		ICProject cproject = CDebugUtils.verifyCProject(config);
		String fileName = CDebugUtils.getProgramName(config);
		if (fileName == null) {
			abort(LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_not_specified"), null, //$NON-NLS-1$

		return programPath;
	}

	/**
	 * @deprecated use {@link CDebugUtils#verifyCProject(ILaunchConfiguration)}
	 */
	protected ICProject verifyCProject(ILaunchConfiguration config) throws CoreException {
		return CDebugUtils.verifyCProject(config);
		if (name == null) {
			abort(LaunchMessages.getString("AbstractCLaunchDelegate.C_Project_not_specified"), null, //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROJECT);
		}
		ICProject cproject = CDebugUtils.getCProject(config);
		if (cproject == null) {
			IProject proj = ResourcesPlugin.getWorkspace().getRoot().getProject(name);
			if (!proj.exists()) {
				abort(
						LaunchMessages.getFormattedString("AbstractCLaunchDelegate.Project_NAME_does_not_exist", name), null, //$NON-NLS-1$
						ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
			} else if (!proj.isOpen()) {
				abort(LaunchMessages.getFormattedString("AbstractCLaunchDelegate.Project_NAME_is_closed", name), null, //$NON-NLS-1$
						ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
			}
			abort(LaunchMessages.getString("AbstractCLaunchDelegate.Not_a_C_CPP_project"), null, //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
		}
		return cproject;
	}

	/**
	 * @deprecated use {@link CDebugUtils#verifyProgramPath(ILaunchConfiguration)
	 */
	protected IPath verifyProgramPath(ILaunchConfiguration config) throws CoreException {
		return CDebugUtils.verifyProgramPath(config);
		IPath programPath = CDebugUtils.getProgramPath(config);
		if (programPath == null || programPath.isEmpty()) {
			abort(LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_not_specified"), null, //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROGRAM);
		}
		if (!programPath.isAbsolute()) {
			IPath location = cproject.getProject().getLocation();
			if (location != null) {
				programPath = location.append(programPath);
				if (!programPath.toFile().exists()) {
					// Try the old way, which is required to support linked resources.
					IFile projFile = null;
					try {
						projFile = project.getFile(CDebugUtils.getProgramPath(config));
					}
					catch (IllegalArgumentException exc) {}	// thrown if relative path that resolves to a root file (e.g., "..\somefile")
					if (projFile != null && projFile.exists()) {
						programPath = projFile.getLocation();
					}
				}
			}
		}
		if (!programPath.toFile().exists()) {
			abort(
					LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_does_not_exist"), //$NON-NLS-1$
					new FileNotFoundException(
							LaunchMessages.getFormattedString(
																"AbstractCLaunchDelegate.PROGRAM_PATH_not_found", programPath.toOSString())), //$NON-NLS-1$
					ICDTLaunchConfigurationConstants.ERR_PROGRAM_NOT_EXIST);
		}
		return programPath;
	}

	protected IPath verifyProgramFile(ILaunchConfiguration config) throws CoreException {

Lines 53-60 Link Here

(-)src/org/eclipse/cdt/launch/internal/CoreFileLaunchDelegate.java (-2 / +2 lines)
53	}	53	}
54	try {	54	try {
55	monitor.worked(1);	55	monitor.worked(1);
56	IPath exePath = verifyProgramPath(config);	56	IPath exePath = CDebugUtils.verifyProgramPath(config);
57	ICProject project = verifyCProject(config);	57	ICProject project = CDebugUtils.verifyCProject(config);
58	IBinaryObject exeFile = verifyBinary(project, exePath);	58	IBinaryObject exeFile = verifyBinary(project, exePath);
59		59
60	ICDebugConfiguration debugConfig = getDebugConfig(config);	60	ICDebugConfiguration debugConfig = getDebugConfig(config);

Lines 1-5 Link Here

(-)src/org/eclipse/cdt/launch/internal/LocalAttachLaunchDelegate.java (-2 / +2 lines)
1	/*******************************************************************************	1	/*******************************************************************************
2	* Copyright (c) 2004, 2006 QNX Software Systems and others.	2	* Copyright (c) 2004, 2010 QNX Software Systems and others.
3	* All rights reserved. This program and the accompanying materials	3	* All rights reserved. This program and the accompanying materials
4	* are made available under the terms of the Eclipse Public License v1.0	4	* are made available under the terms of the Eclipse Public License v1.0
5	* which accompanies this distribution, and is available at	5	* which accompanies this distribution, and is available at
Lines 62-68 Link Here
62	}	62	}
63	try {	63	try {
64	monitor.worked(1);	64	monitor.worked(1);
65	ICProject cproject = verifyCProject(config);	65	ICProject cproject = CDebugUtils.verifyCProject(config);
66	IPath exePath = CDebugUtils.getProgramPath(config);	66	IPath exePath = CDebugUtils.getProgramPath(config);
67	if (exePath != null && !exePath.isEmpty()) {	67	if (exePath != null && !exePath.isEmpty()) {
68	if (!exePath.isAbsolute()) {	68	if (!exePath.isAbsolute()) {




/*******************************************************************************
 * Copyright (c) 2004, 2010 QNX Software Systems and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at

import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;
import org.eclipse.cdt.core.model.ICProject;
import org.eclipse.cdt.debug.core.CDIDebugModel;
import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.core.ICDIDebugger;
import org.eclipse.cdt.debug.core.ICDIDebugger2;
import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;

		}
		monitor.worked( 1 );
		try {
			IPath exePath = CDebugUtils.verifyProgramPath( config );
			File wd = getWorkingDirectory( config );
			if ( wd == null ) {
				wd = new File( System.getProperty( "user.home", "." ) ); //$NON-NLS-1$ //$NON-NLS-2$

		monitor.subTask( LaunchMessages.getString( "LocalCDILaunchDelegate.2" ) ); //$NON-NLS-1$
		ICDISession dsession = null;
		try {
			IPath exePath = CDebugUtils.verifyProgramPath( config );
			ICProject project = CDebugUtils.verifyCProject( config );
			IBinaryObject exeFile = null;
			if ( exePath != null ) {
				exeFile = verifyBinary( project, exePath );

			}
			cancel( "", -1 ); //$NON-NLS-1$
		}
		IPath exePath = CDebugUtils.verifyProgramPath( config );
		if (exePath == null) {
			exePath= getProgramPathForPid(pid);
		}
		ICProject project = CDebugUtils.verifyCProject( config );
		IBinaryObject exeFile = null;
		if ( exePath != null ) {
			exeFile = verifyBinary( project, exePath );

		ICDebugConfiguration debugConfig = getDebugConfig( config );
		String path = config.getAttribute( ICDTLaunchConfigurationConstants.ATTR_COREFILE_PATH, (String)null );
		if ( path == null || path.length() == 0) {
			ICProject project = CDebugUtils.verifyCProject( config );
			IPath corefile = promptForCoreFilePath( (IProject)project.getResource(), debugConfig );
			if ( corefile == null ) {
				cancel( LaunchMessages.getString( "LocalCDILaunchDelegate.6" ), ICDTLaunchConfigurationConstants.ERR_NO_COREFILE ); //$NON-NLS-1$

			cancel( "", -1 ); //$NON-NLS-1$
		}

		IPath exePath = CDebugUtils.verifyProgramPath( config );
		ICProject project = CDebugUtils.verifyCProject( config );
		IBinaryObject exeFile = null;
		if ( exePath != null ) {
			exeFile = verifyBinary( project, exePath );


	private ICDISession launchOldDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger debugger, IProgressMonitor monitor ) throws CoreException {
		IBinaryObject exeFile = null;
		IPath exePath = CDebugUtils.verifyProgramPath( config );
		ICProject project = CDebugUtils.verifyCProject( config );
		if ( exePath != null ) {
			exeFile = verifyBinary( project, exePath );
		}

	}

	private ICDISession launchDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger2 debugger, IProgressMonitor monitor ) throws CoreException {
		IPath path = CDebugUtils.verifyProgramPath( config );
		File exeFile = path != null ? path.toFile() : null;
		return debugger.createSession( launch, exeFile, monitor );
	}




/*******************************************************************************
 * Copyright (c) 2005, 2010 QNX Software Systems and others.
 * All rights reserved. This program and the accompanying materials
 * are made available under the terms of the Eclipse Public License v1.0
 * which accompanies this distribution, and is available at

import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;
import org.eclipse.cdt.core.model.ICProject;
import org.eclipse.cdt.debug.core.CDIDebugModel;
import org.eclipse.cdt.debug.core.CDebugUtils;
import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;
import org.eclipse.cdt.debug.core.ICDebugConfiguration;
import org.eclipse.cdt.debug.core.cdi.CDIException;

		}
		try {
			monitor.worked(1);
			IPath exePath = CDebugUtils.verifyProgramPath(config);
			ICProject project = CDebugUtils.verifyCProject(config);
			if (exePath != null) {
				exeFile = verifyBinary(project, exePath);
			}

Return to bug 310304

Lines 141-147 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDSFDebuggerTab.java (-4 / +4 lines)
141	gdbCommand.setLayoutData(gd);	141	gdbCommand.setLayoutData(gd);
142	gdbCommand.addModifyListener(new ModifyListener() {	142	gdbCommand.addModifyListener(new ModifyListener() {
143	public void modifyText(ModifyEvent e) {	143	public void modifyText(ModifyEvent e) {
144	updateLaunchConfigurationDialog();	144	scheduleUpdateJob();
145	}	145	}
146	});	146	});
147		147
Lines 199-205 Link Here
199	jtagDevice.addModifyListener(new ModifyListener() {	199	jtagDevice.addModifyListener(new ModifyListener() {
200	public void modifyText(ModifyEvent e) {	200	public void modifyText(ModifyEvent e) {
201	updateDeviceIpPort(jtagDevice.getText());	201	updateDeviceIpPort(jtagDevice.getText());
202	updateLaunchConfigurationDialog();	202	scheduleUpdateJob();
203	}	203	}
204	});	204	});
205		205
Lines 211-217 Link Here
211	ipAddress.setLayoutData(gd);	211	ipAddress.setLayoutData(gd);
212	ipAddress.addModifyListener(new ModifyListener() {	212	ipAddress.addModifyListener(new ModifyListener() {
213	public void modifyText(ModifyEvent e) {	213	public void modifyText(ModifyEvent e) {
214	updateLaunchConfigurationDialog();	214	scheduleUpdateJob();
215	}	215	}
216	});	216	});
217		217
Lines 228-234 Link Here
228	});	228	});
229	portNumber.addModifyListener(new ModifyListener() {	229	portNumber.addModifyListener(new ModifyListener() {
230	public void modifyText(ModifyEvent e) {	230	public void modifyText(ModifyEvent e) {
231	updateLaunchConfigurationDialog();	231	scheduleUpdateJob();
232	}	232	}
233	});	233	});
234	}	234	}

Lines 146-152 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/ui/GDBJtagDebuggerTab.java (-6 / +6 lines)
146	gdbCommand.setLayoutData(gd);	146	gdbCommand.setLayoutData(gd);
147	gdbCommand.addModifyListener(new ModifyListener() {	147	gdbCommand.addModifyListener(new ModifyListener() {
148	public void modifyText(ModifyEvent e) {	148	public void modifyText(ModifyEvent e) {
149	updateLaunchConfigurationDialog();	149	scheduleUpdateJob();
150	}	150	}
151	});	151	});
152		152
Lines 186-192 Link Here
186	commandFactory.addModifyListener(new ModifyListener() {	186	commandFactory.addModifyListener(new ModifyListener() {
187	public void modifyText(ModifyEvent e) {	187	public void modifyText(ModifyEvent e) {
188	commandFactoryChanged();	188	commandFactoryChanged();
189	updateLaunchConfigurationDialog();	189	scheduleUpdateJob();
190	}	190	}
191	});	191	});
192	}	192	}
Lines 201-207 Link Here
201	miProtocol = new Combo(comp, SWT.READ_ONLY \| SWT.DROP_DOWN);	201	miProtocol = new Combo(comp, SWT.READ_ONLY \| SWT.DROP_DOWN);
202	miProtocol.addModifyListener(new ModifyListener() {	202	miProtocol.addModifyListener(new ModifyListener() {
203	public void modifyText(ModifyEvent e) {	203	public void modifyText(ModifyEvent e) {
204	updateLaunchConfigurationDialog();	204	scheduleUpdateJob();
205	}	205	}
206	});	206	});
207	}	207	}
Lines 272-278 Link Here
272	jtagDevice.addModifyListener(new ModifyListener() {	272	jtagDevice.addModifyListener(new ModifyListener() {
273	public void modifyText(ModifyEvent e) {	273	public void modifyText(ModifyEvent e) {
274	updateDeviceIpPort(jtagDevice.getText());	274	updateDeviceIpPort(jtagDevice.getText());
275	updateLaunchConfigurationDialog();	275	scheduleUpdateJob();
276	}	276	}
277	});	277	});
278		278
Lines 284-290 Link Here
284	ipAddress.setLayoutData(gd);	284	ipAddress.setLayoutData(gd);
285	ipAddress.addModifyListener(new ModifyListener() {	285	ipAddress.addModifyListener(new ModifyListener() {
286	public void modifyText(ModifyEvent e) {	286	public void modifyText(ModifyEvent e) {
287	updateLaunchConfigurationDialog();	287	scheduleUpdateJob();
288	}	288	}
289	});	289	});
290		290
Lines 301-307 Link Here
301	});	301	});
302	portNumber.addModifyListener(new ModifyListener() {	302	portNumber.addModifyListener(new ModifyListener() {
303	public void modifyText(ModifyEvent e) {	303	public void modifyText(ModifyEvent e) {
304	updateLaunchConfigurationDialog();	304	scheduleUpdateJob();
305	}	305	}
306	});	306	});
307	}	307	}

Lines 398-404 Link Here

(-)src/org/eclipse/cdt/launch/AbstractCLaunchDelegate.java (-53 / +9 lines)
398	*/	398	*/
399	@Deprecated	399	@Deprecated
400	protected IFile getProgramFile(ILaunchConfiguration config) throws CoreException {	400	protected IFile getProgramFile(ILaunchConfiguration config) throws CoreException {
401	ICProject cproject = verifyCProject(config);	401	ICProject cproject = CDebugUtils.verifyCProject(config);
402	String fileName = CDebugUtils.getProgramName(config);	402	String fileName = CDebugUtils.getProgramName(config);
403	if (fileName == null) {	403	if (fileName == null) {
404	abort(LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_not_specified"), null, //$NON-NLS-1$	404	abort(LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_not_specified"), null, //$NON-NLS-1$
Lines 417-478 Link Here
417	return programPath;	417	return programPath;
418	}	418	}
419		419
		420	/**
		421	* @deprecated use {@link CDebugUtils#verifyCProject(ILaunchConfiguration)}
		422	*/
420	protected ICProject verifyCProject(ILaunchConfiguration config) throws CoreException {	423	protected ICProject verifyCProject(ILaunchConfiguration config) throws CoreException {
421	String name = CDebugUtils.getProjectName(config);	424	return CDebugUtils.verifyCProject(config);
422	if (name == null) {
423	abort(LaunchMessages.getString("AbstractCLaunchDelegate.C_Project_not_specified"), null, //$NON-NLS-1$
424	ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROJECT);
425	}
426	ICProject cproject = CDebugUtils.getCProject(config);
427	if (cproject == null) {
428	IProject proj = ResourcesPlugin.getWorkspace().getRoot().getProject(name);
429	if (!proj.exists()) {
430	abort(
431	LaunchMessages.getFormattedString("AbstractCLaunchDelegate.Project_NAME_does_not_exist", name), null, //$NON-NLS-1$
432	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
433	} else if (!proj.isOpen()) {
434	abort(LaunchMessages.getFormattedString("AbstractCLaunchDelegate.Project_NAME_is_closed", name), null, //$NON-NLS-1$
435	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
436	}
437	abort(LaunchMessages.getString("AbstractCLaunchDelegate.Not_a_C_CPP_project"), null, //$NON-NLS-1$
438	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
439	}
440	return cproject;
441	}	425	}
442		426
		427	/**
		428	* @deprecated use {@link CDebugUtils#verifyProgramPath(ILaunchConfiguration)
		429	*/
443	protected IPath verifyProgramPath(ILaunchConfiguration config) throws CoreException {	430	protected IPath verifyProgramPath(ILaunchConfiguration config) throws CoreException {
444	ICProject cproject = verifyCProject(config);	431	return CDebugUtils.verifyProgramPath(config);
445	IPath programPath = CDebugUtils.getProgramPath(config);
446	if (programPath == null \|\| programPath.isEmpty()) {
447	abort(LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_not_specified"), null, //$NON-NLS-1$
448	ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROGRAM);
449	}
450	if (!programPath.isAbsolute()) {
451	IPath location = cproject.getProject().getLocation();
452	if (location != null) {
453	programPath = location.append(programPath);
454	if (!programPath.toFile().exists()) {
455	// Try the old way, which is required to support linked resources.
456	IFile projFile = null;
457	try {
458	projFile = project.getFile(CDebugUtils.getProgramPath(config));
459	}
460	catch (IllegalArgumentException exc) {} // thrown if relative path that resolves to a root file (e.g., "..\somefile")
461	if (projFile != null && projFile.exists()) {
462	programPath = projFile.getLocation();
463	}
464	}
465	}
466	}
467	if (!programPath.toFile().exists()) {
468	abort(
469	LaunchMessages.getString("AbstractCLaunchDelegate.Program_file_does_not_exist"), //$NON-NLS-1$
470	new FileNotFoundException(
471	LaunchMessages.getFormattedString(
472	"AbstractCLaunchDelegate.PROGRAM_PATH_not_found", programPath.toOSString())), //$NON-NLS-1$
473	ICDTLaunchConfigurationConstants.ERR_PROGRAM_NOT_EXIST);
474	}
475	return programPath;
476	}	432	}
477		433
478	protected IPath verifyProgramFile(ILaunchConfiguration config) throws CoreException {	434	protected IPath verifyProgramFile(ILaunchConfiguration config) throws CoreException {

Lines 1-5 Link Here

(-)src/org/eclipse/cdt/launch/internal/LocalCDILaunchDelegate.java (-12 / +13 lines)
1	/*******************************************************************************	1	/*******************************************************************************
2	* Copyright (c) 2004, 2009 QNX Software Systems and others.	2	* Copyright (c) 2004, 2010 QNX Software Systems and others.
3	* All rights reserved. This program and the accompanying materials	3	* All rights reserved. This program and the accompanying materials
4	* are made available under the terms of the Eclipse Public License v1.0	4	* are made available under the terms of the Eclipse Public License v1.0
5	* which accompanies this distribution, and is available at	5	* which accompanies this distribution, and is available at
Lines 23-28 Link Here
23	import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;	23	import org.eclipse.cdt.core.IBinaryParser.IBinaryObject;
24	import org.eclipse.cdt.core.model.ICProject;	24	import org.eclipse.cdt.core.model.ICProject;
25	import org.eclipse.cdt.debug.core.CDIDebugModel;	25	import org.eclipse.cdt.debug.core.CDIDebugModel;
		26	import org.eclipse.cdt.debug.core.CDebugUtils;
26	import org.eclipse.cdt.debug.core.ICDIDebugger;	27	import org.eclipse.cdt.debug.core.ICDIDebugger;
27	import org.eclipse.cdt.debug.core.ICDIDebugger2;	28	import org.eclipse.cdt.debug.core.ICDIDebugger2;
28	import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;	29	import org.eclipse.cdt.debug.core.ICDTLaunchConfigurationConstants;
Lines 80-86 Link Here
80	}	81	}
81	monitor.worked( 1 );	82	monitor.worked( 1 );
82	try {	83	try {
83	IPath exePath = verifyProgramPath( config );	84	IPath exePath = CDebugUtils.verifyProgramPath( config );
84	File wd = getWorkingDirectory( config );	85	File wd = getWorkingDirectory( config );
85	if ( wd == null ) {	86	if ( wd == null ) {
86	wd = new File( System.getProperty( "user.home", "." ) ); //$NON-NLS-1$ //$NON-NLS-2$	87	wd = new File( System.getProperty( "user.home", "." ) ); //$NON-NLS-1$ //$NON-NLS-2$
Lines 130-137 Link Here
130	monitor.subTask( LaunchMessages.getString( "LocalCDILaunchDelegate.2" ) ); //$NON-NLS-1$	131	monitor.subTask( LaunchMessages.getString( "LocalCDILaunchDelegate.2" ) ); //$NON-NLS-1$
131	ICDISession dsession = null;	132	ICDISession dsession = null;
132	try {	133	try {
133	IPath exePath = verifyProgramPath( config );	134	IPath exePath = CDebugUtils.verifyProgramPath( config );
134	ICProject project = verifyCProject( config );	135	ICProject project = CDebugUtils.verifyCProject( config );
135	IBinaryObject exeFile = null;	136	IBinaryObject exeFile = null;
136	if ( exePath != null ) {	137	if ( exePath != null ) {
137	exeFile = verifyBinary( project, exePath );	138	exeFile = verifyBinary( project, exePath );
Lines 201-211 Link Here
201	}	202	}
202	cancel( "", -1 ); //$NON-NLS-1$	203	cancel( "", -1 ); //$NON-NLS-1$
203	}	204	}
204	IPath exePath = verifyProgramPath( config );	205	IPath exePath = CDebugUtils.verifyProgramPath( config );
205	if (exePath == null) {	206	if (exePath == null) {
206	exePath= getProgramPathForPid(pid);	207	exePath= getProgramPathForPid(pid);
207	}	208	}
208	ICProject project = verifyCProject( config );	209	ICProject project = CDebugUtils.verifyCProject( config );
209	IBinaryObject exeFile = null;	210	IBinaryObject exeFile = null;
210	if ( exePath != null ) {	211	if ( exePath != null ) {
211	exeFile = verifyBinary( project, exePath );	212	exeFile = verifyBinary( project, exePath );
Lines 273-279 Link Here
273	ICDebugConfiguration debugConfig = getDebugConfig( config );	274	ICDebugConfiguration debugConfig = getDebugConfig( config );
274	String path = config.getAttribute( ICDTLaunchConfigurationConstants.ATTR_COREFILE_PATH, (String)null );	275	String path = config.getAttribute( ICDTLaunchConfigurationConstants.ATTR_COREFILE_PATH, (String)null );
275	if ( path == null \|\| path.length() == 0) {	276	if ( path == null \|\| path.length() == 0) {
276	ICProject project = verifyCProject( config );	277	ICProject project = CDebugUtils.verifyCProject( config );
277	IPath corefile = promptForCoreFilePath( (IProject)project.getResource(), debugConfig );	278	IPath corefile = promptForCoreFilePath( (IProject)project.getResource(), debugConfig );
278	if ( corefile == null ) {	279	if ( corefile == null ) {
279	cancel( LaunchMessages.getString( "LocalCDILaunchDelegate.6" ), ICDTLaunchConfigurationConstants.ERR_NO_COREFILE ); //$NON-NLS-1$	280	cancel( LaunchMessages.getString( "LocalCDILaunchDelegate.6" ), ICDTLaunchConfigurationConstants.ERR_NO_COREFILE ); //$NON-NLS-1$
Lines 289-296 Link Here
289	cancel( "", -1 ); //$NON-NLS-1$	290	cancel( "", -1 ); //$NON-NLS-1$
290	}	291	}
291		292
292	IPath exePath = verifyProgramPath( config );	293	IPath exePath = CDebugUtils.verifyProgramPath( config );
293	ICProject project = verifyCProject( config );	294	ICProject project = CDebugUtils.verifyCProject( config );
294	IBinaryObject exeFile = null;	295	IBinaryObject exeFile = null;
295	if ( exePath != null ) {	296	if ( exePath != null ) {
296	exeFile = verifyBinary( project, exePath );	297	exeFile = verifyBinary( project, exePath );
Lines 329-336 Link Here
329		330
330	private ICDISession launchOldDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger debugger, IProgressMonitor monitor ) throws CoreException {	331	private ICDISession launchOldDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger debugger, IProgressMonitor monitor ) throws CoreException {
331	IBinaryObject exeFile = null;	332	IBinaryObject exeFile = null;
332	IPath exePath = verifyProgramPath( config );	333	IPath exePath = CDebugUtils.verifyProgramPath( config );
333	ICProject project = verifyCProject( config );	334	ICProject project = CDebugUtils.verifyCProject( config );
334	if ( exePath != null ) {	335	if ( exePath != null ) {
335	exeFile = verifyBinary( project, exePath );	336	exeFile = verifyBinary( project, exePath );
336	}	337	}
Lines 338-344 Link Here
338	}	339	}
339		340
340	private ICDISession launchDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger2 debugger, IProgressMonitor monitor ) throws CoreException {	341	private ICDISession launchDebugSession( ILaunchConfiguration config, ILaunch launch, ICDIDebugger2 debugger, IProgressMonitor monitor ) throws CoreException {
341	IPath path = verifyProgramPath( config );	342	IPath path = CDebugUtils.verifyProgramPath( config );
342	File exeFile = path != null ? path.toFile() : null;	343	File exeFile = path != null ? path.toFile() : null;
343	return debugger.createSession( launch, exeFile, monitor );	344	return debugger.createSession( launch, exeFile, monitor );
344	}	345	}

Lines 12-21 Link Here

(-)src/org/eclipse/cdt/debug/core/CDebugUtils.java (-1 / +128 lines)
12	package org.eclipse.cdt.debug.core;	12	package org.eclipse.cdt.debug.core;
13		13
14	import java.io.ByteArrayOutputStream;	14	import java.io.ByteArrayOutputStream;
		15	import java.io.FileNotFoundException;
15	import java.io.IOException;	16	import java.io.IOException;
16	import java.nio.charset.Charset;	17	import java.nio.charset.Charset;
17	import java.nio.charset.CharsetDecoder;	18	import java.nio.charset.CharsetDecoder;
18	import com.ibm.icu.text.MessageFormat;
19	import java.util.ArrayList;	19	import java.util.ArrayList;
20	import java.util.Arrays;	20	import java.util.Arrays;
21	import java.util.Iterator;	21	import java.util.Iterator;
Lines 40-45 Link Here
40	import org.eclipse.cdt.debug.core.model.ICWatchpoint;	40	import org.eclipse.cdt.debug.core.model.ICWatchpoint;
41	import org.eclipse.cdt.debug.core.model.ICWatchpoint2;	41	import org.eclipse.cdt.debug.core.model.ICWatchpoint2;
42	import org.eclipse.cdt.debug.internal.core.model.CFloatingPointValue;	42	import org.eclipse.cdt.debug.internal.core.model.CFloatingPointValue;
		43	import org.eclipse.core.resources.IFile;
43	import org.eclipse.core.resources.IMarker;	44	import org.eclipse.core.resources.IMarker;
44	import org.eclipse.core.resources.IProject;	45	import org.eclipse.core.resources.IProject;
45	import org.eclipse.core.resources.ResourcesPlugin;	46	import org.eclipse.core.resources.ResourcesPlugin;
Lines 47-58 Link Here
47	import org.eclipse.core.runtime.IPath;	48	import org.eclipse.core.runtime.IPath;
48	import org.eclipse.core.runtime.IStatus;	49	import org.eclipse.core.runtime.IStatus;
49	import org.eclipse.core.runtime.Path;	50	import org.eclipse.core.runtime.Path;
		51	import org.eclipse.core.runtime.Status;
50	import org.eclipse.debug.core.DebugPlugin;	52	import org.eclipse.debug.core.DebugPlugin;
51	import org.eclipse.debug.core.ILaunchConfiguration;	53	import org.eclipse.debug.core.ILaunchConfiguration;
52	import org.eclipse.debug.core.IStatusHandler;	54	import org.eclipse.debug.core.IStatusHandler;
53	import org.eclipse.debug.core.model.IBreakpoint;	55	import org.eclipse.debug.core.model.IBreakpoint;
54	import org.w3c.dom.Document;	56	import org.w3c.dom.Document;
55		57
		58	import com.ibm.icu.text.MessageFormat;
		59
56	/**	60	/**
57	* Utility methods.	61	* Utility methods.
58	*/	62	*/
Lines 561-564 Link Here
561	}	565	}
562	return new Path(path);	566	return new Path(path);
563	}	567	}
		568
		569	/**
		570	* Returns the ICProject associated with the project setting in the Main tab
		571	* of a CDT launch configuration, or throws a CoreException providing a
		572	* reason (e.g., the setting is empty, the project no longer exists, the
		573	* isn't a CDT one, etc).
		574	*
		575	* @param config
		576	* the launch configuration
		577	* @return an ICProject; never null.
		578	* @throws CoreException
		579	* @since 7.0
		580	*/
		581	public static ICProject verifyCProject(ILaunchConfiguration config) throws CoreException {
		582	String name = CDebugUtils.getProjectName(config);
		583	if (name == null) {
		584	throwCoreException(DebugCoreMessages.getString("CDebugUtils.C_Project_not_specified"), //$NON-NLS-1$
		585	ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROJECT);
		586	}
		587	ICProject cproject = CDebugUtils.getCProject(config);
		588	if (cproject == null) {
		589	IProject proj = ResourcesPlugin.getWorkspace().getRoot().getProject(name);
		590	if (!proj.exists()) {
		591	throwCoreException(DebugCoreMessages.getFormattedString("CDebugUtils.Project_NAME_does_not_exist", name), //$NON-NLS-1$
		592	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
		593	} else if (!proj.isOpen()) {
		594	throwCoreException(DebugCoreMessages.getFormattedString("CDebugUtils.Project_NAME_is_closed", name), //$NON-NLS-1$
		595	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
		596	}
		597	throwCoreException(DebugCoreMessages.getString("CDebugUtils.Not_a_C_CPP_project"), //$NON-NLS-1$
		598	ICDTLaunchConfigurationConstants.ERR_NOT_A_C_PROJECT);
		599	}
		600	return cproject;
		601	}
		602
		603	/**
		604	* Returns an IPath for the file referenced in the <i>C/C++ Application</i>
		605	* setting of a CDT launch configuration. Typically, the file is obtained by
		606	* combining the <i>C/C++ Application</i> setting with the <i>Project</i>
		607	* setting. If unable to combine and resolve these settings to a valid file,
		608	* a CoreException is thrown that provides the reason why. There are many
		609	* such possible reasons (a problem with the <i>Project</i> setting, an
		610	* empty <i>C/C++ Application</i> setting, the combined settings doesn't
		611	* resolve to an existing file, etc).
		612	*
		613	* @param config
		614	* the launch configuration
		615	* @param ignoreProjectSetting
		616	* if true, resolve the file using only the <i>C/C++
		617	* Application</i> setting. Do not take the <i>Project</i>
		618	* setting into account.
		619	* @return an IPath; never null
		620	* @throws CoreException
		621	* @since 7.0
		622	*/
		623	public static IPath verifyProgramPath(ILaunchConfiguration config, boolean ignoreProjectSetting) throws CoreException {
		624	ICProject cproject = null;
		625	if (!ignoreProjectSetting) {
		626	cproject = verifyCProject(config); // will throw exception if project setting not valid
		627	}
		628	IPath programPath = CDebugUtils.getProgramPath(config);
		629	if (programPath == null \|\| programPath.isEmpty()) {
		630	throwCoreException(DebugCoreMessages.getString("CDebugUtils.Program_file_not_specified"), //$NON-NLS-1$
		631	ICDTLaunchConfigurationConstants.ERR_UNSPECIFIED_PROGRAM);
632	}
633
634	if (programPath != null) { // this check is here only to avoid warning; compiler can't tell we'll throw an exception above
635	if (!programPath.isAbsolute() && (cproject != null)) {
636	// See if we can brute-force append the program path to the
637	// project location. This allows us to support the program file
638	// being outside the project, even outside the workspace, without
639	// requiring a linked resource (e.g., the setting could be
640	// "..\..\some\dir\myprogram.exe")
641	IPath location = cproject.getProject().getLocation();
642	if (location != null) {
643	programPath = location.append(programPath);
644	if (!programPath.toFile().exists()) {
645	// Try looking in the project for the file. This
646	// supports linked resources.
647	IFile projFile = null;
648	try {
649	projFile = cproject.getProject().getFile(CDebugUtils.getProgramPath(config));
650	}
651	catch (IllegalArgumentException exc) {
652	// thrown if relative path that resolves to a root file (e.g., "..\somefile")
653	}
654	if (projFile != null && projFile.exists()) {
655	programPath = projFile.getLocation();
656	}
657	}
658	}
659	}
660	if (!programPath.toFile().exists()) {
661	throwCoreException(
662	DebugCoreMessages.getString("CDebugUtils.Program_file_does_not_exist"), //$NON-NLS-1$
663	new FileNotFoundException(
664	DebugCoreMessages.getFormattedString("CDebugUtils.PROGRAM_PATH_not_found", programPath.toOSString())), //$NON-NLS-1$
665	ICDTLaunchConfigurationConstants.ERR_PROGRAM_NOT_EXIST);
666	}
667	}
668	return programPath;
669	}
670
671	/**
672	* Variant that expects (requires) the launch configuration to have a valid
673	* <i>Project</i> setting. See
674	* {@link #verifyProgramPath(ILaunchConfiguration, boolean)}
675	*
676	* @since 7.0
677	*/
678	public static IPath verifyProgramPath(ILaunchConfiguration config) throws CoreException {
679	return verifyProgramPath(config, false);
680	}
681
682	/** Throws a CoreException. Clutter-reducing utility method. */
683	private static void throwCoreException(String msg, int code) throws CoreException {
684	throwCoreException(msg, null, code);
685	}
686
687	/** Throws a CoreException. Clutter-reducing utility method. */
688	private static void throwCoreException(String msg, Exception innerException, int code) throws CoreException {
689	throw new CoreException(new Status(IStatus.ERROR, CDebugCorePlugin.getUniqueIdentifier(), code, msg, innerException));
690	}
564	}	691	}

Lines 22-27 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/core/GDBJtagDSFFinalLaunchSequence.java (-72 / +131 lines)
22	import java.util.List;	22	import java.util.List;
23	import java.util.Properties;	23	import java.util.Properties;
24		24
		25	import org.eclipse.cdt.debug.core.CDebugUtils;
25	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContribution;	26	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContribution;
26	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContributionFactory;	27	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.GDBJtagDeviceContributionFactory;
27	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.IGDBJtagDevice;	28	import org.eclipse.cdt.debug.gdbjtag.core.jtagdevice.IGDBJtagDevice;
Lines 345-351 Link Here
345	// Below steps are specific to JTag hardware debugging	346	// Below steps are specific to JTag hardware debugging
346		347
347	/*	348	/*
348	* Retrieve the JTag device	349	* Retrieve the IGDBJtagDevice instance
349	*/	350	*/
350	new Step() {	351	new Step() {
351	@Override	352	@Override
Lines 365-370 Link Here
365	rm.done();	366	rm.done();
366	}},	367	}},
367	/*	368	/*
		369	* Execute symbol loading
		370	*/
		371	new Step() {
		372	@Override
		373	public void execute(RequestMonitor rm) {
		374	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
		375	try {
		376	if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS)) {
		377	String symbolsFileName = null;
		378
		379	// New setting in Helios. Default is true. Check for existence
		380	// in order to support older launch configs
		381	if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS) &&
		382	config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_SYMBOLS, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_SYMBOLS)) {
		383	IPath programFile = CDebugUtils.verifyProgramPath(config);
		384	if (programFile != null) {
		385	symbolsFileName = programFile.toOSString();
		386	}
		387	}
		388	else {
		389	symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, IGDBJtagConstants.DEFAULT_SYMBOLS_FILE_NAME);
		390	if (symbolsFileName.length() > 0) {
		391	symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName);
		392	} else {
		393	symbolsFileName = null;
		394	}
		395	}
		396
		397	if (symbolsFileName == null) {
		398	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, Messages.getString("GDBJtagDebugger.err_no_img_file"), null)); //$NON-NLS-1$
		399	rm.done();
		400	return;
		401	}
		402
		403	// Escape windows path separator characters TWICE, once for Java and once for GDB.
		404	symbolsFileName = symbolsFileName.replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$
		405
		406	String symbolsOffset = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, IGDBJtagConstants.DEFAULT_SYMBOLS_OFFSET);
		407	if (symbolsOffset.length() > 0) {
		408	symbolsOffset = "0x" + symbolsOffset;
		409	}
		410	List<String> commands = new ArrayList<String>();
		411	fGdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
		412	queueCommands(commands, rm);
		413
		414	} else {
		415	rm.done();
		416	}
		417	} catch (CoreException e) {
		418	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot load symbol", e)); //$NON-NLS-1$
		419	rm.done();
		420	}
		421	}},
		422
		423	/*
368	* Hook up to remote target	424	* Hook up to remote target
369	*/	425	*/
370	new Step() {	426	new Step() {
Lines 394-400 Link Here
394	public void execute(RequestMonitor rm) {	450	public void execute(RequestMonitor rm) {
395	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	451	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
396	try {	452	try {
397	if (config.getAttribute(IGDBJtagConstants.ATTR_DO_RESET, true)) {	453	if (config.getAttribute(IGDBJtagConstants.ATTR_DO_RESET, IGDBJtagConstants.DEFAULT_DO_RESET)) {
398	List<String> commands = new ArrayList<String>();	454	List<String> commands = new ArrayList<String>();
399	fGdbJtagDevice.doReset(commands);	455	fGdbJtagDevice.doReset(commands);
400	queueCommands(commands, rm);	456	queueCommands(commands, rm);
Lines 431-437 Link Here
431	public void execute(RequestMonitor rm) {	487	public void execute(RequestMonitor rm) {
432	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	488	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
433	try {	489	try {
434	if (config.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, true)) {	490	if (config.getAttribute(IGDBJtagConstants.ATTR_DO_HALT, IGDBJtagConstants.DEFAULT_DO_HALT)) {
435	List<String> commands = new ArrayList<String>();	491	List<String> commands = new ArrayList<String>();
436	fGdbJtagDevice.doHalt(commands);	492	fGdbJtagDevice.doHalt(commands);
437	queueCommands(commands, rm);	493	queueCommands(commands, rm);
Lines 451-466 Link Here
451	public void execute(RequestMonitor rm) {	507	public void execute(RequestMonitor rm) {
452	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	508	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
453	try {	509	try {
454	String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, ""); //$NON-NLS-1$	510	String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_INIT_COMMANDS, IGDBJtagConstants.DEFAULT_INIT_COMMANDS);
455	userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);	511	userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);
456	String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$	512	if (userCmd.length() > 0) {
457		513	String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$
458	CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);	514
459	crm.setDoneCount(commands.length);	515	CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);
460	for (int i = 0; i < commands.length; ++i) {	516	crm.setDoneCount(commands.length);
461	fCommandControl.queueCommand(	517	for (int i = 0; i < commands.length; ++i) {
462	new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),	518	fCommandControl.queueCommand(
463	new DataRequestMonitor<MIInfo>(getExecutor(), crm));	519	new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),
		520	new DataRequestMonitor<MIInfo>(getExecutor(), crm));
		521	}
		522	}
		523	else {
		524	rm.done();
464	}	525	}
465	} catch (CoreException e) {	526	} catch (CoreException e) {
466	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined init commands", e)); //$NON-NLS-1$	527	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined init commands", e)); //$NON-NLS-1$
Lines 475-494 Link Here
475	public void execute(RequestMonitor rm) {	536	public void execute(RequestMonitor rm) {
476	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	537	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
477	try {	538	try {
		539	String imageFileName = null;
478	if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE)) {	540	if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_IMAGE, IGDBJtagConstants.DEFAULT_LOAD_IMAGE)) {
479	// Escape windows path separator characters TWICE, once for Java and once for GDB.	541	// New setting in Helios. Default is true. Check for existence
480	String imageFileName = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, ""); //$NON-NLS-1$	542	// in order to support older launch configs
481	if (imageFileName.length() > 0) {	543	if (config.hasAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE) &&
482	imageFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(imageFileName).replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$	544	config.getAttribute(IGDBJtagConstants.ATTR_USE_PROJ_BINARY_FOR_IMAGE, IGDBJtagConstants.DEFAULT_USE_PROJ_BINARY_FOR_IMAGE)) {
483	String imageOffset = (imageFileName.endsWith(".elf")) ? "" : "0x" + config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, ""); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$ //$NON-NLS-4$	545	IPath programFile = CDebugUtils.verifyProgramPath(config);
484	List<String> commands = new ArrayList<String>();	546	if (programFile != null) {
485	fGdbJtagDevice.doLoadImage(imageFileName, imageOffset, commands);	547	imageFileName = programFile.toOSString();
486	queueCommands(commands, rm);	548	}
487	} else {
488	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Image name cannot be empty", null)); //$NON-NLS-1$
489	rm.done();
490	}	549	}
491	} else {	550	else {
		551	imageFileName = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_FILE_NAME, IGDBJtagConstants.DEFAULT_IMAGE_FILE_NAME); //$NON-NLS-1$
		552	if (imageFileName.length() > 0) {
		553	imageFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(imageFileName);
		554	} else {
		555	imageFileName = null;
		556	}
		557	}
		558
		559	if (imageFileName == null) {
		560	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, Messages.getString("GDBJtagDebugger.err_no_img_file"), null)); //$NON-NLS-1$
		561	rm.done();
		562	return;
		563	}
		564
		565	// Escape windows path separator characters TWICE, once for Java and once for GDB.
		566	imageFileName = imageFileName.replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$
		567
		568	String imageOffset = config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET);
		569	if (imageOffset.length() > 0) {
		570	imageOffset = (imageFileName.endsWith(".elf")) ? "" : "0x" + config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_IMAGE_OFFSET); //$NON-NLS-2$ //$NON-NLS-4$
		571	}
		572	List<String> commands = new ArrayList<String>();
		573	fGdbJtagDevice.doLoadImage(imageFileName, imageOffset, commands);
		574	queueCommands(commands, rm);
		575	}
		576	else {
492	rm.done();	577	rm.done();
493	}	578	}
494	} catch (CoreException e) {	579	} catch (CoreException e) {
Lines 496-531 Link Here
496	rm.done();	581	rm.done();
497	}	582	}
498	}},	583	}},
499	/*
500	* Execute symbol loading
501	*/
502	new Step() {
503	@Override
504	public void execute(RequestMonitor rm) {
505	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
506	try {
507	if (config.getAttribute(IGDBJtagConstants.ATTR_LOAD_SYMBOLS, IGDBJtagConstants.DEFAULT_LOAD_SYMBOLS)) {
508	// Escape windows path separator characters TWICE, once for Java and once for GDB.
509	String symbolsFileName = config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_FILE_NAME, ""); //$NON-NLS-1$
510	if (symbolsFileName.length() > 0) {
511	symbolsFileName = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(symbolsFileName).replace("\\", "\\\\"); //$NON-NLS-1$ //$NON-NLS-2$
512	String symbolsOffset = "0x" + config.getAttribute(IGDBJtagConstants.ATTR_SYMBOLS_OFFSET, ""); //$NON-NLS-1$ //$NON-NLS-2$
513	List<String> commands = new ArrayList<String>();
514	fGdbJtagDevice.doLoadSymbol(symbolsFileName, symbolsOffset, commands);
515	queueCommands(commands, rm);
516	} else {
517	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Symbol name cannot be empty", null)); //$NON-NLS-1$
518	rm.done();
519	}
520	} else {
521	rm.done();
522	}
523	} catch (CoreException e) {
524	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot load symbol", e)); //$NON-NLS-1$
525	rm.done();
526	}
527	}
528	},
529	/*	584	/*
530	* Start tracking the breakpoints once we know we are connected to the target (necessary for remote debugging)	585	* Start tracking the breakpoints once we know we are connected to the target (necessary for remote debugging)
531	*/	586	*/
Lines 550-556 Link Here
550	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	605	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
551	try {	606	try {
552	if (config.getAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER)) {	607	if (config.getAttribute(IGDBJtagConstants.ATTR_SET_PC_REGISTER, IGDBJtagConstants.DEFAULT_SET_PC_REGISTER)) {
553	String pcRegister = config.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, "")); //$NON-NLS-1$	608	String pcRegister = config.getAttribute(IGDBJtagConstants.ATTR_PC_REGISTER, config.getAttribute(IGDBJtagConstants.ATTR_IMAGE_OFFSET, IGDBJtagConstants.DEFAULT_PC_REGISTER)); //$NON-NLS-1$
554	List<String> commands = new ArrayList<String>();	609	List<String> commands = new ArrayList<String>();
555	fGdbJtagDevice.doSetPC(pcRegister, commands);	610	fGdbJtagDevice.doSetPC(pcRegister, commands);
556	queueCommands(commands, rm);	611	queueCommands(commands, rm);
Lines 571-577 Link Here
571	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	626	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
572	try {	627	try {
573	if (config.getAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT)) {	628	if (config.getAttribute(IGDBJtagConstants.ATTR_SET_STOP_AT, IGDBJtagConstants.DEFAULT_SET_STOP_AT)) {
574	String stopAt = config.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, ""); //$NON-NLS-1$	629	String stopAt = config.getAttribute(IGDBJtagConstants.ATTR_STOP_AT, IGDBJtagConstants.DEFAULT_STOP_AT); //$NON-NLS-1$
575	List<String> commands = new ArrayList<String>();	630	List<String> commands = new ArrayList<String>();
576	fGdbJtagDevice.doStopAt(stopAt, commands);	631	fGdbJtagDevice.doStopAt(stopAt, commands);
577	queueCommands(commands, rm);	632	queueCommands(commands, rm);
Lines 612-627 Link Here
612	public void execute(RequestMonitor rm) {	667	public void execute(RequestMonitor rm) {
613	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();	668	ILaunchConfiguration config = fLaunch.getLaunchConfiguration();
614	try {	669	try {
615	String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, ""); //$NON-NLS-1$	670	String userCmd = config.getAttribute(IGDBJtagConstants.ATTR_RUN_COMMANDS, IGDBJtagConstants.DEFAULT_RUN_COMMANDS); //$NON-NLS-1$
616	userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);	671	if (userCmd.length() > 0) {
617	String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$	672	userCmd = VariablesPlugin.getDefault().getStringVariableManager().performStringSubstitution(userCmd);
618		673	String[] commands = userCmd.split("\\r?\\n"); //$NON-NLS-1$
619	CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);	674
620	crm.setDoneCount(commands.length);	675	CountingRequestMonitor crm = new CountingRequestMonitor(getExecutor(), rm);
621	for (int i = 0; i < commands.length; ++i) {	676	crm.setDoneCount(commands.length);
622	fCommandControl.queueCommand(	677	for (int i = 0; i < commands.length; ++i) {
623	new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),	678	fCommandControl.queueCommand(
624	new DataRequestMonitor<MIInfo>(getExecutor(), crm));	679	new CLICommand<MIInfo>(fCommandControl.getContext(), commands[i]),
		680	new DataRequestMonitor<MIInfo>(getExecutor(), crm));
		681	}
		682	}
		683	else {
		684	rm.done();
625	}	685	}
626	} catch (CoreException e) {	686	} catch (CoreException e) {
627	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined run commands", e)); //$NON-NLS-1$	687	rm.setStatus(new Status(IStatus.ERROR, Activator.PLUGIN_ID, -1, "Cannot run user defined run commands", e)); //$NON-NLS-1$
Lines 687-698 Link Here
687	private IGDBJtagDevice getGDBJtagDevice (ILaunchConfiguration config)	747	private IGDBJtagDevice getGDBJtagDevice (ILaunchConfiguration config)
688	throws CoreException, NullPointerException {	748	throws CoreException, NullPointerException {
689	IGDBJtagDevice gdbJtagDevice = null;	749	IGDBJtagDevice gdbJtagDevice = null;
690	String jtagDeviceName = config.getAttribute(IGDBJtagConstants.ATTR_JTAG_DEVICE, ""); //$NON-NLS-1$	750	String jtagDeviceName = config.getAttribute(IGDBJtagConstants.ATTR_JTAG_DEVICE, IGDBJtagConstants.DEFAULT_JTAG_DEVICE); //$NON-NLS-1$
691	GDBJtagDeviceContribution[] availableDevices = GDBJtagDeviceContributionFactory.	751	GDBJtagDeviceContribution[] availableDevices = GDBJtagDeviceContributionFactory.getInstance().getGDBJtagDeviceContribution();
692	getInstance().getGDBJtagDeviceContribution();	752	for (GDBJtagDeviceContribution availableDevice : availableDevices) {
693	for (int i = 0; i < availableDevices.length; i++) {	753	if (jtagDeviceName.equals(availableDevice.getDeviceName())) {
694	if (jtagDeviceName.equals(availableDevices[i].getDeviceName())) {	754	gdbJtagDevice = availableDevice.getDevice();
695	gdbJtagDevice = availableDevices[i].getDevice();
696	break;	755	break;
697	}	756	}
698	}	757	}

Lines 72-79 Link Here

(-)src/org/eclipse/cdt/debug/gdbjtag/core/jtagdevice/DefaultGDBJtagDeviceImpl.java (-4 / +21 lines)
72	*/	72	*/
73	public void doLoadImage(String imageFileName, String imageOffset, Collection commands) {	73	public void doLoadImage(String imageFileName, String imageOffset, Collection commands) {
74	String file = escapeScpaces(imageFileName);	74	String file = escapeScpaces(imageFileName);
75	String cmd = "restore " + file + " " + imageOffset;	75
76	addCmd(commands, cmd);	76	if (imageOffset.length() > 0) {
		77	// 'restore' simply puts the program into memory.
		78	addCmd(commands, "restore " + file + " " + imageOffset);
		79	}
		80	else {
		81	// 'load' puts the program into memory and sets the PC. To see why
		82	// we do this when no offset is specified, see
		83	// https://bugs.eclipse.org/bugs/show_bug.cgi?id=310304#c20
		84	addCmd(commands, "load " + file);
		85	}
		86
		87	// 'exec-file' specifies the program as the context for getting memory.
		88	// Basically, it tells gdb "this is the program we'll be debugging"
		89	addCmd(commands, "exec-file " + file);
77	}	90	}
78		91
79	/* (non-Javadoc)	92	/* (non-Javadoc)
Lines 81-88 Link Here
81	*/	94	*/
82	public void doLoadSymbol(String symbolFileName, String symbolOffset, Collection commands) {	95	public void doLoadSymbol(String symbolFileName, String symbolOffset, Collection commands) {
83	String file = escapeScpaces(symbolFileName);	96	String file = escapeScpaces(symbolFileName);
84	String cmd = "add-sym " + file + " " + symbolOffset;	97	if (symbolOffset == null \|\| (symbolOffset.length() == 0)) {
85	addCmd(commands, cmd);	98	addCmd(commands, "symbol-file " + file);
		99	}
		100	else {
		101	addCmd(commands, "add-sym " + file + " " + symbolOffset);
		102	}
86	}	103	}
87		104
88	protected String escapeScpaces(String file) {	105	protected String escapeScpaces(String file) {