Author: [EMAIL PROTECTED]
Date: Mon Oct  6 17:10:48 2008
New Revision: 3723

Modified:
    changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/GWTShell.java

Log:
Correct the launch URL in the case of a pre-existing query parameter.


Modified:  
changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/GWTShell.java
==============================================================================
--- changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/GWTShell.java      
 
(original)
+++ changes/jat/oophm-branch/dev/core/src/com/google/gwt/dev/GWTShell.java      
 
Mon Oct  6 17:10:48 2008
@@ -502,7 +502,13 @@
       * TODO(jat): properly support launching arbitrary browsers; waiting on
       * Freeland's work with BrowserScanner and the trunk merge to get it.
       */
-    url += "?gwt.hosted=" + listener.getEndpointIdentifier();
+    String separator;
+    if (url.contains("?")) {
+      separator = "&";
+    } else {
+      separator = "?";
+    }
+    url += separator + "gwt.hosted=" + listener.getEndpointIdentifier();
      TreeLogger branch = getTopLogger().branch(TreeLogger.INFO,
          "Launching firefox with " + url, null);
      try {

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

Reply via email to