[Pydev-cvs] org.python.pydev.debug/src/org/python/pydev/debug/ui/launching PythonRunner.java, 1.35, 1.36 PythonRunnerConfig.java, 1.65, 1.66

[Fabio Z. <fa...@us...>]

Update of /cvsroot/pydev/org.python.pydev.debug/src/org/python/pydev/debug/ui/launching
In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv4564/src/org/python/pydev/debug/ui/launching

Modified Files:
	PythonRunner.java PythonRunnerConfig.java 
Log Message:
- No longer using Runtime.exec(String), only Runtime.exec(String[])
- Updating the markers in a better (faster) way

Index: PythonRunner.java
===================================================================
RCS file: /cvsroot/pydev/org.python.pydev.debug/src/org/python/pydev/debug/ui/launching/PythonRunner.java,v
retrieving revision 1.35
retrieving revision 1.36
diff -C2 -d -r1.35 -r1.36
*** PythonRunner.java	28 Apr 2008 23:03:16 -0000	1.35
--- PythonRunner.java	21 May 2008 01:38:15 -0000	1.36
***************
*** 171,175 ****
          // Launch & connect to the debugger		
          subMonitor.subTask("Constructing command_line...");
!         String commandLineAsString = SimpleRunner.getCommandLineAsString(cmdLine);
          //System.out.println("running command line: "+commandLineAsString);
          Map<Object, Object> processAttributes = new HashMap<Object, Object>();
--- 171,175 ----
          // Launch & connect to the debugger		
          subMonitor.subTask("Constructing command_line...");
!         String commandLineAsString = SimpleRunner.getArgumentsAsStr(cmdLine);
          //System.out.println("running command line: "+commandLineAsString);
          Map<Object, Object> processAttributes = new HashMap<Object, Object>();

Index: PythonRunnerConfig.java
===================================================================
RCS file: /cvsroot/pydev/org.python.pydev.debug/src/org/python/pydev/debug/ui/launching/PythonRunnerConfig.java,v
retrieving revision 1.65
retrieving revision 1.66
diff -C2 -d -r1.65 -r1.66
*** PythonRunnerConfig.java	13 Apr 2008 16:55:17 -0000	1.65
--- PythonRunnerConfig.java	21 May 2008 01:38:15 -0000	1.66
***************
*** 612,616 ****
  
  	
! 	
      private String[] getVMArguments(ILaunchConfiguration configuration) throws CoreException {
      	String args = configuration.getAttribute(Constants.ATTR_VM_ARGUMENTS, (String) null);
--- 612,619 ----
  
  	
! 	/**
! 	 * @return an array with the vm arguments in the given configuration.
! 	 * @throws CoreException
! 	 */
      private String[] getVMArguments(ILaunchConfiguration configuration) throws CoreException {
      	String args = configuration.getAttribute(Constants.ATTR_VM_ARGUMENTS, (String) null);
***************
*** 622,630 ****
  	}
  
  	public String getCommandLineAsString() throws JDTNotAvailableException {
  		String[] args;
          try {
              args = getCommandLine(false);
!             return SimpleRunner.getCommandLineAsString(args);
          } catch (CoreException e) {
              throw new RuntimeException(e);
--- 625,638 ----
  	}
  
+     /**
+      * @return A command line to be shown to the user. Note that this command line should not actually be used for
+      * an execution (only String[] should be passed to Runtie.exec)
+      * @throws JDTNotAvailableException
+      */
  	public String getCommandLineAsString() throws JDTNotAvailableException {
  		String[] args;
          try {
              args = getCommandLine(false);
!             return SimpleRunner.getArgumentsAsStr(args);
          } catch (CoreException e) {
              throw new RuntimeException(e);