cleanups

Author: emgee3

lib/ddp-client.js

@@ -31,7 +31,7 @@ DDPClient = function(opts) {
 
   // Expose EJSON object, so client can use EJSON.addType(...)
   if (self.use_ejson)
-    self.EJSON = EJSON
+    self.EJSON = EJSON;
 
   // very very simple collections (name -> [{id -> document}])
   if (self.maintain_collections)
@@ -107,13 +107,7 @@ DDPClient.prototype._send = function(data) {
 DDPClient.prototype._message = function(data) {
   var self = this;
 
-  // XXX: In theory data could be a Buffer (for binary frames), perhaps when
-  // there is streaming over DDP.  We should check that data is a String.
-  if (self.use_ejson) {
-    data = EJSON.parse(data);
-  } else {
-    data = JSON.parse(data);
-  }
+  data = self.use_ejson ? EJSON.parse(data) : JSON.parse(data);
 
   // TODO: 'addedBefore' -- not yet implemented in Meteor
   // TODO: 'movedBefore' -- not yet implemented in Meteor
@@ -225,7 +219,7 @@ DDPClient.prototype._message = function(data) {
   // minimal heartbeat response for ddp pre2
   } else if (data.msg === 'ping') {
     self._send(
-      _.has(data, "id") ? { msg : 'pong', id : data.id } : { msg : 'pong' }
+      _.has(data, 'id') ? { msg : 'pong', id : data.id } : { msg : 'pong' }
     );
   }
 };