Cleanup todos

googleapis · Jan 13, 2016 · 67f6c67 · 67f6c67
1 parent 903cec6
commit 67f6c67
Show file tree

Hide file tree

Showing 5 changed files with 2 additions and 12 deletions.
diff --git a/lib/cls.js b/lib/cls.js
@@ -42,10 +42,9 @@ module.exports = {
   getNamespace: getNamespace,
 
   getRootContext: function getRootContext() {
-    // TODO: First getNamespace check is necessary in case any
+    // First getNamespace check is necessary in case any
     // patched closures escaped before the agent was stopped and the
-    // namespace was destroyed. There should be a cleaner way to
-    // avoid this (only collect traces once we have a project num)
+    // namespace was destroyed.
     if (getNamespace() && getNamespace().get('root')) {
       return getNamespace().get('root');
     }

diff --git a/lib/trace-labels.js b/lib/trace-labels.js
@@ -16,9 +16,6 @@
 
 'use strict';
 
-// TODO(ofrobots): replace this file with the protobuf
-// c.f. trace/proto/span_details.proto
-
 /**
  * Well-known trace span label values.
  */

diff --git a/lib/trace-span.js b/lib/trace-span.js
@@ -16,8 +16,6 @@
 
 'use strict';
 
-// TODO(ofrobots): replace this file with the protobuf
-
 /**
  * Creates a trace span object.
  * @constructor

diff --git a/lib/trace-writer.js b/lib/trace-writer.js
@@ -208,8 +208,6 @@ TraceWriter.prototype.publish_ = function(projectId, json) {
     headers: headers
   }, function(err, response, body) {
     if (err) {
-      // TODO(ofrobots): If we failed to publish due to a permanent error, stop
-      // the agent.
       that.logger_.error('TraceWriter: error: ',
         (response && response.statusCode) || '', err);
     } else {

diff --git a/lib/trace.js b/lib/trace.js
@@ -16,8 +16,6 @@
 
 'use strict';
 
-// TODO(ofrobots): replace this file with the protobuf
-
 /**
  * Creates a trace object.
  * @constructor