cleanups

Author: emgee3

README.markdown

@@ -52,17 +52,18 @@ ddpclient.connect(function(error) {
  * Useful for debugging and learning the ddp protocol
  */
 ddpclient.on('message', function(msg) {
-	console.log("ddp message: " + msg);
-});	
+  console.log("ddp message: " + msg);
+});
 
 /* 
  * If you need to do something specific on close or errors.
  * (You can also disable auto_reconnect and call ddpclient.connect()
- *  when you are ready to re-connect.)
+ * when you are ready to re-connect.)
 */
 ddpclient.on('socket-close', function(code, message) {
   console.log("Close: %s %s", code, message);
 });
+
 ddpclient.on('socket-error', function(error) {
   console.log("Error: %j", error);
 });

examples/example.js

@@ -10,7 +10,7 @@ var ddpclient = new DDPClient({
 
 ddpclient.connect(function(error) {
   console.log('connected!');
-  
+
   if (error) {
     console.log('DDP connection error!');
     return;
@@ -30,7 +30,7 @@ ddpclient.connect(function(error) {
  * Useful for debugging and learning the ddp protocol
  */
 ddpclient.on('message', function(msg) {
-	console.log("ddp message: " + msg);
+  console.log("ddp message: " + msg);
 });
 
 /* 
@@ -41,6 +41,7 @@ ddpclient.on('message', function(msg) {
 ddpclient.on('socket-close', function(code, message) {
   console.log("Close: %s %s", code, message);
 });
+
 ddpclient.on('socket-error', function(error) {
   console.log("Error: %j", error);
-});
+});

lib/ddp-client.js

@@ -44,7 +44,7 @@ DDPClient.prototype._prepareHandlers = function() {
     };
 
     // if reconnecting, try existing DDP session  
-    // removed for now, per conversatino with sixolet on IRC,
+    // removed for now, per conversation with sixolet on IRC,
     // reconnect on server side still needs work 
 
     /*
@@ -76,7 +76,7 @@ DDPClient.prototype._recoverNetworkError = function() {
   if (self.auto_reconnect && ! self._connectionFailed && ! self._isClosing) {
     setTimeout(function() { self.connect(); }, self.auto_reconnect_timer);
   }
-}
+};
 
 ///////////////////////////////////////////////////////////////////////////
 // RAW, low level functions
@@ -93,6 +93,7 @@ DDPClient.prototype._send = function(data) {
     // This callback to avoid an exception being thrown.
     // if an error happened, the socket will throw an event and we will recover.
   });
+
 };
 
 // handle a message from the server
@@ -112,10 +113,10 @@ DDPClient.prototype._message = function(data, flags) {
 
   if (!data.msg) {
     return;
-  
+
   } else if (data.msg === 'failed') {
     self.emit('failed', data);
-    
+
   } else if (data.msg === 'connected') {
     self.session = data.session;
     self.emit('connected');
@@ -224,10 +225,10 @@ DDPClient.prototype.connect = function(connected) {
     self.addListener("connected", connected);
     self.addListener("failed", function(error) {
       self._connectionFailed = true;
-      connected(error)
+      connected(error);
     });
   }
-  
+
   // websocket
   var protocol = self.use_ssl ? 'wss://' : 'ws://';
   self.socket = new WebSocket(protocol + self.host + ':' + self.port + '/' + self.path);

test/ddp-client.js

@@ -1,5 +1,5 @@
 var assert = require('assert'),
-    sinon = require('sinon'),
+    sinon  = require('sinon'),
     rewire = require('rewire'),
     events = require('events');
 
@@ -26,7 +26,7 @@ describe("Connect to remote server", function() {
 
     assert(wsConstructor.calledOnce);
     assert(wsConstructor.calledWithNew());
-    assert(wsConstructor.call)
+    assert(wsConstructor.call);
     assert.deepEqual(wsConstructor.args, [['ws://localhost:3000/websocket']]);
   });
   it('should connect to the provided host', function() {
@@ -72,7 +72,7 @@ describe('Automatic reconnection', function() {
 describe("Network errors", function() {
   beforeEach(function() {
     prepareMocks();
-  })
+  });
 
   // For some weird reasons (hard to reproduce) it happens that we try to send a message and
   // get an exception throws at us because the connection is not opened anymore.