[Rubyeclipse-commit] SF.net SVN: rubyeclipse: [2285] trunk/org.rubypeople.rdt.launching/src/org/ rubypeople/rdt/internal/launching/StandardVMRunner.java

[<caw...@us...>]

Revision: 2285
          http://svn.sourceforge.net/rubyeclipse/?rev=2285&view=rev
Author:   cawilliams
Date:     2007-04-05 11:03:13 -0700 (Thu, 05 Apr 2007)

Log Message:
-----------
revert setting stdout and stderr to sync/auto-flush, it messes up __FILE__ global and breaks the if __FILE__ == $0 idiom

Modified Paths:
--------------
    trunk/org.rubypeople.rdt.launching/src/org/rubypeople/rdt/internal/launching/StandardVMRunner.java

Modified: trunk/org.rubypeople.rdt.launching/src/org/rubypeople/rdt/internal/launching/StandardVMRunner.java
===================================================================
--- trunk/org.rubypeople.rdt.launching/src/org/rubypeople/rdt/internal/launching/StandardVMRunner.java	2007-04-05 18:02:49 UTC (rev 2284)
+++ trunk/org.rubypeople.rdt.launching/src/org/rubypeople/rdt/internal/launching/StandardVMRunner.java	2007-04-05 18:03:13 UTC (rev 2285)
@@ -195,13 +195,13 @@
 		// options like '-client' & '-server' which are required to be the first option
 		String[] allVMArgs = combineVmArgs(config, fVMInstance);
 		addArguments(allVMArgs, arguments);
-		// FIXME Find a better place for this to go?
-		arguments.add("-e");
-		arguments.add("STDOUT.sync=true");
-		arguments.add("-e");
-		arguments.add("STDERR.sync=true");
-		arguments.add("-e");
-		arguments.add("load($0=ARGV.shift)");
+		// FIXME Find a way to set stderr and stdout to sync/auto-flush without messing up value of __FILE__ (becomes absolute which messes up the 'if __FILE__ == $0' idiom)
+//		arguments.add("-e");
+//		arguments.add("STDOUT.sync=true");
+//		arguments.add("-e");
+//		arguments.add("STDERR.sync=true");
+//		arguments.add("-e");
+//		arguments.add("load($0=ARGV.shift)");
 		
 		String[] lp= config.getLoadPath();
 		if (lp.length > 0) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.