From f29a101537501789e01405172b366297319b9a8a Mon Sep 17 00:00:00 2001
From: Michael Jumper <zhangmaike@users.sourceforge.net>
Date: Sun, 11 Sep 2011 16:10:38 -0700
Subject: [PATCH] Partial implementation of new format instructions in output.

---
 .../src/main/resources/guacamole.js           | 34 ++++---------------
 .../src/main/resources/tunnel.js              | 29 +++++++++++++---
 2 files changed, 30 insertions(+), 33 deletions(-)

diff --git a/guacamole-common-js/src/main/resources/guacamole.js b/guacamole-common-js/src/main/resources/guacamole.js
index 0e5a5af1c..ec925f415 100644
--- a/guacamole-common-js/src/main/resources/guacamole.js
+++ b/guacamole-common-js/src/main/resources/guacamole.js
@@ -102,7 +102,7 @@ Guacamole.Client = function(display, tunnel) {
         if (!isConnected())
             return;
 
-        tunnel.sendMessage("key:" +  keysym + "," + pressed + ";");
+        tunnel.sendMessage("key", keysym, pressed);
     };
 
     guac_client.sendMouseState = function(mouseState) {
@@ -128,7 +128,7 @@ Guacamole.Client = function(display, tunnel) {
         if (mouseState.down)   buttonMask |= 16;
 
         // Send message
-        tunnel.sendMessage("mouse:" + mouseState.x + "," + mouseState.y + "," + buttonMask + ";");
+        tunnel.sendMessage("mouse", mouseState.x, mouseState.y, buttonMask);
     };
 
     guac_client.setClipboard = function(data) {
@@ -137,7 +137,7 @@ Guacamole.Client = function(display, tunnel) {
         if (!isConnected())
             return;
 
-        tunnel.sendMessage("clipboard:" + escapeGuacamoleString(data) + ";");
+        tunnel.sendMessage("clipboard", data);
     };
 
     // Handlers
@@ -384,7 +384,7 @@ Guacamole.Client = function(display, tunnel) {
 
                 // Send sync response when layers are finished
                 if (layersToSync == 0)
-                    tunnel.sendMessage("sync:" + timestamp + ";");
+                    tunnel.sendMessage("sync", timestamp);
 
             }
 
@@ -402,7 +402,7 @@ Guacamole.Client = function(display, tunnel) {
             // If all layers are ready, then we didn't install any hooks.
             // Send sync message now,
             if (layersToSync == 0)
-                tunnel.sendMessage("sync:" + timestamp + ";");
+                tunnel.sendMessage("sync", timestamp);
 
         }
       
@@ -425,35 +425,13 @@ Guacamole.Client = function(display, tunnel) {
                 && currentState != STATE_DISCONNECTING) {
 
             setState(STATE_DISCONNECTING);
-            tunnel.sendMessage("disconnect;");
+            tunnel.sendMessage("disconnect");
             tunnel.disconnect();
             setState(STATE_DISCONNECTED);
         }
 
     }
 
-    function escapeGuacamoleString(str) {
-
-        var escapedString = "";
-
-        for (var i=0; i<str.length; i++) {
-
-            var c = str.charAt(i);
-            if (c == ",")
-                escapedString += "\\c";
-            else if (c == ";")
-                escapedString += "\\s";
-            else if (c == "\\")
-                escapedString += "\\\\";
-            else
-                escapedString += c;
-
-        }
-
-        return escapedString;
-
-    }
-
     guac_client.disconnect = disconnect;
     guac_client.connect = function(data) {
 
diff --git a/guacamole-common-js/src/main/resources/tunnel.js b/guacamole-common-js/src/main/resources/tunnel.js
index a58c3be80..ae7a85c69 100644
--- a/guacamole-common-js/src/main/resources/tunnel.js
+++ b/guacamole-common-js/src/main/resources/tunnel.js
@@ -48,10 +48,10 @@ Guacamole.Tunnel = function() {
      * Send the given message through the tunnel to the service on the other
      * side. All messages are guaranteed to be received in the order sent.
      * 
-     * @param {String} message The message to send to the service on the other
-     *                         side of the tunnel.
+     * @param {...} elements The elements of the message to send to the
+     *                       service on the other side of the tunnel.
      */
-    this.sendMessage = function(message) {};
+    this.sendMessage = function(elements) {};
     
     /**
      * Fired whenever an error is encountered by the tunnel.
@@ -109,13 +109,32 @@ Guacamole.HTTPTunnel = function(tunnelURL) {
     var sendingMessages = false;
     var outputMessageBuffer = "";
 
-    this.sendMessage = function(message) {
+    this.sendMessage = function() {
 
         // Do not attempt to send messages if not connected
         if (currentState != STATE_CONNECTED)
             return;
 
-        // Add event to queue, restart send loop if finished.
+        // Do not attempt to send empty messages
+        if (arguments.length == 0)
+            return;
+
+        function getElement(value) {
+            var string = new String(value);
+            return string.length + "." + string; 
+        }
+
+        // Initialized message with first element
+        var message = getElement(arguments[0]);
+
+        // Append remaining elements
+        for (var i=1; i<arguments.length; i++)
+            message += "," + getElement(arguments[i]);
+
+        // Final terminator
+        message += ";";
+
+        // Add message to buffer, restart send loop if finished.
         outputMessageBuffer += message;
         if (!sendingMessages)
             sendPendingMessages();