Revision: 6993
Author: amitman...@google.com
Date: Wed Nov 18 11:33:25 2009
Log: Remove the now unnecessary reference to -XstartOnFirstThread argument.

Patch by: amitmanjhi
Review by: jat


http://code.google.com/p/google-web-toolkit/source/detail?r=6993

Modified:
  /trunk/user/src/com/google/gwt/user/tools/App.launchsrc
  /trunk/user/src/com/google/gwt/user/tools/WebAppCreator.java
  /trunk/user/src/com/google/gwt/user/tools/project.ant.xmlsrc

=======================================
--- /trunk/user/src/com/google/gwt/user/tools/App.launchsrc     Fri Feb  6  
13:06:24 2009
+++ /trunk/user/src/com/google/gwt/user/tools/App.launchsrc     Wed Nov 18  
11:33:25 2009
@@ -7,7 +7,7 @@
  <listEntry value="&lt;?xml version=&quot;1.0&quot;  
encoding=&quot;UTF-8&quot;?&gt;&#10;&lt;runtimeClasspathEntry  
internalArchive=&quot;/@moduleShortName/src&quot; path=&quot;3&quot;  
type=&quot;2&quot;/&gt;&#10;"/>
  <listEntry value="&lt;?xml version=&quot;1.0&quot;  
encoding=&quot;UTF-8&quot;?&gt;&#10;&lt;runtimeClasspathEntry  
id=&quot;org.eclipse.jdt.launching.classpathentry.defaultClasspath&quot;&gt;&#10;&lt;memento
  
project=&quot;@moduleShortName&quot;/&gt;&#10;&lt;/runtimeClasspathEntry&gt;&#10;"/>
  </listAttribute>
-<stringAttribute key="org.eclipse.jdt.launching.VM_ARGUMENTS"  
value="-xmx2...@vmargs"/>
+<stringAttribute key="org.eclipse.jdt.launching.VM_ARGUMENTS"  
value="-Xmx256M"/>
  <stringAttribute key="org.eclipse.jdt.launching.PROGRAM_ARGUMENTS"  
value="-startupUrl @startupUrl&#10;@moduleName"/>
  <stringAttribute key="org.eclipse.jdt.launching.PROJECT_ATTR"  
value="@moduleShortName"/>
  <booleanAttribute key="org.eclipse.debug.core.appendEnvironmentVariables"  
value="true"/>
=======================================
--- /trunk/user/src/com/google/gwt/user/tools/WebAppCreator.java        Tue Nov 
 3  
13:02:30 2009
+++ /trunk/user/src/com/google/gwt/user/tools/WebAppCreator.java        Wed Nov 
18  
11:33:25 2009
@@ -227,10 +227,6 @@
            + "//EN\" \"http://google-web-toolkit.googlecode.com/svn/tags/";
            + About.getGwtVersionNum()  
+ "/distro-source/core/src/gwt-module.dtd\">";
      }
-
-    // Figure out what platform we're on
-    boolean isMacOsX = gwtDevPath.substring(gwtDevPath.lastIndexOf('/') +  
1).indexOf(
-        "mac") >= 0;

      // Compute module package and name.
      int pos = moduleName.lastIndexOf('.');
@@ -262,9 +258,6 @@
      replacements.put("@shellClass", DevMode.class.getName());
      replacements.put("@compileClass", Compiler.class.getName());
      replacements.put("@startupUrl", moduleShortName + ".html");
-    replacements.put("@antVmargs", isMacOsX
-        ? "\n<jvmarg value=\"-XstartOnFirstThread\"/>" : "");
-    replacements.put("@vmargs",  
isMacOsX ? "&#10;-XstartOnFirstThread" : "");
      replacements.put("@renameTo", moduleShortName.toLowerCase());

      String antEclipseRule = "";
=======================================
--- /trunk/user/src/com/google/gwt/user/tools/project.ant.xmlsrc        Fri Nov 
13  
16:42:27 2009
+++ /trunk/user/src/com/google/gwt/user/tools/project.ant.xmlsrc        Wed Nov 
18  
11:33:25 2009
@@ -37,7 +37,7 @@
          <path refid="project.class.path"/>
        </classpath>
        <!-- add jvmarg -Xss16M or similar if you see a StackOverflowError  
-->
-      <jvmarg value="-Xmx256M"/>@antVmargs
+      <jvmarg value="-Xmx256M"/>
        <!-- Additional arguments like -style PRETTY or -logLevel DEBUG -->
        <arg value="@moduleName"/>
      </java>
@@ -49,7 +49,7 @@
          <pathelement location="src"/>
          <path refid="project.class.path"/>
        </classpath>
-      <jvmarg value="-Xmx256M"/>@antVmargs
+      <jvmarg value="-Xmx256M"/>
        <arg value="-startupUrl"/>
        <arg value="@startupUrl"/>
        <!-- Additional arguments like -style PRETTY or -logLevel DEBUG -->

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to