runat-1128988: Clarify some comments

Signed-off-by: Joe Walker <jwalker@mozilla.com>

Author: joewalker

lib/gcli/system.js

@@ -290,10 +290,13 @@ function addLocalFunctions(specs, front) {
   // all the remote types
   specs.forEach(function(commandSpec) {
     // HACK: Tack the front to the command so we know how to remove it
-    // in syncItems() below
+    // in syncItems() above
     commandSpec.front = front;
 
-    // TODO: syncItems() doesn't remove types, so do we need this?
+    // Tell the type instances for a command how to contact their counterparts
+    // Don't confuse this with setting the front on the commandSpec which is
+    // about associating a proxied command with it's source for later removal.
+    // This is actually going to be used by the type
     commandSpec.params.forEach(function(param) {
       if (typeof param.type !== 'string') {
         param.type.front = front;

lib/gcli/types/delegate.js

@@ -90,7 +90,7 @@ exports.items = [
     stringify: function(value, context) {
       // remote types are client only, and we don't attempt to transfer value
       // objects to the client (we can't be sure the are jsonable) so it is a
-      // but strange to be asked to stringify a value object, however since
+      // bit strange to be asked to stringify a value object, however since
       // parse creates a Conversion with a (fake) value object we might be
       // asked to stringify that. We can stringify fake value objects.
       if (typeof value.stringified === 'string') {

lib/gcli/types/types.js

@@ -947,8 +947,8 @@ function Type() {
 
 /**
  * Get a JSONable data structure that entirely describes this type.
- * commandName and paramName are the names of the command and parameter which we
- * are remoting to help the server get back to the remoted action.
+ * commandName and paramName are the names of the command and parameter which
+ * we are remoting to help the server get back to the remoted action.
  */
 Type.prototype.getSpec = function(commandName, paramName) {
   throw new Error('Not implemented');